Merge pull request #16 from sfaqer/feature/beforeSyncInterceptor

Добавлен новый тип перехватчика "Перед синхронизацией полей"

README.md

@@ -480,6 +480,12 @@ docs - todo;
 #### После
 Перехватчик вызывается после выполнения перехватываемого метода
 
+#### ПослеВызоваИсключения
+Перехватчик вызывается если перехватываемый метод упал в исключение, в момент перед пробросом исключения
+
+#### ПередСинхронизациейПолей
+Перехватчик вызывается в самом начале, перед синхронизацией полей декоратора и оригинального метода
+
 ### Перечисление ТипыПодключенияБибилотек
 
 #### ПоИмени

packagedef

@@ -1,6 +1,6 @@
 // BSLLS:CodeOutOfRegion-off
 Описание.Имя("decorator")
-        .Версия("1.6.0")
+        .Версия("1.7.0")
         .Автор("Nikita Gryzlov")
         .АдресАвтора("[email protected]")
         .Описание("Библиотека для создания декораторов над объектами")

src/internal/Модули/КэшируемыеДанные.os

@@ -52,11 +52,8 @@
 |// {АннотацияМетода}
 |{Функция} Декоратор_ИмяМетода(Декоратор_ОписаниеПараметровМетода) // {Экспорт}
 |
-|	Если {Декоратор_ЭтоДекорируемыйМетод} Тогда
-|		ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);
-|	Иначе
-|		ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);
-|	КонецЕсли;
+|	// {Декоратор_ПередСинхронизациейПолей}
+|	// {Декоратор_СинхронизироватьПоляПеред}
 |
 |	Декоратор_ВозвращаемоеИзМетодаЗначение = NULL;
 |
@@ -68,23 +65,15 @@
 |
 |	Исключение
 |
-|		Если {Декоратор_ЭтоДекорируемыйМетод} Тогда
-|			ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);
-|		Иначе
-|			ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);
-|		КонецЕсли;
+|		// {Декоратор_СинхронизироватьПоляПосле}
 |
 |		// {Декоратор_ПослеВызоваИсключения}
 |
 |		ВызватьИсключение;
 |
 |	КонецПопытки;
 |
-|	Если {Декоратор_ЭтоДекорируемыйМетод} Тогда
-|		ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);
-|	Иначе
-|		ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);
-|	КонецЕсли;
+|	// {Декоратор_СинхронизироватьПоляПосле}
 |
 |	Если Декоратор_ВозвращаемоеИзМетодаЗначение = Декоратор_ИсходныйОбъект Тогда
 |		Декоратор_ВозвращаемоеИзМетодаЗначение = ЭтотОбъект;

src/Классы/КонструкторДекоратора.os

@@ -438,9 +438,20 @@
 		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Экспорт}", "Экспорт");
 		НовыйМетод = СтрЗаменить(НовыйМетод, "{Функция}", "Функция");
 		НовыйМетод = СтрЗаменить(НовыйМетод, "{КонецФункции}", "КонецФункции");
-		НовыйМетод = СтрЗаменить(НовыйМетод, "{Декоратор_ЭтоДекорируемыйМетод}", "Истина");
 		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Декоратор_ТекстМетода}", ТекстМетода);
-
+
+		НовыйМетод = СтрЗаменить(
+			НовыйМетод,
+			"// {Декоратор_СинхронизироватьПоляПеред}",
+			"ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);"
+		);
+
+		НовыйМетод = СтрЗаменить(
+			НовыйМетод,
+			"// {Декоратор_СинхронизироватьПоляПосле}",
+			"ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);"
+		);
+
 		ДобавитьПерехватчикиМетода(ИмяМетода, НовыйМетод, Истина);
 
 		НовыйМетод = НовыйМетод + Символы.ПС;
@@ -473,10 +484,22 @@
 		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Экспорт}", "Экспорт");
 		НовыйМетод = СтрЗаменить(НовыйМетод, "{Функция}", "Функция");
 		НовыйМетод = СтрЗаменить(НовыйМетод, "{КонецФункции}", "КонецФункции");
