Оптимизация логики получения УправленияКонфигуратором

Author: khorevaa

src/core/Классы/МенеджерСинхронизации.os

@@ -23,7 +23,7 @@
 
 Перем ХранилищеКонфигурации; // Объект.МенеджерХранилищаКонфигурации
 Перем АвторизацияВХранилище; // Структура ключи <Пользователь>, <ПарольПользователя>
-Перем МенеджерКонфигуратора; // Объект.МенеджерКонфигуратора
+Перем РабочийКонфигуратор; // Объект.УправлениеКонфигуратором
 Перем ПутьКИсполняемомуФайлуGit; // Строка, полный путь к исполняемому файлу git
 Перем ИмяРасширения; // Строка, Имя расширения для синхронизации
 
@@ -234,6 +234,7 @@
 
 	Лог.Информация("Начата синхронизация с git");
 
+	РабочийКонфигуратор = ПолучитьУправлениеКонфигуратором();
 	ХранилищеКонфигурации = ПолучитьХранилищеКонфигурации(ПутьКХранилищу);
 
 	ТаблицаИсторииХранилища = ПрочитатьИзХранилищаИсториюКоммитовСАвторами(ПутьКХранилищу);
@@ -313,13 +314,6 @@
 
 КонецПроцедуры
 
-Процедура СинхронизироватьРасширение(Знач КаталогРабочейКопии, Знач ПутьКХранилищу)
-
-	Сообщить("Я тут! СинхронизироватьРасширение");
-
-	
-
-КонецПроцедуры
 #КонецОбласти
 
 #Область Функциональность_обработки_версии_хранилища_конфигурации
@@ -335,7 +329,7 @@
 														Знач ПутьКХранилищу,
 														Знач НомерВерсии) Экспорт
 
-	Конфигуратор = ПолучитьМенеджерКонфигуратора();
+	Конфигуратор = ПолучитьРабочийКонфигуратор();
 
 	ОбработчикПодписок.ПередНачаломВыгрузкиВерсииХранилищаКонфигурации(Конфигуратор,
 																		КаталогРабочейКопии,
@@ -399,16 +393,16 @@
 //   НомерВерсии - Число - номер выгружаемой версии
 //
 Процедура ЗагрузитьВерсиюХранилищаВКонфигурацию(Знач Конфигуратор,
-	Знач КаталогРабочейКопии,
-	Знач ПутьКХранилищу,
-	Знач НомерВерсии) Экспорт
+												Знач КаталогРабочейКопии,
+												Знач ПутьКХранилищу,
+												Знач НомерВерсии) Экспорт
 	СтандартнаяОбработка = Истина;
 
 	ОбработчикПодписок.ПриЗагрузкеВерсииХранилищаВКонфигурацию(Конфигуратор,
-	КаталогРабочейКопии,
-	ПутьКХранилищу,
-	НомерВерсии,
-	СтандартнаяОбработка);
+																КаталогРабочейКопии,
+																ПутьКХранилищу,
+																НомерВерсии,
+																СтандартнаяОбработка);
 
 	Если СтандартнаяОбработка Тогда
 
@@ -653,33 +647,46 @@
 // Возвращаемое значение:
 //   Объект.УправлениеКонфигуратором - инициализированный класс <УправлениеКонфигуратором>
 //
-Функция ПолучитьМенеджерКонфигуратора() Экспорт
+Функция ПолучитьРабочийКонфигуратор() Экспорт
+	
+	Если РабочийКонфигуратор = Неопределено Тогда
+		
+		РабочийКонфигуратор = ПолучитьУправлениеКонфигуратором();
+		
+	КонецЕсли;
 
-	Если МенеджерКонфигуратора = Неопределено Тогда
+	Возврат РабочийКонфигуратор;
+	
+КонецФункции
+
+// Подготавливает управление конфигуратором для выполнения операций
+//
+// Возвращаемое значение:
+//   Объект.УправлениеКонфигуратором - инициализированный класс <УправлениеКонфигуратором>
+//
+Функция ПолучитьУправлениеКонфигуратором() Экспорт
 
-		МенеджерКонфигуратора = Новый УправлениеКонфигуратором;
-		Логирование.ПолучитьЛог("oscript.lib.v8runner").УстановитьУровень(Лог.Уровень());
-		КаталогСборки = ВременныеФайлы.СоздатьКаталог();
-		МенеджерКонфигуратора.КаталогСборки(КаталогСборки);
+	УправлениеКонфигуратором = Новый УправлениеКонфигуратором;
+	Логирование.ПолучитьЛог("oscript.lib.v8runner").УстановитьУровень(Лог.Уровень());
+	КаталогСборки = ВременныеФайлы.СоздатьКаталог();
+	УправлениеКонфигуратором.КаталогСборки(КаталогСборки);
+	
+	Если ЗначениеЗаполнено(ТекущийПутьКПлатформе) Тогда
 
-		Если ЗначениеЗаполнено(ТекущийПутьКПлатформе) Тогда
+		Если Не ФС.ФайлСуществует(ТекущийПутьКПлатформе) Тогда
 
-			Если Не ФС.ФайлСуществует(ТекущийПутьКПлатформе) Тогда
-				
-				ВызватьИсключение СтрШаблон("Исполняемый файл платформы 1С <%1> не найден", ТекущийПутьКПлатформе);
-				
-			КонецЕсли;
+			ВызватьИсключение СтрШаблон("Исполняемый файл платформы 1С <%1> не найден", ТекущийПутьКПлатформе);
 
-			МенеджерКонфигуратора.ПутьКПлатформе1С(ТекущийПутьКПлатформе);
-			Лог.Информация("Используется путь к платформе 1С <%1>", ТекущийПутьКПлатформе);
-		Иначе
-			МенеджерКонфигуратора.ИспользоватьВерсиюПлатформы(ТекущаяВерсияПлатформы);
-			Лог.Отладка("Используется платформа 1С версии <%1>", ТекущийПутьКПлатформе);
 		КонецЕсли;
 
+		УправлениеКонфигуратором.ПутьКПлатформе1С(ТекущийПутьКПлатформе);
+		Лог.Информация("Используется путь к платформе 1С <%1>", ТекущийПутьКПлатформе);
+	Иначе
+		УправлениеКонфигуратором.ИспользоватьВерсиюПлатформы(ТекущаяВерсияПлатформы);
+		Лог.Отладка("Используется платформа 1С версии <%1>", ТекущийПутьКПлатформе);
 	КонецЕсли;
 
-	Возврат МенеджерКонфигуратора;
+	Возврат УправлениеКонфигуратором;
 
 КонецФункции
 
@@ -1155,7 +1162,7 @@
 Функция ПолучитьХранилищеКонфигурации(Знач ПутьКХранилищу)
 
 	МенеджерХранилищаКонфигурации = Новый МенеджерХранилищаКонфигурации;
-	НовыйУправлениеКонфигуратором = ПолучитьМенеджерКонфигуратора();
+	НовыйУправлениеКонфигуратором = ПолучитьРабочийКонфигуратор();
 	НовыйУправлениеКонфигуратором.УстановитьКодЯзыка("RU"); // TODO: Костыль выгрузки истории хранилища с другими языками выгружает только в RU
 	МенеджерХранилищаКонфигурации.УстановитьУправлениеКонфигуратором(НовыйУправлениеКонфигуратором);