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

Mr-Rm · Nov 6, 2023 · 08bcb49 · 08bcb49
1 parent 4ef63ba
commit 08bcb49
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/src/ScriptEngine/Compiler/ModuleCompilerContext.cs b/src/ScriptEngine/Compiler/ModuleCompilerContext.cs
@@ -27,6 +27,10 @@ public ModuleCompilerContext(CompilerContext outerContext)
 
         public SymbolBinding DefineMethod(MethodInfo method)
         {
+            if (_outerCtx.TryGetMethod(method.Name, out _))
+            {
+                throw new CompilerException($"Symbol already defined {method.Name}");
+            }
             var sb = _moduleCtx.DefineMethod(method);
             ShiftIndex(ref sb);
 

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