fix EvilBeaver#1580.4,6,10: пустой список колонок означает все колонки

Author: Mr-Rm

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

@@ -157,41 +157,39 @@ public ArrayImpl UnloadColumn(IValue column)
             return result;
         }
 
-        private List<ValueTableColumn> GetProcessingColumnList(string ColumnNames, bool EmptyListInCaseOfNull = false)
+        private List<ValueTableColumn> GetProcessingColumnList(string columnNames, bool emptyListInCaseOfNull = false)
         {
-            List<ValueTableColumn> processing_list = new List<ValueTableColumn>();
-            if (ColumnNames != null)
-            {
-                if (ColumnNames.Trim().Length == 0)
-                {
-                    // Передали пустую строку вместо списка колонок
-                    return processing_list;
-                }
+            var processing_list = new List<ValueTableColumn>();
+            if (string.IsNullOrEmpty(columnNames)) // Передали пустую строку вместо списка колонок
+            {
+                if (!emptyListInCaseOfNull)
+                {
+                    processing_list.AddRange(Columns);
+                }
+                return processing_list;
+            }
 
-                foreach (var column_name in ColumnNames.Split(','))
-                {
-                    var name = column_name.Trim();
-                    var Column = Columns.FindColumnByName(name);
+            foreach (var column_name in columnNames.Split(','))
+            {
+                var name = column_name.Trim();
+                var Column = Columns.FindColumnByName(name);
 
-                    if (Column == null)
-                        throw ColumnException.WrongColumnName(name);
+                if (Column == null)
+                    throw ColumnException.WrongColumnName(column_name);
 
-                    if (processing_list.Find( x=> x.Name==name ) == null)
-                        processing_list.Add(Column);
-                }
-            }
-            else if (!EmptyListInCaseOfNull)
-            {
-                processing_list.AddRange(Columns);
+                if (processing_list.Find(x => x.Name == name) == null)
+                    processing_list.Add(Column);
             }
+ 
             return processing_list;
         }
 
         /// <summary>
         /// Заполнить колонку/колонки указанным значением
         /// </summary>
         /// <param name="value">Произвольный - Устанавливаемое значение</param>
-        /// <param name="columnNames">Строка - Список имен колонок для установки значения (разделены запятыми)</param>
+        /// <param name="columnNames">Строка - Список имен колонок для установки значения (разделены запятыми).
+        /// Если параметр не указан или передана пустая строка, будут заполнены все колонки</param>
         [ContextMethod("ЗаполнитьЗначения", "FillValues")]
         public void FillValues(IValue value, string columnNames = null)
         {
@@ -517,7 +515,8 @@ public void Move(BslValue row, int offset)
         /// <summary>
         /// Создает новую таблицу значений с указанными колонками. Данные не копируются.
         /// </summary>
-        /// <param name="columnNames">Строка - Имена колонок для копирования, разделены запятыми</param>
+        /// <param name="columnNames">Строка - Имена колонок для копирования, разделены запятыми
+        /// Если параметр не указан или передана пустая строка, будут скопированы все колонки</param>
         /// <returns>ТаблицаЗначений</returns>
         [ContextMethod("СкопироватьКолонки", "CopyColumns")]
         public ValueTable CopyColumns(string columnNames = null)

tests/valuetable.os

@@ -57,6 +57,11 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИсключениеПриЗагрузкеКолонки");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИсключениеПриНеверномПараметреСортировки");
 
+	ВсеТесты.Добавить("ТестДолжен_Проверить_ЗаполнитьЗначения_СПустымСпискомКолонок");
+	ВсеТесты.Добавить("ТестДолжен_Проверить_Найти_СПустымСпискомКолонок");
+	ВсеТесты.Добавить("ТестДолжен_Проверить_СкопироватьКолонки_СПустымСпискомКолонок");
+	ВсеТесты.Добавить("ТестДолжен_Проверить_Скопировать_СПустымСпискомКолонок");
+	
 	Возврат ВсеТесты;
 
 КонецФункции
@@ -997,3 +1002,43 @@
 	ВызватьИсключение "Ожидали исключение, но его не было";
 КонецПроцедуры
 
+Процедура ТестДолжен_Проверить_ЗаполнитьЗначения_СПустымСпискомКолонок() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	ТЗ.Добавить().Тест = 1;	
+	ТЗ.ЗаполнитьЗначения(3,"");
+	
+	юТест.ПроверитьРавенство(ТЗ[0].Тест, 3);
+КонецПроцедуры
+
+Процедура ТестДолжен_Проверить_Найти_СПустымСпискомКолонок() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	ТЗ.Добавить().Тест = -3;	
+	ТЗ.Добавить().Тест = 7;
+	Рез = ТЗ.Найти(7,"");
+
+	юТест.ПроверитьНеРавенство(Рез, Неопределено);
+КонецПроцедуры
+
+Процедура ТестДолжен_Проверить_СкопироватьКолонки_СПустымСпискомКолонок() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	ТЗ.Добавить().Тест = 1;	
+	ТЗ2 = ТЗ.СкопироватьКолонки();
+	
+	юТест.ПроверитьРавенство(ТЗ2.Колонки.Количество(), 1);
+	юТест.ПроверитьРавенство(ТЗ2.Колонки[0].Имя, "Тест");
+КонецПроцедуры
+
+Процедура ТестДолжен_Проверить_Скопировать_СПустымСпискомКолонок() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	ТЗ.Добавить().Тест = 1;	
+	ТЗ.Добавить().Тест = 2;	
+	ТЗ2 = ТЗ.Скопировать(,"");
+	
+	юТест.ПроверитьРавенство(ТЗ2.Количество(), 2);
+	юТест.ПроверитьРавенство(ТЗ2.Колонки.Количество(), 1);
+	юТест.ПроверитьРавенство(ТЗ2.Колонки[0].Имя, "Тест");
+КонецПроцедуры