Fixed smells

firstBitSportivnaya · Oct 10, 2024 · 86f9662 · 86f9662
1 parent 00a86f6
commit 86f9662
Show file tree

Hide file tree

Showing 11 changed files with 24 additions and 34 deletions.
diff --git a/src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl b/src/cf/Catalogs/пбп_ИнтеграционныеПотоки/Ext/ObjectModule.bsl
@@ -73,7 +73,7 @@
 	КонецЕсли;
 
 	Если Не пбп_ОбщегоНазначенияСлужебныйКлиентСервер.СвойствоСтруктуры(ДополнительныеСвойства, "ЭтоНовый", Ложь) Тогда
-		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(ЭтотОбъект.Ссылка, ЭтотОбъект.ЭтоГруппа);
+		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(Ссылка, ЭтоГруппа);
 	КонецЕсли;
 
 КонецПроцедуры

diff --git a/src/cf/Catalogs/пбп_ИнтегрируемыеСистемы/Ext/ObjectModule.bsl b/src/cf/Catalogs/пбп_ИнтегрируемыеСистемы/Ext/ObjectModule.bsl
@@ -19,7 +19,7 @@
 	КонецЕсли;
 
 	Если Не пбп_ОбщегоНазначенияСлужебныйКлиентСервер.СвойствоСтруктуры(ДополнительныеСвойства, "ЭтоНовый", Ложь) Тогда
-		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(ЭтотОбъект.Ссылка, ЭтотОбъект.ЭтоГруппа);
+		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(Ссылка, ЭтоГруппа);
 	КонецЕсли;
 
 КонецПроцедуры

diff --git a/src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl b/src/cf/Catalogs/пбп_НастройкиИнтеграции/Ext/ObjectModule.bsl
@@ -125,7 +125,7 @@
 	КонецЕсли;
 
 	Если Не пбп_ОбщегоНазначенияСлужебныйКлиентСервер.СвойствоСтруктуры(ДополнительныеСвойства, "ЭтоНовый", Ложь) Тогда
-		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(ЭтотОбъект.Ссылка, ЭтотОбъект.ЭтоГруппа);
+		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(Ссылка, ЭтоГруппа);
 	КонецЕсли;
 
 КонецПроцедуры

diff --git a/src/cf/Catalogs/пбп_ТипСоответствияОбъектовИБ/Ext/ObjectModule.bsl b/src/cf/Catalogs/пбп_ТипСоответствияОбъектовИБ/Ext/ObjectModule.bsl
@@ -20,7 +20,7 @@
 	КонецЕсли;
 
 	Если Не пбп_ОбщегоНазначенияСлужебныйКлиентСервер.СвойствоСтруктуры(ДополнительныеСвойства, "ЭтоНовый", Ложь) Тогда
-		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(ЭтотОбъект.Ссылка, ЭтотОбъект.ЭтоГруппа);
+		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(Ссылка, ЭтоГруппа);
 	КонецЕсли;
 
 КонецПроцедуры

diff --git a/src/cf/ChartsOfCharacteristicTypes/пбп_ПредопределенныеЗначения/Ext/ObjectModule.bsl b/src/cf/ChartsOfCharacteristicTypes/пбп_ПредопределенныеЗначения/Ext/ObjectModule.bsl
@@ -20,13 +20,11 @@
 	КонецЕсли;
 
 	Если Не пбп_ОбщегоНазначенияСлужебныйКлиентСервер.СвойствоСтруктуры(ДополнительныеСвойства, "ЭтоНовый", Ложь) Тогда
-		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(ЭтотОбъект.Ссылка, ЭтотОбъект.ЭтоГруппа);
+		РегистрыСведений.пбп_СостоянияПредопределенныхЭлементов.ОбновитьХешЭлемента(Ссылка, ЭтоГруппа);
 	КонецЕсли;
 
 КонецПроцедуры
 
 #КонецОбласти
 
