Feature/spell #68
Feature/spell #68
Conversation
pumbaEO
commented
Feb 20, 2017
•
pumbaEO
commented
- Добавил конвертацию файлов Проверить конвертацию файлов для разных платформ вниз #69
- Исправил мелкие орф.ошибки.
|
SonarQube analysis reported 8 issues
|
tools/runner.os
Outdated
| @@ -789,7 +793,7 @@ | |||
|
|
|||
| ТекущаяПроцедура = "ЗапуститьВРежимеПредприятия"; | |||
|
|
|||
| Ожидаем.Что(СтрокаПодключения, ТекущаяПроцедура+" не заданая строка подключения").Заполнено(); | |||
| Ожидаем.Что(СтрокаПодключения, ТекущаяПроцедура+" не заданна строка подключения").Заполнено(); | |||
tools/runner.os
Outdated
| @@ -1385,6 +1389,31 @@ | |||
|
|
|||
| КонецФункции | |||
|
|
|||
| Процедура КонвертироватьФайлы(Каталог, Знач СтрокаПодключения="", Знач Пользователь="", Знач Пароль="", Знач ВерсияПлатформы="") Экспорт | |||
There was a problem hiding this comment.
Это конверт из 8.1 в 8.2? О_о
There was a problem hiding this comment.
Методу КонвертироватьФайлы место в v8runner
There was a problem hiding this comment.
Нужно добавить Знач Каталог
There was a problem hiding this comment.
- Это конверт из 8.3.10 с последовательной конвертацией в 8.2, т.к. запускаем базу на 8.3.9 и конвертируем, потом запускаем на версии 8.3.8 и т.д.
- С учетом труднойстей выпуска и релиза v8runner, пока сделал здесь, для отладки выпуска behavior, т.к. все собираю и храню на 8.3.10 и надо попробовать, что-бы это заработало.
| @@ -52,6 +52,8 @@ | |||
| мВозможныеКоманды.Вставить("ОбновитьИзХранилища", "loadrepo"); | |||
| мВозможныеКоманды.Вставить("Следить", "watch"); | |||
| мВозможныеКоманды.Вставить("Конфигуратор", "designer"); | |||
|
|
|||
| мВозможныеКоманды.Вставить("КонвертироватьФайлы", "convertfiles"); | |||
There was a problem hiding this comment.
Увидел новую команду КонвертироватьФайлы, о которой в описании коммита "Мелкие изменения" ничего не сказано :(
Женя, зачем так описывать свои изменения?
tools/runner.os
Outdated
| Конфигуратор.ИспользоватьВерсиюПлатформы(ВерсияПлатформы); | ||
| КонецЕсли; | ||
|
|
||
| //Синхронизатор = Новый СинхронизаторХранилища(); |
There was a problem hiding this comment.
закомментированный код нужно удалить.
tools/runner.os
Outdated
| //Синхронизатор = Новый СинхронизаторХранилища(); | ||
|
|
||
| ПараметрыЗапуска = Конфигуратор.ПолучитьПараметрыЗапуска(); | ||
| //ПараметрыЗапуска.Добавить("/Visible"); |
There was a problem hiding this comment.
закомментированный код нужно удалить.
tools/runner.os
Outdated
| @@ -1691,24 +1720,24 @@ | |||
| ФайлСобранный = Новый Файл(НовыйПутьВыгрузки); | |||
| Кэш.Вставить(ФайлСобранный.ПолноеИмя, ФайлСобранный.ПолучитьВремяИзменения()); | |||
| КэшОбновляемый.Вставить(ФайлСобранный.ПолноеИмя, ФайлСобранный.ПолучитьВремяИзменения()); | |||
| //СписокОбработаных.Вставить(ФайлСобранный.ПолноеИмя, Истина); | |||
| //СписокОбработанных.Вставить(ФайлСобранный.ПолноеИмя, Истина); | |||
There was a problem hiding this comment.
закомментированный код нужно удалить.
| Исключение | ||
| Лог.Ошибка("Ошибка загрузки файлов конфигурации:"+ОписаниеОшибки()); | ||
| КонецПопытки; | ||
|
|
||
| Если КонфигурацияЗагруженна = Истина И АвтоОбновление = Истина Тогда | ||
| Если КонфигурацияЗагружена = Истина И АвтоОбновление = Истина Тогда |
There was a problem hiding this comment.
Опять использование Если Перем = Истина ? Тот же Сонар на это ругается.
ИМХО нужно явно инициализовать переменные и тогда код станет намного проще
There was a problem hiding this comment.
Ну вообще-то я с Сонаром по этому поводу не согласен, т.к. тут будет у меня неявное преобразование к чему-либо, а при Если КонфигурацияЗагружена Тогда - это будет ошибка.
There was a problem hiding this comment.
Так я и говорю, что нужно явно инициализировать переменные, чтобы исключить неявные преобразования.
There was a problem hiding this comment.
Значение этих переменных читается из конфигов, поэтому или писать кучу Eсли на проверку и преобразование из 1 или текста "true" в проинициализированную переменную или же таким образом сразу делать преобразование, имхо это я готов вынести на отдельное обсуждение. Но сейчас тут больше проверка орфографии чем исправление тех.долга.
tools/runner.os
Outdated
| @@ -1385,6 +1389,31 @@ | |||
|
|
|||
| КонецФункции | |||
|
|
|||
| Процедура КонвертироватьФайлы(Каталог, Знач СтрокаПодключения="", Знач Пользователь="", Знач Пароль="", Знач ВерсияПлатформы="") Экспорт | |||
There was a problem hiding this comment.
Методу КонвертироватьФайлы место в v8runner
tools/runner.os
Outdated
| @@ -1385,6 +1389,31 @@ | |||
|
|
|||
| КонецФункции | |||
|
|
|||
| Процедура КонвертироватьФайлы(Каталог, Знач СтрокаПодключения="", Знач Пользователь="", Знач Пароль="", Знач ВерсияПлатформы="") Экспорт | |||
There was a problem hiding this comment.
Нужно добавить Знач Каталог
|
По поводу v8runner oscript-library/v8runner#33 |
|
Добавил в заглавный коммент PR строку "Добавил конвертацию файлов #69" |
|
В целом у меня явных возражений нет, кроме переноса метода КонвертироватьФайлы в v8runner. |
