fix EvilBeaver#1580.3: исключение при лишних символах в параметре сортировки

Author: Mr-Rm

src/OneScript.Language/Localization/BilingualString.cs

@@ -73,5 +73,21 @@ public static string Localize(string russian, string english)
 
             return russian;
         }
+
+        public static bool BiEquals(string str, string lang1, string lang2)
+        {
+            StringComparison comparison = StringComparison.CurrentCultureIgnoreCase;
+            return string.Equals(str, lang1, comparison) || string.Equals(str, lang2, comparison);
+        }
+    }
+
+    public static class BilingualStringExtension
+    {
+        public static bool BilingualEquals(this string str, string lang1, string lang2, 
+            StringComparison comparison = StringComparison.CurrentCultureIgnoreCase)
+        {
+            return string.Equals(str, lang1, comparison) || string.Equals(str, lang2, comparison);
+        }
     }
+
 }

src/OneScript.StandardLibrary/Collections/ValueTable/ValueTable.cs

@@ -17,6 +17,7 @@ This Source Code Form is subject to the terms of the
 using ScriptEngine.Machine;
 using ScriptEngine.Machine.Contexts;
 using OneScript.StandardLibrary.Collections.Exceptions;
+using OneScript.Localization;
 
 namespace OneScript.StandardLibrary.Collections.ValueTable
 {
@@ -619,20 +620,15 @@ private List<ValueTableSortRule> GetSortRules(string Columns)
                 string[] description = column.Trim().Split(' ');
                 if (description.Length == 0)
                     throw ColumnException.WrongColumnName();
-
-                ValueTableSortRule Desc = new ValueTableSortRule();
-                Desc.Column = this.Columns.FindColumnByName(description[0]);
-
-                if (description.Length > 1)
-                {
-                    if (String.Compare(description[1], "DESC", true) == 0 || String.Compare(description[1], "УБЫВ", true) == 0)
-                        Desc.direction = -1;
-                    else
-                        Desc.direction = 1;
-                }
-                else
-                    Desc.direction = 1;
-
+                if (description.Length > 2)
+                    throw RuntimeException.InvalidNthArgumentValue(1);
+
+                var Desc = new ValueTableSortRule
+                {
+                    Column = this.Columns.FindColumnByName(description[0]),
+                    direction = (description.Length > 1 && description[1].BilingualEquals("УБЫВ", "DESC")) ? -1 : 1
+                };
+
                 Rules.Add(Desc);
             }
 

tests/valuetable.os

@@ -55,6 +55,7 @@
 
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьВставкуВнеРазмераТаблицы");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИсключениеПриЗагрузкеКолонки");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьИсключениеПриНеверномПараметреСортировки");
 
 	Возврат ВсеТесты;
 
@@ -980,3 +981,19 @@
 
 	ВызватьИсключение "Ожидали исключение, но его не было";
 КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьИсключениеПриНеверномПараметреСортировки() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	ТЗ.Добавить().Тест = 1;	
+	ТЗ.Добавить().Тест = 2;
+	
+	Попытка
+		ТЗ.Сортировать("Тест УБЫВ и_лишнее_через_пробел");
+	Исключение
+		Возврат;
+	КонецПопытки;
+	
+	ВызватьИсключение "Ожидали исключение, но его не было";
+КонецПроцедуры
+