-#КонецЕсли
-
-
+#КонецЕсли
diff --git a/...sOfCharacteristicTypes/пбп_ПредопределенныеЗначения/Forms/ФормаСписка/Ext/Form/Module.bsl b/...sOfCharacteristicTypes/пбп_ПредопределенныеЗначения/Forms/ФормаСписка/Ext/Form/Module.bsl
@@ -29,7 +29,3 @@
 КонецПроцедуры
 
 #КонецОбласти
-
-#Область СлужебныеПроцедурыИФункции
-
-#КонецОбласти
diff --git a/...cf/CommonForms/пбп_ФормаРазрешенияКонфликтовПредопределенныхЭлементов/Ext/Form/Module.bsl b/...cf/CommonForms/пбп_ФормаРазрешенияКонфликтовПредопределенныхЭлементов/Ext/Form/Module.bsl
@@ -1,8 +1,4 @@
 
-#Область ОписаниеПеременных
-
-#КонецОбласти
-
 #Область ОбработчикиСобытийФормы
 
 &НаСервере
@@ -33,7 +29,7 @@
 	КонецЕсли;
 
 КонецПроцедуры
-	
+
 #КонецОбласти
 
 #Область ОбработчикиКомандФормы
@@ -159,16 +155,17 @@
 		Возврат;
 	КонецЕсли;
 
+	ИдентификаторНастройки = "ИдентификаторНастройки";
 	Таблица = ПолучитьИзВременногоХранилища(АдресТаблицы);
 
 	Если Не ЗначениеЗаполнено(Таблица) Тогда
 		Возврат;
 	КонецЕсли;
 
 	УдалитьИзВременногоХранилища(АдресТаблицы);
-	Таблица.Индексы.Добавить("ИдентификаторНастройки");
+	Таблица.Индексы.Добавить(ИдентификаторНастройки);
 	Для Каждого Строка Из ТаблицаКонфликтов Цикл
-		Стр = Таблица.Найти(Строка[ПрефиксКод + "ИдентификаторНастройки"], "ИдентификаторНастройки");
+		Стр = Таблица.Найти(Строка[ПрефиксКод + ИдентификаторНастройки], ИдентификаторНастройки);
 		Если Строка.ЗаменитьЭлементомИзКода Тогда
 			Стр.Служеб_ОбновитьЭлемент = Истина;
 		Иначе
@@ -184,9 +181,3 @@
 КонецПроцедуры
 
 #КонецОбласти
-
-#Область Инициализация
-
-#КонецОбласти
-
-
diff --git a/src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПредопределенныеЗначения/Ext/Module.bsl
@@ -56,7 +56,7 @@
 // на неиспользуемые элементы, отбор предопределенных значений, создание предопределенных элементов.
 //
 // Параметры:
-//  ТаблицаПредопределенных - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных.
+//  ТаблицаПредопределенных - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных
 //  МенеджерОбъекта			- СправочникМенеджер, ПланВидовХарактеристикМенеджер - менеджер объекта.
 //
 Функция ОбработатьПредопределенныеЗначения(ТаблицаПредопределенных, МенеджерОбъекта) Экспорт
@@ -79,7 +79,7 @@
 // При установке пометки удаления для групп, также устанавливается для всех подчиенных элементов.
 //
 // Параметры:
-//  ТаблицаПредопределенных - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных.
+//  ТаблицаПредопределенных - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных
 //  ПолноеИмяОбъекта		- Строка - Полное имя объекта метаданных.
 //
 Процедура ОбработатьНеиспользуемыеЭлементы(ТаблицаПредопределенных, ПолноеИмяОбъекта = "") Экспорт
@@ -125,7 +125,7 @@
 // для создания предопределенных элементов.
 //
 // Параметры:
