Дополнительные параметры, подмешиваемые к любой команде + исправление мелких проблем #139
Conversation
подмешиваются к любой команде, если использовать метод "ИспользоватьДополнительныеПараметры"
src/v8runner.os
Outdated
| @@ -1410,32 +1411,41 @@ | |||
| Возврат СтандартныеПараметрыЗапускаКонфигуратора(); | |||
| КонецФункции | |||
|
|
|||
| // Устанавливаются дополнительные параметры, которые добавляются (подмешиваются) к очередной выполняемой команде | |||
There was a problem hiding this comment.
@artbear может тогда вместо "Устанавливаются" -> "Установить" или "Устанавливает"?
| // | ||
| // Параметры: | ||
| // ДопПараметры - Массив - Параметры, которые добавляются к очередной выполняемой команде | ||
| // |
There was a problem hiding this comment.
@artbear небольшой пример здесь был бы кстати
| @@ -1989,6 +1999,13 @@ | |||
| ПараметрыЗапуска.Добавить("/DisableStartupMessages"); | |||
| ПараметрыЗапуска.Добавить("/DisableStartupDialogs"); | |||
|
|
|||
| Если ЗначениеЗаполнено(ДополнительныеПараметры) Тогда | |||
| Для каждого ДопПараметр Из ДополнительныеПараметры Цикл | |||
| ПараметрыЗапуска.Добавить(ДопПараметр); | |||
There was a problem hiding this comment.
@artbear контроль уникальности отдаем пользователям либы?
There was a problem hiding this comment.
да, им отдаем.
я думал о контроле дублей, но простым способом его не сделать, решил отдать запускателям.
There was a problem hiding this comment.
Билдер коллекции параметров нужен. Вот такая глобальная коллекция, за которой надо следить - мне не нравится.
|
@artbear а в текущем проекте нет задачи по этому ПР? |
такой задачи нет. |
- добавил пример использования
|
@artbear мне не нравятся универсальные апи SELECT * FROM * Можешь привести пример, что такое "дополнительный параметр, примешиваемый к любой команде"? |
| ДопПараметры.Добавить("/DumpDBCfgList -AllExtensions"); | ||
|
|
||
| УправлениеКонфигуратором.ИспользоватьДополнительныеПараметры(ДопПараметры); | ||
| УправлениеКонфигуратором.ВыполнитьКоманду(УправлениеКонфигуратором.ПолучитьПараметрыЗапуска()); |
There was a problem hiding this comment.
ВыполнитьКоманду получает на вход массив аргументов. В этот массив и должны добавляться "доп. параметры". А так они установлены где-то выше по коду, в месте вызова никак не ясно, что вызывается.
Я бы предложил какой-то билдер параметров, а не универсальный флажок где-то выше по стеку
|
@EvilBeaver предложил следующий вариант мой ответ: предварительно такой вариант для расширений подойдет текущий ПР закрываю. |
No description provided.