fix: Исправления по замечаниям Sonar

arkuznetsov · Feb 4, 2022 · d2c3cca · d2c3cca
1 parent 5e85784
commit d2c3cca
Show file tree

Hide file tree

Showing 11 changed files with 29 additions and 30 deletions.
diff --git a/src/Классы/ВыгрузкаКаталогаКонфигурацииВГит.os b/src/Классы/ВыгрузкаКаталогаКонфигурацииВГит.os
@@ -380,11 +380,12 @@
 		ДатаКоммита = Служебный.ДатаPOSIX(ТекОписание.Дата);
 
 		ВерсииДляОбновления = СтрСоединить(ТекОписание.ВерсииДляОбновления, ", ");
-		СообщениеКоммита = СтрШаблон("Обновление версии конфигурации поставщика на %1%4%4Страница релиза:%4https://releases.1c.ru%2%4Версии для обновления: %3",
-			ТекОписание.Версия,
-			ТекОписание.Путь,
-			ВерсииДляОбновления,
-			Символы.ПС);
+		СообщениеКоммита = СтрШаблон("Обновление версии конфигурации поставщика на %1%4%4
+		                             |Страница релиза:%4https://releases.1c.ru%2%4Версии для обновления: %3",
+		                             ТекОписание.Версия,
+		                             ТекОписание.Путь,
+		                             ВерсииДляОбновления,
+		                             Символы.ПС);
 
 		Выгрузка = Новый ВыгрузкаКонфигурацииВГит();
 		Выгрузка.УстановитьПараметрОбработкиДанных("ВерсияПлатформы"        , ВерсияПлатформы);

diff --git a/src/Классы/ВыгрузкаКонфигурацииВГит.os b/src/Классы/ВыгрузкаКонфигурацииВГит.os
@@ -436,7 +436,7 @@
 
 	ГитРепозиторий.УстановитьНастройку("user.name", ИмяАвтора);
 	ГитРепозиторий.УстановитьНастройку("user.email", ПочтаАвтора);
-	ПредставлениеАвтора = ИмяАвтора + " <" + ПочтаАвтора + ">";
+	ПредставлениеАвтора = СтрШаблон("%1 <%2>", ИмяАвтора, ПочтаАвтора);
 	ГитРепозиторий.Закоммитить(СообщениеКоммита, Истина, , ПредставлениеАвтора, ДатаКоммита, , ДатаКоммита);
 
 	Лог.Информация("[%1]: Помещение изменений в Git завершено", ТипЗнч(ЭтотОбъект));

diff --git a/src/Классы/ЗагрузчикРелизов1С.os b/src/Классы/ЗагрузчикРелизов1С.os
@@ -864,7 +864,7 @@
 			РаспаковщикРелиза.ОбработатьДанные();
 
 			Если УдалитьПослеРаспаковкиEFD Тогда
-				УдалитьФайлы(ФайлАрхива.Путь, "*.*");
+				УдалитьФайлы(ФайлАрхива.Путь, ПолучитьМаскуВсеФайлы());
 				УдалитьФайлы(ФайлАрхива.Путь);
 				Лог.Информация("[%1]: Удален дистрибутив ""%2""", СокрЛП(ЭтотОбъект), ФайлАрхива.Путь);
 			КонецЕсли;

diff --git a/src/Классы/МенеджерОбработкиДанных.os b/src/Классы/МенеджерОбработкиДанных.os
@@ -194,7 +194,7 @@
 	УстановитьПараметрыОбработчика(ОписаниеОбработчика);
 
 	Если ОписаниеОбработчика.Свойство("ПроцедураОбработки") Тогда
-		Выполнить("ОписаниеОбработчика.Обработка." + ОписаниеОбработчика.ПроцедураОбработки + "()");
+		Выполнить(СтрШаблон("ОписаниеОбработчика.Обработка.%1()", ОписаниеОбработчика.ПроцедураОбработки));
 	Иначе
 		ОписаниеОбработчика.Обработка.ОбработатьДанные();
 	КонецЕсли;

diff --git a/src/Классы/ОбозревательСайта1С.os b/src/Классы/ОбозревательСайта1С.os
@@ -469,14 +469,12 @@
 
 	Совпадения = Служебный.НайтиСовпаденияВТексте(Текст, ШаблонПоискаСтрокиРегистрации);
 
-	execution = "";
+	Токен = "";
 	Если Совпадения.Количество() > 0 Тогда
-		execution = Совпадения[0].Группы[1].Значение;
+		Токен = Совпадения[0].Группы[1].Значение;
 	КонецЕсли;
 
-	СтрокаРегистрации = СтрШаблон(ШаблонСтрокиРегистрации, Имя, Пароль, execution);
-
-	Возврат СтрокаРегистрации;
+	Возврат СтрШаблон(ШаблонСтрокиРегистрации, Имя, Пароль, Токен);
 
 КонецФункции // ПолучитьСтрокуРегистрации()
 

diff --git a/src/Классы/СборщикКонфигураций1СВКаталоге.os b/src/Классы/СборщикКонфигураций1СВКаталоге.os
@@ -256,7 +256,7 @@
 		               Формат(ТекОписание.Дата, "ДФ=dd.MM.yyyy; ДП=-"),
 		               ТекОписание.Имя);
 
-		НайденаКонфигурацияДляОбновления = Ложь;
+		КонфигурацияДляОбновления = Неопределено;
 
 		Для Каждого ТекВерсия Из ТекОписание.ВерсииДляОбновления Цикл
 
