Artur Ayukhanov

Results 592 comments of Artur Ayukhanov

В комментарии https://github.com/1c-syntax/bsl-language-server/issues/1814#issuecomment-937713793 я указал, как хранится информация о варианте расширения (Вместо и т.п.) для обработчика события формы

Связанные и уже реализованные ишью для плагина ЕДТ - https://github.com/1C-Company/v8-code-style/issues/319 - https://github.com/1C-Company/v8-code-style/issues/497

> Не требуется если нет поддержки обычного приложения и для разработки мобильных приложений. Это неверно. В стандарте написано >Однако в толстом клиенте, в режиме управляемого приложения, клиент-сервер, возможны ситуации, когда...

> 1. не обработан вызов Новый(Тип) интересно, сделаю > 2. я не понимаю, зачем это правило в бсл лс. правило для аудита стороннего кода, от подрядчиков, от внешних специалистов может...

> Помимо код смеллов и потенциальных уязвимостей есть ещё и обычные уязвимости ага, с ними все в порядке, переделаю на обычную уязвимость

@nixel2007 я переделал - тип обычная уязвимость + обработка Новый(Тип) + выключил правило еще замечания есть?

@theshadowco замечания исправлены, прекоммит выполнен, заодно и замечание кодокролика поправил

@serg-beatl интересный ПР. к сожалению, пока не могу его принять - исходники хмл зря добавлены, они меняют совместимость - их нужно откатить - замена НачатьПомещениеФайла на НачатьПомещениеФайловНаСервер не будет работать...

@ImHunter Интересная доработка по получению свойств конфигурации. один вопрос - зачем подобную доработку делать именно в обработке ЗакрытьПредприятие? почему не сделать отдельную обработку, выполняющую действия по получению свойств конфигурации?

@ImHunter извини за молчание, пропустил твой вопрос. да, лучше сделай новую обработку, новые команды лучше явно отделить от фичи закрытия