-//  ТаблицаПредопределенных - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных.
+//  ТаблицаПредопределенных - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных
 //  ПолноеИмяОбъекта		- Строка - Полное имя объекта метаданных.
 //
 Функция УдалитьИзТаблицыСуществующиеЭлементы(ТаблицаПредопределенных, ПолноеИмяОбъекта = "") Экспорт
@@ -242,7 +242,7 @@
 		КлючиХеша = ?(ЭтоГруппа, ДополнительныеПараметры.КлючиХешаГруппа, ДополнительныеПараметры.КлючиХешаЭлемент);
 
 		Если СтрокаТаблицы.Служеб_ОбновитьЭлемент Тогда
-			ОбновитьЭлемент(МенеджерОбъекта, СтрокаТаблицы, КлючиХеша, Группы, ЭтоГруппа, КолонкаСуществует)
+			ОбновитьЭлемент(МенеджерОбъекта, СтрокаТаблицы, КлючиХеша, Группы, ЭтоГруппа, КолонкаСуществует);
 		ИначеЕсли СтрокаТаблицы.Служеб_УстановитьФлагРучноеИзменение Тогда
 			УстановитьФлагРучноеИзменение(СтрокаТаблицы.Служеб_ПредопределенныйЭлемент);
 		Иначе
@@ -255,10 +255,11 @@
 // Подготавливает параметры необходимые для обработки предопределенных элементов
 //
 // Параметры:
-//  МенеджерОбъекта - СправочникМенеджер, ПланВидовХарактеристикМенеджер - менеджер объекта.
+//  Таблица			 - ТаблицаЗначений - см. пбп_ПредопределенныеЗначенияПереопределяемый.ТаблицаПредопределенных.
+//  МенеджерОбъекта	 - СправочникМенеджер, ПланВидовХарактеристикМенеджер	 - менеджер объекта.
 // 
 // Возвращаемое значение:
-//  Структура 	- Данные Менеджера объекта
+//  Структура - Данные Менеджера объекта
 //  * Менеджер 	- СправочникМенеджер, ПланВидовХарактеристикМенеджер - менеджер объекта.
 //  * ПолноеИмя - Строка - Полное имя объекта метаданных.
 //

diff --git a/src/cf/CommonModules/пбп_ПредопределенныеЗначенияКлиент/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПредопределенныеЗначенияКлиент/Ext/Module.bsl
@@ -23,6 +23,11 @@
 
 #Область ПрограммныйИнтерфейс
 
+// Создание предопределенных значений на основании заполненной таблицы
+//
+// Параметры:
+//  Форма - ФормаКлиентскогоПриложения - Форма из которой запущена обработка обновление предопределенных элементов
+//
 Процедура СоздатьОбновитьПредопределенныеЗначения(Форма) Экспорт
 
 	// Возвращаем адрес временного хранилища на таблицу и есть ли конфликты

diff --git a/src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПредопределенныеЗначенияПереопределяемый/Ext/Module.bsl
@@ -373,7 +373,7 @@
 Процедура СоздатьКолонкиТаблицыПредопределенныхЭлементов(СтруктураСКолонками, Таблица)
 
 	Для Каждого КлючЗначение Из СтруктураСКолонками Цикл
-		Если Не Таблица.Колонки.Найти(КлючЗначение.Ключ) = Неопределено Тогда
+		Если Таблица.Колонки.Найти(КлючЗначение.Ключ) <> Неопределено Тогда
 			Продолжить;
 		КонецЕсли;
 

diff --git a/src/cf/InformationRegisters/пбп_СостоянияПредопределенныхЭлементов/Ext/ManagerModule.bsl b/src/cf/InformationRegisters/пбп_СостоянияПредопределенныхЭлементов/Ext/ManagerModule.bsl
@@ -75,5 +75,4 @@
 
 #КонецОбласти
 
-#КонецЕсли
-
+#КонецЕсли
Original file line number	Diff line number	Diff line change
Expand Up		@@ -75,5 +75,4 @@

		#КонецОбласти

		#КонецЕсли

		#КонецЕсли