Возможность загрузки исходников в конфигурацию, подключенную к хранилищу #176
Conversation
…ю к хранилищу (реализована передача параметров подключения к хранилищу)
src/v8runner.os
Outdated
| Знач ОбновитьФайлВерсий = Истина, | ||
| Знач ОбновитьКонфигурациюИБ = Ложь) Экспорт | ||
| Знач ОбновитьКонфигурациюИБ = Ложь, | ||
| Знач ПараметрыХранилища = Неопределено) Экспорт |
There was a problem hiding this comment.
Как-то непривычно гитхаб работает с табуляцией. В vscode все ровно.
There was a problem hiding this comment.
Табы вместо пробелов дают такой эффект. Рекомендуются пробелы.
src/v8runner.os
Outdated
| // ОбновитьФайлВерсий - Булево - указывает, что в конце загрузки в каталоге будет создан файл версий ConfigDumpInfo.xml, | ||
| // соответствующий загруженной конфигурации. | ||
| // Если выполняется частичная загрузка (используется опция -files или -listFile), файл версий будет обновлен. | ||
| // ПараметрыХранилища - Структура, Неопределено - параметры подключения к хранилищу для возможности работы с такими базами. |
There was a problem hiding this comment.
Отступ. И кажется, было бы неплохо документировать ожидаемые свойства параметров хранилища и добавить фабричный метод, который бы создавал такую структуру с правильными свойствами сразу.
There was a problem hiding this comment.
Зачем нужен фабричный метод если структура внутри кода нигде не создается? Можно переписать на три отдельных параметра, как реализовано в других командах, но хорошо ли это - большое количество параметров, тем более что они связаны друг с другом?
There was a problem hiding this comment.
Ну так... Это же параметр. Значит нам его будут передавать извне. А как они узнают, что нужно передать, какие поля нужны у структуры и т.п?
Фабричный метод не для нас, а для тех, кто будет библиотекой пользоваться, чтобы им проще было. Ну или документировать тут свойства этой структуры
src/v8runner.os
Outdated
| Если ЗначениеЗаполнено(ПараметрыХранилища) Тогда | ||
| Параметры.Добавить("/ConfigurationRepositoryF " + ОбернутьВКавычки(ПараметрыХранилища["СтрокаСоединения"])); | ||
| Параметры.Добавить("/ConfigurationRepositoryN " + ОбернутьВКавычки(ПараметрыХранилища["ПользовательХранилища"])); | ||
| Если Не ПустаяСтрока(ПараметрыХранилища["ПарольХранилища"]) Тогда |
There was a problem hiding this comment.
Почему через квадратные скобки, раз уж это структура? Почему не через точку?
There was a problem hiding this comment.
Весь код взят из vanessa-runner :) Но с радостью исправлю замечания.
|
Давайте добьем, пожалуйста, я все исправил. А то мне еще в vanessa runner заливать изменения. |
2 participants
Добавлена возможность загрузки исходников в конфигурацию, подключенную к хранилищу, через список файлов
(реализована передача параметров подключения к хранилищу)