fixes #1354 переопределение методов глобального контекста

EvilBeaver · Nov 6, 2023 · de1cc1e · de1cc1e
1 parent 8fabb4b
commit de1cc1e
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 3 deletions.
diff --git a/src/ScriptEngine/Compiler/StackMachineCodeGenerator.cs b/src/ScriptEngine/Compiler/StackMachineCodeGenerator.cs
@@ -287,15 +287,16 @@ protected override void VisitMethod(MethodNode methodNode)
             methodInfo.SetRuntimeParameters(entryPoint, GetVariableNames(methodCtx));
 
             SymbolBinding binding;
-            try
+            if (!_ctx.TryFindMethodBinding(signature.MethodName, out _))
             {
                 binding = _ctx.DefineMethod(methodInfo.ToSymbol());
             }
-            catch (CompilerException)
+            else
             {
                 AddError(LocalizedErrors.DuplicateMethodDefinition(signature.MethodName), signature.Location);
                 binding = default;
             }
+
             _module.MethodRefs.Add(binding);
             _module.Methods.Add(methodInfo);
         }

diff --git a/tests/engine-behaviors.os b/tests/engine-behaviors.os
@@ -52,6 +52,7 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИнформацияОбОшибкеВнеИсключения");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьПередачуПараметровПоСсылке");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьСравнениеНаБольшеМеньше");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьНевозможностьПереопределенияГлобальногоКонтекста");
 
 	Возврат ВсеТесты;
 
@@ -686,8 +687,25 @@
 
 	юТест.ПроверитьКодСОшибкой(
 		"Массив = Новый Массив(1);
-		|Массив2 = Новый Массив(1);;
+		|Массив2 = Новый Массив(1);
 		|Рез = (Массив <= Массив2);",
 		Ошибка, "Для одинаковых объектов" );
 
 КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьНевозможностьПереопределенияГлобальногоКонтекста() Экспорт
+
+    Код = "Функция СтрНайти(знач Стр1, знач Стр2)
+    |	Возврат -1;
+    |КонецФункции
+    |
+    |Рез = СтрНайти(""Ай!"", ""!"");
+    |Если Рез < 0 Тогда
+    |	Сообщить(""Ой!"");
+    |КонецЕсли";
+
+    юТест.ПроверитьКодСОшибкой(Код, "СтрНайти");
+
+    юТест.ПроверитьКодСОшибкой("#native" + Символы.ПС + Код, "СтрНайти");
+
+КонецПроцедуры