-		НовыйМетод = СтрЗаменить(НовыйМетод, "{Декоратор_ЭтоДекорируемыйМетод}", "Ложь");
 		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Декоратор_ВыполняемаяСтрока}", ВыполняемаяСтрока);
 		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Декоратор_ВозвращаемоеЗначение}", ВозвращаемоеЗначение);		
-
+
+		// Для будущих археологов, если вы подумаете что нужно тут добавить
+		// ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);
+		// то проверьте что работает АктивнаяЗапись в библиотеки entity
+		// сейчас этого куска нет потому что при его наличии в декораторе задавятся поля
+		// декоратора которые были заполнены не через методы, а просто установкой свойств.
+		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Декоратор_СинхронизироватьПоляПеред}", "");
+
+		НовыйМетод = СтрЗаменить(
+			НовыйМетод,
+			"// {Декоратор_СинхронизироватьПоляПосле}",
+			"ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);"
+		);
+
 		ДобавитьПерехватчикиМетода(ИмяМетода, НовыйМетод);
 
 		ОбластьМетодов = ОбластьМетодов + НовыйМетод;

src/Классы/ПостроительДекоратора.os

@@ -442,7 +442,17 @@
 
 		НовыйМетод = Метод.Описание();
 
-		НовыйМетод = СтрЗаменить(НовыйМетод, "{Декоратор_ЭтоДекорируемыйМетод}", "Истина");
+		НовыйМетод = СтрЗаменить(
+			НовыйМетод,
+			"// {Декоратор_СинхронизироватьПоляПеред}",
+			"ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);"
+		);
+
+		НовыйМетод = СтрЗаменить(
+			НовыйМетод,
+			"// {Декоратор_СинхронизироватьПоляПосле}",
+			"ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);"
+		);
 
 		ДобавитьПерехватчикиМетода(ИмяМетода, НовыйМетод);
 
@@ -458,7 +468,18 @@
 
 		НовыйМетод = Метод.Описание();
 
-		НовыйМетод = СтрЗаменить(НовыйМетод, "{Декоратор_ЭтоДекорируемыйМетод}", "Ложь");
+		// Для будущих археологов, если вы подумаете что нужно тут добавить
+		// ОбработкаДекоратора.СинхронизироватьПоля(Декоратор_ИсходныйОбъект, ЭтотОбъект);
+		// то проверьте что работает АктивнаяЗапись в библиотеки entity
+		// сейчас этого куска нет потому что при его наличии в декораторе задавятся поля
+		// декоратора которые были заполнены не через методы, а просто установкой свойств.
+		НовыйМетод = СтрЗаменить(НовыйМетод, "// {Декоратор_СинхронизироватьПоляПеред}", "");
+
+		НовыйМетод = СтрЗаменить(
+			НовыйМетод,
+			"// {Декоратор_СинхронизироватьПоляПосле}",
+			"ОбработкаДекоратора.СинхронизироватьПоля(ЭтотОбъект, Декоратор_ИсходныйОбъект);"
+		);
 
 		ДобавитьПерехватчикиМетода(Метод.ИмяМетода(), НовыйМетод);
 
@@ -474,7 +495,9 @@
 
 	ЗаменяемаяСтрокаПеред = "// {Декоратор_ПередВызовомМетода}";
 	ЗаменяемаяСтрокаПосле = "// {Декоратор_ПослеВызоваМетода}";
-	ЗаменяемаяСтрокаПередИсключением = "// {Декоратор_ПослеВызоваИсключения}";
+
+	ЗаменяемаяСтрокаПослеИсключения          = "// {Декоратор_ПослеВызоваИсключения}";
+	ЗаменяемаяСтрокаПередСинхронизациейПолей = "// {Декоратор_ПередСинхронизациейПолей}";
 
 	ТекстМетода = СтрЗаменить(
 		ТекстМетода,
@@ -490,10 +513,16 @@
 
 	ТекстМетода = СтрЗаменить(
 		ТекстМетода,
-		ЗаменяемаяСтрокаПередИсключением,
+		ЗаменяемаяСтрокаПослеИсключения,
 		СтрСоединить(ТекстыПерехватчиков.Получить(ТипыПерехватчиковМетода.ПослеВызоваИсключения), Символы.ПС)
 	);
 
+	ТекстМетода = СтрЗаменить(
+		ТекстМетода,
+		ЗаменяемаяСтрокаПередСинхронизациейПолей,
+		СтрСоединить(ТекстыПерехватчиков.Получить(ТипыПерехватчиковМетода.ПередСинхронизациейПолей), Символы.ПС)
+	);
+
 КонецПроцедуры
 
 Функция ТекстыПерехватчиков(ИмяМетода)
@@ -502,7 +531,9 @@
 
 	Результат.Вставить(ТипыПерехватчиковМетода.Перед, Новый Массив);
 	Результат.Вставить(ТипыПерехватчиковМетода.После, Новый Массив);
-	Результат.Вставить(ТипыПерехватчиковМетода.ПослеВызоваИсключения, Новый Массив);
+
+	Результат.Вставить(ТипыПерехватчиковМетода.ПослеВызоваИсключения,    Новый Массив);
+	Результат.Вставить(ТипыПерехватчиковМетода.ПередСинхронизациейПолей, Новый Массив);
 
 	Для каждого Перехватчик Из ПерехватчикиМетодов Цикл
 

src/Модули/ТипыПерехватчиковМетода.os

@@ -1,7 +1,11 @@
 Перем Перед Экспорт;
 Перем После Экспорт;
+
 Перем ПослеВызоваИсключения Экспорт;
+Перем ПередСинхронизациейПолей Экспорт;
 
 Перед = "Перед";
 После = "После";
-ПослеВызоваИсключения = "ПослеВызоваИсключения";
+
+ПослеВызоваИсключения    = "ПослеВызоваИсключения";
+ПередСинхронизациейПолей = "ПередСинхронизациейПолей";

tests/Тесты_ПостроительДекоратора.os

@@ -794,3 +794,60 @@
 	Ожидаем.Что(Декоратор.СчетчикПерехватчикаПослеВызоваИсключения).Равно(1);
 
 КонецПроцедуры
+
+&Тест
+Процедура ПерехватчикиПередСинхронизациейПолейДекорируемыхФункцийРаботаютКорректно() Экспорт
+
+	// Дано
+	МойКласс = Новый МойКласс;
+	МойКласс.Поле = 123;
+
+	ПерехватчикПередСинхронизациейПолей = Новый Перехватчик("ВернутьЭтотОбъект")
+		.ТипПерехватчика(ТипыПерехватчиковМетода.ПередСинхронизациейПолей)
+		.Тело("Поле = 124;");
+
+	Декоратор = Новый ПостроительДекоратора(МойКласс)
+		.Перехватчик(ПерехватчикПередСинхронизациейПолей)
+		.Построить();
+
+	// Когда
+
+	Результат = Декоратор.ВернутьЭтотОбъект();
+
+	// Тогда
+
+	Ожидаем.Что(Результат.Поле).Равно(124);
+	Ожидаем.Что(МойКласс.Поле).Равно(124);
+
+КонецПроцедуры
+
+&Тест
+Процедура ПерехватчикиПередСинхронизациейПолейДобавленныхФункцийРаботаютКорректно() Экспорт
+
+	// Дано
+	МойКласс = Новый МойКласс;
+	МойКласс.Поле = 123;
+
+	НовыйМетод = Новый Метод("ВернутьСебя")
+		.Публичный()
+		.ТелоМетода("Возврат ЭтотОбъект;");
+
+	ПерехватчикПередСинхронизациейПолей = Новый Перехватчик("ВернутьСебя")
+		.ТипПерехватчика(ТипыПерехватчиковМетода.ПередСинхронизациейПолей)
+		.Тело("Поле = 124;");
+
+	Декоратор = Новый ПостроительДекоратора(МойКласс)
+		.Метод(НовыйМетод)
+		.Перехватчик(ПерехватчикПередСинхронизациейПолей)
+		.Построить();
+
+	// Когда
+
+	Результат = Декоратор.ВернутьСебя();
+
+	// Тогда
+
+	Ожидаем.Что(Результат.Поле).Равно(124);
+	Ожидаем.Что(МойКласс.Поле).Равно(124);
+
+КонецПроцедуры