fixed some smells

firstBitSportivnaya · Nov 29, 2024 · bbcd552 · bbcd552
1 parent 5c73d2b
commit bbcd552
Show file tree

Hide file tree

Showing 10 changed files with 95 additions and 28 deletions.
diff --git a/src/cf/Catalogs/пбп_ПользовательскиеФункции/Ext/ManagerModule.bsl b/src/cf/Catalogs/пбп_ПользовательскиеФункции/Ext/ManagerModule.bsl
@@ -130,11 +130,11 @@
 				СправочникСсылка = Неопределено;
 
 				Если ВыводитьОшибки Тогда
+					ШаблонСообщения = НСтр("ru='Не удалось найти предопределенный элемент справочника ""Пользовательские функции"" по имени %1';
+						|en='Failed to find a predefined element of the directory ""User-defined functions"" named %1'");
+					ТекстСообщения = СтрШаблон(ШаблонСообщения, ОписаниеЭлемента.ИмяПредопределенного);
 
-					пбп_Переадресация.СообщитьПользователю(СтрШаблон(
-						НСтр("ru='Не удалось найти предопределенный элемент справочника ""Пользовательские функции"" по имени %1';
-						|en='Failed to find a predefined element of the directory ""User-defined functions"" named %1'"),
-						ОписаниеЭлемента.ИмяПредопределенного));
+					пбп_Переадресация.СообщитьПользователю(ТекстСообщения);
 				КонецЕсли;
 
 			КонецПопытки;
@@ -148,11 +148,12 @@
 	КонецЦикла;
 
 	Если ВыводитьИнформацию Тогда
-
-		пбп_Переадресация.СообщитьПользователю(СтрШаблон(
+		ШаблонСообщения =
 			НСтр("ru='Выполнена проверка версий предопределенных элементов справочника ""Пользовательские функции"". Обновлено %1 элементов.';
-			|en='Successfully checked the versions of predefined elements of the directory ""User functions"". %1 elements updated.'"),
-			Строка(Ном)));
+			|en='Successfully checked the versions of predefined elements of the directory ""User functions"". %1 elements updated.'");
+		ТекстСообщения = СтрШаблон(ШаблонСообщения, Строка(Ном));
+
+		пбп_Переадресация.СообщитьПользователю(ТекстСообщения);
 	КонецЕсли;
 
 КонецПроцедуры

diff --git a/src/cf/Catalogs/пбп_ПользовательскиеФункции/Forms/ФормаСтруктуры/Ext/Form/Module.bsl b/src/cf/Catalogs/пбп_ПользовательскиеФункции/Forms/ФормаСтруктуры/Ext/Form/Module.bsl
@@ -58,7 +58,7 @@
 	ТекДанные = Элементы.Структура.ТекущиеДанные;
 	Если ЗначениеЗаполнено(ТекДанные.Ключ) Тогда
 		Результат = СтрЗаменить(ТекДанные.Ключ, " ", "");
-		Пока пбп_ПереадресацияКлиентСервер.ЭтоЧисло(Сред(Результат,1,1)) Цикл
+		Пока пбп_ПереадресацияКлиентСервер.ЭтоЧисло(Сред(Результат, 1, 1)) Цикл
 			Результат = Прав(Результат, СтрДлина(Результат) - 1);
 		КонецЦикла;
 		пбп_ОбщегоНазначенияКлиентСервер.УдалитьНедопустимыеСимволы(Результат);

diff --git a/src/cf/CommonModules/пбп_МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl b/src/cf/CommonModules/пбп_МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl
@@ -566,24 +566,24 @@
 Функция ПолучитьИмяМетода(Форма, ИмяПроцедурыВход)
 
 	ИмяФормы = Форма.ИмяФормы;
-	МассивПодимен = пбп_ПереадресацияКлиентСервер.РазложитьСтрокуВМассивПодстрок(ИмяФормы, ".");
+	ЧастиИмениФормы = пбп_ПереадресацияКлиентСервер.РазложитьСтрокуВМассивПодстрок(ИмяФормы, ".");
 
-	КолЭлементов = МассивПодимен.Количество();
+	КолЭлементов = ЧастиИмениФормы.Количество();
 	Если КолЭлементов = 0 Тогда
 		Возврат Неопределено;
 	КонецЕсли; 
 
-	ПостфиксМодуля = МассивПодимен.Получить(0);
+	ПостфиксМодуля = ЧастиИмениФормы.Получить(0);
 
 	// Удалим класс
-	МассивПодимен.Удалить(0);
+	ЧастиИмениФормы.Удалить(0);
 	Если КолЭлементов > 2 Тогда
 		// Удалим слово "Форма"
-		МассивПодимен.Удалить(1);
+		ЧастиИмениФормы.Удалить(1);
 	КонецЕсли;
 
 	ИмяМодуля = "пбп_МодификацияКонфигурацииПереопределяемый" + ПостфиксМодуля;
-	ИмяПроцедуры = СтрСоединить(МассивПодимен, "") + ИмяПроцедурыВход;
+	ИмяПроцедуры = СтрСоединить(ЧастиИмениФормы, "") + ИмяПроцедурыВход;
 	ИмяМетода = ИмяМодуля + "." + ИмяПроцедуры;
 
 	Возврат ИмяМетода;

diff --git a/src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl b/src/cf/CommonModules/пбп_ОбщегоНазначенияСервер/Ext/Module.bsl
@@ -307,7 +307,7 @@
 		КонецЕсли;
 
 		ЗаписьЖурналаРегистрации("пбп_.ОшибкаЗаписиСправочника", УровеньЖурналаРегистрации.Ошибка,,,
-			ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+			ОбработкаОшибок.ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
 	КонецПопытки;
 
 	Возврат СправочникЗаписан;

diff --git a/src/cf/CommonModules/пбп_Переадресация/Ext/Module.bsl b/src/cf/CommonModules/пбп_Переадресация/Ext/Module.bsl
@@ -21,6 +21,17 @@
 
 #Область ПрограммныйИнтерфейс
 
+// Функция ОпределитьМодульПереадресации предназначена для проверки наличия общего модуля
+// БСП с указанным именем и получения этого модуля, если он существует.
+//
+// Параметры:
+//  ИмяМодуля	 - Строка - имя общего модуля, который необходимо найти.
+// 
+// Возвращаемое значение:
+//  Структура - общий модуль и что он существует
+//  * Модуль 	- ОбщийМодуль - общий модуль из БСП.
+//  * МодульСуществует - Булево - Если флаг истина, модуль существует.
+//
 Функция ОпределитьМодульПереадресации(ИмяМодуля) Экспорт
 
 	Результат = Новый Структура("Модуль, МодульСуществует", , Ложь);
@@ -69,7 +80,8 @@
 		ИмяСобытия = "Событие"; // не локализуется, чтобы не допускать остановки запуска в частично переведенной конфигурации
 	КонецЕсли;
 
-	ЗаписьЖурналаРегистрации(ИмяСобытия, Уровень, ОбъектМетаданных, Данные, Комментарий, РежимТранзакцииЗаписиЖурналаРегистрации.Независимая);
+	ЗаписьЖурналаРегистрации(ИмяСобытия, Уровень,
+		ОбъектМетаданных, Данные, Комментарий, РежимТранзакцииЗаписиЖурналаРегистрации.Независимая);
 
 КонецПроцедуры
 
@@ -1684,7 +1696,7 @@
 //
 Процедура УдалитьВременныйФайл(Знач Путь) Экспорт
 
-	Результат = ПереадресацияМодуляОбщегоНазначения();
+	Результат = ПереадресацияМодуляФайловаяСистема();
 	Если Результат.МодульСуществует Тогда
 		Результат.Модуль.УдалитьВременныйФайл(Путь);
 		Возврат;
@@ -1726,7 +1738,7 @@
 //
 Функция ПараметрыЗапускаПрограммы() Экспорт
 
-	Результат = ПереадресацияМодуляОбщегоНазначения();
+	Результат = ПереадресацияМодуляФайловаяСистема();
 	Если Результат.МодульСуществует Тогда
 		Возврат Результат.Модуль.ПараметрыЗапускаПрограммы();
 	КонецЕсли;
@@ -1798,7 +1810,7 @@
 //
 Функция ЗапуститьПрограмму(Знач КомандаЗапуска, ПараметрыЗапускаПрограммы = Неопределено) Экспорт 
 
-	Результат = ПереадресацияМодуляОбщегоНазначения();
+	Результат = ПереадресацияМодуляФайловаяСистема();
 	Если Результат.МодульСуществует Тогда
 		Возврат Результат.Модуль.ЗапуститьПрограмму(КомандаЗапуска, ПараметрыЗапускаПрограммы);
 	КонецЕсли;
@@ -1898,7 +1910,6 @@
 
 #КонецОбласти
 
-
 #КонецОбласти
 
 #Область СерверныеОповещения
@@ -2394,6 +2405,10 @@
 	Возврат пбп_ПереадресацияПовтИсп.ОпределитьМодульПереадресации("ПолучениеФайловИзИнтернета");
 КонецФункции
 
+Функция ПереадресацияМодуляФайловаяСистема()
+	Возврат пбп_ПереадресацияПовтИсп.ОпределитьМодульПереадресации("ФайловаяСистема");
+КонецФункции
+
 #КонецОбласти
 
 #Область ПереадресацияМетодов

diff --git a/src/cf/CommonModules/пбп_ПереадресацияКлиент/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПереадресацияКлиент/Ext/Module.bsl
@@ -21,6 +21,17 @@
 
 #Область ПрограммныйИнтерфейс
 
+// Функция ОпределитьМодульПереадресации предназначена для проверки наличия общего модуля
+// БСП с указанным именем и получения этого модуля, если он существует.
+//
+// Параметры:
+//  ИмяМодуля	 - Строка - имя общего модуля, который необходимо найти.
+// 
+// Возвращаемое значение:
+//  Структура - общий модуль и что он существует
+//  * Модуль 	- ОбщийМодуль - общий модуль из БСП.
+//  * МодульСуществует - Булево - Если флаг истина, модуль существует.
+//
 Функция ОпределитьМодульПереадресации(ИмяМодуля) Экспорт
 
 	Результат = Новый Структура("Модуль, МодульСуществует", , Ложь);

diff --git a/src/cf/CommonModules/пбп_ПереадресацияКлиентПовтИсп/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПереадресацияКлиентПовтИсп/Ext/Module.bsl
@@ -19,8 +19,19 @@
 // URL: https://github.com/firstBitSportivnaya/PSSL/
 ///////////////////////////////////////////////////////////////////////////////////////////////////////
 
-#Область ПрограммныйИнтерфейс
+#Область СлужебныйПрограммныйИнтерфейс
 
+// Функция ОпределитьМодульПереадресации предназначена для проверки наличия общего модуля
+// БСП с указанным именем и получения этого модуля, если он существует.
+//
+// Параметры:
+//  ИмяМодуля	 - Строка - имя общего модуля, который необходимо найти.
+// 
+// Возвращаемое значение:
+//  Структура - общий модуль и что он существует
+//  * Модуль 	- ОбщийМодуль - общий модуль из БСП.
+//  * МодульСуществует - Булево - Если флаг истина, модуль существует.
+//
 Функция ОпределитьМодульПереадресации(ИмяМодуля) Экспорт
 
 	Возврат пбп_ПереадресацияКлиент.ОпределитьМодульПереадресации(ИмяМодуля);

diff --git a/src/cf/CommonModules/пбп_ПереадресацияКлиентСервер/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПереадресацияКлиентСервер/Ext/Module.bsl
@@ -21,6 +21,17 @@
 
 #Область ПрограммныйИнтерфейс
 
+// Функция ОпределитьМодульПереадресации предназначена для проверки наличия общего модуля
+// БСП с указанным именем и получения этого модуля, если он существует.
+//
+// Параметры:
+//  ИмяМодуля	 - Строка - имя общего модуля, который необходимо найти.
+// 
+// Возвращаемое значение:
+//  Структура - общий модуль и что он существует
+//  * Модуль 	- ОбщийМодуль - общий модуль из БСП.
+//  * МодульСуществует - Булево - Если флаг истина, модуль существует.
+//
 Функция ОпределитьМодульПереадресации(ИмяМодуля) Экспорт
 
 #Если Сервер Или ТолстыйКлиентОбычноеПриложение Или ВнешнееСоединение Тогда
@@ -371,7 +382,7 @@
 
 		Результат = СтрРазделить(Значение, Разделитель, Ложь);
 		Для Индекс = 0 По Результат.ВГраница() Цикл
-			Результат[Индекс] = СокрЛП(Результат[Индекс])
+			Результат[Индекс] = СокрЛП(Результат[Индекс]);
 		КонецЦикла;
 		Возврат Результат;
 
@@ -520,7 +531,7 @@
 
 #КонецОбласти
 
-#Область Пользователи
+#Область ПользователиКлиентСервер
 
 // Аналог метода БСП. Только для внутреннего использования.
 Функция ТекущийПользователь(АвторизованныйПользователь) Экспорт

diff --git a/src/cf/CommonModules/пбп_ПереадресацияПовтИсп/Ext/Module.bsl b/src/cf/CommonModules/пбп_ПереадресацияПовтИсп/Ext/Module.bsl
@@ -19,8 +19,19 @@
 // URL: https://github.com/firstBitSportivnaya/PSSL/
 ///////////////////////////////////////////////////////////////////////////////////////////////////////
 
-#Область ПрограммныйИнтерфейс
+#Область СлужебныйПрограммныйИнтерфейс
 
+// Функция ОпределитьМодульПереадресации предназначена для проверки наличия общего модуля
+// БСП с указанным именем и получения этого модуля, если он существует.
+//
+// Параметры:
+//  ИмяМодуля	 - Строка - имя общего модуля, который необходимо найти.
+// 
+// Возвращаемое значение:
+//  Структура - общий модуль и что он существует
+//  * Модуль 	- ОбщийМодуль - общий модуль из БСП.
+//  * МодульСуществует - Булево - Если флаг истина, модуль существует.
+//
 Функция ОпределитьМодульПереадресации(ИмяМодуля) Экспорт
 
 	Возврат пбп_Переадресация.ОпределитьМодульПереадресации(ИмяМодуля);
@@ -75,6 +86,11 @@
 //
 Функция ОписаниеТипаВсеСсылки() Экспорт
 
+	Результат = ПереадресацияМодуляСтандартныеПодсистемыПовтИсп();
+	Если Результат.МодульСуществует Тогда
+		Возврат Результат.Модуль.ОписаниеТипаВсеСсылки();
+	КонецЕсли;
+
 	Возврат Новый ОписаниеТипов(Новый ОписаниеТипов(Новый ОписаниеТипов(Новый ОписаниеТипов(Новый ОписаниеТипов(
 		Новый ОписаниеТипов(Новый ОписаниеТипов(Новый ОписаниеТипов(Новый ОписаниеТипов(
 			Справочники.ТипВсеСсылки(),

diff --git a/src/cf/Documents/ДемоРаботаСФормами/Forms/ФормаДокумента/Ext/Form/Module.bsl b/src/cf/Documents/ДемоРаботаСФормами/Forms/ФормаДокумента/Ext/Form/Module.bsl
@@ -54,9 +54,11 @@
 &НаКлиенте
 Процедура ПодсказатьКогдаОбед(Команда)
 
-	пбп_ПереадресацияКлиент.СообщитьПользователю(СтрШаблон(
-		НСтр("ru='Привет, текущее время %1'; en='Hello, the current time is %1'"),
-		Формат(пбп_ОбщегоНазначенияВызовСервера.ТекущаяДатаПользователя(), "ДЛФ=DT")));
+	ШаблонСообщения = НСтр("ru='Привет, текущее время %1'; en='Hello, the current time is %1'");
+	ТекстСообщения = СтрШаблон(ШаблонСообщения,
+		Формат(пбп_ОбщегоНазначенияВызовСервера.ТекущаяДатаПользователя(), "ДЛФ=DT"));
+
+	пбп_ПереадресацияКлиент.СообщитьПользователю(ТекстСообщения);
 
 КонецПроцедуры