Рефакторинг по замечаниям Sonarqube

src/cf/CommonModules/БотОбщегоНазначения/Ext/Module.bsl

@@ -140,6 +140,7 @@
 		ДанныеКонфигурации.ТекущаяВерсия,
 		ДанныеКонфигурации.АктуальнаяВерсия);
 
+		Если РезультатСравненияВерсий <> Неопределено Тогда
 			Если РезультатСравненияВерсий < 0 Тогда
 				Диагностика = "Необходимо обновить текущую версию.";
 			ИначеЕсли РезультатСравненияВерсий > 0 Тогда
@@ -147,6 +148,9 @@
 			Иначе
 				Диагностика = "Обновление текущей версии не требуется.";
 			КонецЕсли;
+		Иначе
+			Диагностика = "";
+		КонецЕсли;
 		
 		ТекстОтвета = СтрШаблон("&#8505 %1%2<code>%3</code> &#8594 <code>%4</code>%5%6",
 		ДанныеКонфигурации.Наименование,

src/cf/CommonModules/ОбновленияРегламент/Ext/Module.bsl

@@ -24,16 +24,19 @@
 	КонфигурацииСНекорректнымиВерсиями = Новый Массив;
 	
 	Для каждого ДанныеКонфигурации Из КонфигурацииДляВывода Цикл
-		Если ДанныеКонфигурации.РезультатСравнения > 0 Тогда
+		Если ДанныеКонфигурации.РезультатСравнения = Неопределено Тогда
+			Продолжить;
+		ИначеЕсли ДанныеКонфигурации.РезультатСравнения > 0 Тогда
 			КонфигурацииСНекорректнымиВерсиями.Добавить(ДанныеКонфигурации);
 			Продолжить;
-		КонецЕсли;
-		
+		Иначе
 			ТекстСообщения = ТекстСообщения + ФорматСтрокиСообщенияОНеактуальнойВерсии("&#128293", ДанныеКонфигурации);
+		КонецЕсли;
 	КонецЦикла;
 	
 	Если КонфигурацииСНекорректнымиВерсиями.Количество() > 0 Тогда
-		ТекстСообщения = ТекстСообщения + СтрШаблон("Внимание! Текущая конфигурация старше актуальной:%1%2", Символы.ПС, Символы.ПС);
+		ТекстСообщения = ТекстСообщения + СтрШаблон("Внимание! Текущая конфигурация старше актуальной:%1%2",
+		Символы.ПС, Символы.ПС);
 		
 		Для каждого ДанныеКонфигурации Из КонфигурацииСНекорректнымиВерсиями Цикл
 			ТекстСообщения = ТекстСообщения + ФорматСтрокиСообщенияОНеактуальнойВерсии("&#128683", ДанныеКонфигурации);

src/cf/CommonModules/РаботаСОбновлениямиВызовСервера/Ext/Module.bsl

@@ -259,8 +259,10 @@
 			ВерсииКонфигурации.Вставить("КонфигурацияПредставление", ВыборкаДетальныеЗаписи.КонфигурацияПредставление);
 			ВерсииКонфигурации.Вставить("ТекущаяВерсия", ВыборкаДетальныеЗаписи.ТекущаяВерсия);
 			ВерсииКонфигурации.Вставить("АктуальнаяВерсия", ВыборкаДетальныеЗаписи.АктуальнаяВерсия);
-			ВерсииКонфигурации.Вставить("РезультатСравнения", СравнитьВерсииКонфигураций(ВыборкаДетальныеЗаписи.ТекущаяВерсия,
-				ВыборкаДетальныеЗаписи.АктуальнаяВерсия));
+			
+			РезультатСравнения = СравнитьВерсииКонфигураций(ВыборкаДетальныеЗаписи.ТекущаяВерсия,
+				ВыборкаДетальныеЗаписи.АктуальнаяВерсия);
+			ВерсииКонфигурации.Вставить("РезультатСравнения", РезультатСравнения);
 			ДанныеВерсийКонфигураций.Добавить(ВерсииКонфигурации);
 		КонецЦикла;
 		
@@ -272,9 +274,12 @@
 	
 КонецФункции
 
-
 // Возвращает результат сравнения двух версий конфигураций: текущей и актуальной.
 //
+// Параметры:
+//  ТекущаяВерсия - Строка - версия, установленная у пользователя
+//  АктуальнаяВерсия - Строка - версия, полученная с сайта вендора
+//
 // Возвращаемое значение:
 // Отрицательное число - текущая версия меньше актуальной
 // Положительное число - актуальная версия меньше текущей
@@ -282,8 +287,8 @@
 //
 Функция СравнитьВерсииКонфигураций(ТекущаяВерсия, АктуальнаяВерсия) Экспорт
 	
-	НаборГруппТекущаяВерсия = СтрРазделить(ТекущаяВерсия,".");
-	НаборГруппАктуальнаяВерсия = СтрРазделить(АктуальнаяВерсия,".");
+	НаборГруппТекущаяВерсия = СтрРазделить(ТекущаяВерсия, ".");
+	НаборГруппАктуальнаяВерсия = СтрРазделить(АктуальнаяВерсия, ".");
 	
 	Если НаборГруппТекущаяВерсия.Количество() = 0 ИЛИ НаборГруппАктуальнаяВерсия.Количество() = 0 Тогда
 		Возврат Неопределено;

src/cf/VERSION

@@ -1,2 +1,2 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<VERSION>43</VERSION>
+<VERSION>44</VERSION>