@@ -265,21 +265,21 @@
 
 			ФайлКонфигурации = Новый Файл(ОбъединитьПути(КаталогОбновляемойВерсии, "1cv8.cf"));
 			Если ФайлКонфигурации.Существует() Тогда
-				НайденаКонфигурацияДляОбновления = Истина;
+				КонфигурацияДляОбновления = ТекВерсия;
 				Лог.Отладка("[%1]: Найдена версия %2 конфигурации %3 для обновления на версию %4 (%5).",
 				            СокрЛП(ЭтотОбъект),
-				            ТекВерсия,
+				            КонфигурацияДляОбновления,
 				            ТекОписание.Имя,
 				            ТекОписание.Версия,
 				            Формат(ТекОписание.Дата, "ДФ=dd.MM.yyyy; ДП=-"));
 				Прервать;
 			КонецЕсли;
 		КонецЦикла;
-		Если НайденаКонфигурацияДляОбновления Тогда
+		Если НЕ КонфигурацияДляОбновления = Неопределено Тогда
 
 			Лог.Информация("[%1]: Начало обновления %2 конфигурации %3 на версию %4 (%5).",
 			              СокрЛП(ЭтотОбъект),
-			              ТекВерсия,
+			              КонфигурацияДляОбновления,
 			              ТекОписание.Имя,
 			              ТекОписание.Версия,
 			              Формат(ТекОписание.Дата, "ДФ=dd.MM.yyyy; ДП=-"));

diff --git a/src/Модули/ПараметрыПриложения.os b/src/Модули/ПараметрыПриложения.os
@@ -126,7 +126,6 @@
 
 	Если ЛогПриложения = Неопределено Тогда
 		ЛогПриложения = Логирование.ПолучитьЛог(ИмяЛогаПриложения());
-
 	КонецЕсли;
 
 	Возврат ЛогПриложения;

diff --git a/src/Модули/Перечисления.os b/src/Модули/Перечисления.os
@@ -9,7 +9,7 @@
 
 #Использовать "../Макеты"
 
-Перем ВариантыРаспаковкиКаталогов Экспорт;
+Перем ВариантыРаспаковкиКаталогов Экспорт;  // Структура - перечисление вариантов распаковки каталога
 
 // Функция - возвращает значение перечисления по полному текстовому представлению
 // вида: <имя перечисления>.<значение перечисления>

diff --git a/src/Модули/Распаковщик.os b/src/Модули/Распаковщик.os
@@ -12,7 +12,7 @@
 //
 // ----------------------------------------------------------
 
-Перем Лог;
+Перем Лог;     // Объект       - объект записи лога приложения
 
 #Область ПрограммныйИнтерфейс
 
@@ -312,11 +312,8 @@
 	Р2 = 2; // смещение 2 байта
 	Р4 = 4; // смещение 4 байта
 
-	РазмерLocalFileHeader = Р4 + Р2 + Р2 + Р2 + Р2 + Р2 + Р4 + Р4 + Р4 + Р2 + Р2
-	                      + РазмерИмениИсходногоФайла + Р0;
+	Возврат Р4 + Р2 + Р2 + Р2 + Р2 + Р2 + Р4 + Р4 + Р4 + Р2 + Р2 + РазмерИмениИсходногоФайла + Р0;
 
-	Возврат РазмерLocalFileHeader;
-
 КонецФункции // ПолучитьРазмерЗаголовкаZIPФайла()
 
 // Функция - вычисляет размер окончания ZIP-файла в байтах

diff --git a/src/Модули/Служебный.os b/src/Модули/Служебный.os
@@ -201,9 +201,13 @@
 //
 Функция ДатаPOSIX(Знач Дата) Экспорт
 
-	Возврат "" + Год(Дата) + "-" + ФорматДвузначноеЧисло(Месяц(Дата)) + "-" + ФорматДвузначноеЧисло(День(Дата)) + " "
-	+ ФорматДвузначноеЧисло(Час(Дата)) + ":" + ФорматДвузначноеЧисло(Минута(Дата))
-	+ ":" + ФорматДвузначноеЧисло(Секунда(Дата));
+	Возврат СтрШаблон("%1-%2-%3 %4:%5:%6",
+	                  Год(Дата),
+	                  ФорматДвузначноеЧисло(Месяц(Дата)),
+	                  ФорматДвузначноеЧисло(День(Дата)),
+	                  ФорматДвузначноеЧисло(Час(Дата)),
+	                  ФорматДвузначноеЧисло(Минута(Дата)),
+	                  ФорматДвузначноеЧисло(Секунда(Дата)));
 
 КонецФункции // ДатаPOSIX()
 
@@ -302,7 +306,7 @@
 		Возврат;
 	КонецЕсли;
 
-	ФайлыВКаталоге = НайтиФайлы(ПутьККаталогу, "*.*");
+	ФайлыВКаталоге = НайтиФайлы(ПутьККаталогу, ПолучитьМаскуВсеФайлы());
 	Если ФайлыВКаталоге.Количество() > 0 Тогда
 		Возврат;
 	КонецЕсли;

diff --git a/tests/yard-test.os b/tests/yard-test.os
@@ -15,7 +15,7 @@
 Перем ЮнитТест;
 Перем ВременныйКаталог;
 
-Перем Лог;
+Перем Лог;     // Объект       - объект записи лога приложения
 
 // Процедура выполняется после запуска теста
 //