Skip to content

Conversation

@artbear
Copy link
Member

@artbear artbear commented Jan 10, 2017

Исправлена ошибка с неверным определением платформы 1С #10 #7 #9 #3

/cc @nixel2007 @realMaxA @Tur8008 Проверьте, пожалуйста, свои варианты

@artbear artbear added the bug label Jan 10, 2017
@artbear
Copy link
Member Author

artbear commented Jan 10, 2017

/cc @nixel2007 @realMaxA @Tur8008 Проверьте, пожалуйста, свои варианты

@Tur8008
Copy link
Contributor

Tur8008 commented Jan 10, 2017

У меня работают оба варианта одинаково. Как Ваш так и мой. Ваш, конечно поприличнее и я его приветствую:) Но я не нашел ситуации, когда мой код не сработал бы. Если версия платформы передается однозначно, например 8.3 то ВозможныйПуть будет иметь только одно правильное значение. Я присваивал его переменной ПодтвержденныйПуть и это срабатывает. Если маску задать например "8" конечно мой код не сработает, но зачем это делать?

@nixel2007
Copy link
Member

@artbear я не участвовал в воспроизведении багов, не могу сказать. Нужна проверка от @realMaxA.

@realMaxA
Copy link

У меня платформа нашлась.

@EvilBeaver
Copy link
Member

@artbear @nixel2007 вы более меня в контексте этих исправлений. Решите, что делать с этим кодом, плиз.

@nixel2007 nixel2007 merged commit 3661a5a into develop Jan 11, 2017
@nixel2007 nixel2007 deleted the feature/bug-search-version-10 branch January 11, 2017 07:52
@nixel2007
Copy link
Member

Всем спасибо за тесты. Релизю.

@Tur8008
Copy link
Contributor

Tur8008 commented Jan 11, 2017

Вот теперь понятно, почему мой код не сработал. Было 2 каталога с версиями 8.3. Предложенное @artbear правильно, конечно. Спасибо!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants