Функциональность остановки (прерывания) синхронизации при незаполненном комментарии к версии хранилища #67
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…нилищем, если не заполнен комментарий к версии
artbear
requested changes
May 3, 2017
| @@ -969,6 +973,11 @@ | |||
| ВызватьИсключение "Нашли следующую версию, а автор не прописан "+СледующаяВерсия + " автор "+СтрокаВерсии.Автор; | |||
| КонецЕсли; | |||
|
|
|||
| Если ПустаяСтрока(СтрокаВерсии.Комментарий) | |||
| И ПрерватьВыполнениеБезКомментарияКВерсии Тогда | |||
| ВызватьИсключение "Нашли следующую версию, а комментарий не задан "+СледующаяВерсия + " автор "+СтрокаВерсии.Автор; | |||
There was a problem hiding this comment.
Заюзай СтрШаблон, плиз.
Постепенно выправляем код.
| @@ -952,6 +952,10 @@ | |||
| Формат = РежимВыгрузкиФайлов.Авто; | |||
| КонецЕсли; | |||
|
|
|||
| если ПрерватьВыполнениеБезКомментарияКВерсии = Неопределено Тогда | |||
artbear
reviewed
May 4, 2017
| @@ -969,6 +973,11 @@ | |||
| ВызватьИсключение "Нашли следующую версию, а автор не прописан "+СледующаяВерсия + " автор "+СтрокаВерсии.Автор; | |||
| КонецЕсли; | |||
|
|
|||
| Если ПустаяСтрока(СтрокаВерсии.Комментарий) | |||
| И ПрерватьВыполнениеБезКомментарияКВерсии Тогда | |||
| ВызватьИсключение СтрШаблон("Нашли следующую версию, а комментарий не задан %1 автор %2.",СледующаяВерсия, СтрокаВерсии.Автор); | |||
There was a problem hiding this comment.
Как-то некрасиво.
Предлагаю СтрШаблон("Нашли следующую версию %1, а комментарий не задан! Автор %2.", СледующаяВерсия, СтрокаВерсии.Автор);
|
Очень желательно прогнать текущие тесты перед мереджем. Пока сам не могу, проблема с запуском 1С. Не актуально |
|
Тесты на linux не проходят. Т.к. Написаны под win.... |
|
Скорректировал тесты для прохождения на Linux. Прогнал все текущие тесты. Зеленая полоса :) |
|
Up. На подходе следующий PR. |
|
@EvilBeaver - Я никуда не спешу )) все равно уже себе все слил в девелоп и выпустил версию ) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
По установленному флагу останавливает синхронизацию с хранилищем. если попадается версия без заполенного комментария.
Нужна помощь в написание тестов и прогоне тестирования.