Захват и помещение объектов в хранилище #211
Conversation
|
@Stepa86 Исправишь замечания Сонара? |
|
@artbear я несколько в замешательстве от его наезда. Первые 2 это как бы из шаблона и везде единая сигнатура. Если править, то или 1) Править все команды 2) Закостылить, чтоб сонар не ругался 3ий пункт - опять же все другие методы по работе с хранилищем именно такую сигнатуру имеют, если переделывать, то все. 4-9 опять следствие копирования шаблона 10 ок, поправлю |
|
@nixel2007 @artbear а куда тыкать, чтоб посмотреть все замечания Сонара, а не только топ 10? |
| " Захват объектов в хранилище 1С. | ||
| | "; | ||
|
|
||
| ОписаниеКоманды = Парсер.ОписаниеКоманды(ПараметрыСистемы.ВозможныеКоманды().ЗахватитьВХранилище, |
There was a problem hiding this comment.
Здесь проще юзать всегда ИмяКоманды вместо явного указания ПараметрыСистемы.ВозможныеКоманды().ЗахватитьВХранилище
There was a problem hiding this comment.
Без исправления я лично несколько раз нарывался при копи-пасте, т.к. забывал команду поправить.
А если исправить, будет сразу подхватываться верное значение :)
| " Помещение изменений объектов в хранилище конфигурации. | ||
| | "; | ||
|
|
||
| ОписаниеКоманды = Парсер.ОписаниеКоманды(ПараметрыСистемы.ВозможныеКоманды().ПоместитьВХранилище, |
There was a problem hiding this comment.
Здесь проще юзать всегда ИмяКоманды вместо явного указания ПараметрыСистемы.ВозможныеКоманды().ЗахватитьВХранилище
|
Хотел написать "Пропишите хоть чуть-чуть сценарии проверки.", но потом вспомнил, что долбаная 1с с их хранилищем конфгирации и поддержкой cli |
|
SonarQube analysis reported 8 issues
|
|
@Stepa86 к сожалению, сейчас такой возможности нет. Есть в очень платной версии Сонара, на которую у нас денег нет :) |
| | Если опция используется, будет выполнена попытка захватить только объекты, указанные в файле. | ||
| | Если опция не используется, будут захвачены все объекты конфигурации. | ||
| | Если в списке указаны объекты, захваченные другим пользователем, эти объекты не будут захвачены и будет выдана ошибка. При этом доступные для захвата объекта будут захвачены. | ||
| | Подробнее о формате файла см в документации http://its.1c.ru/db/v8310doc#bookmark:adm:TI000000564 ."); |
There was a problem hiding this comment.
@Stepa86 Похоже, здесь неправильно ссылка на документацию указана. Надо бы поправить на https://its.1c.ru/db/v8311doc#bookmark:adm:TI000000698
There was a problem hiding this comment.
А я ведь это из 1Сной справки скопипастил. Сейчас проверил на 8.3.12, там ссылка ведет на https://its.1c.ru/db/v8312doc#bookmark:adm:TI000000564
There was a problem hiding this comment.
О-о-о-о, супер, спасибо, ща баг им зарепортю )
| | Если опция используется, будет выполнена попытка поместить только объекты, указанные в файле. | ||
| | Если опция не используется, будут помещены изменения всех объектов конфигурации. | ||
| | При наличии в списке объектов, не захваченных текущим пользователем или захваченных другим пользователем, ошибка выдана не будет. | ||
| | Подробнее о формате файла см в документации http://its.1c.ru/db/v838doc#bookmark:adm:TI000000564"); |
There was a problem hiding this comment.
@Stepa86 Здесь аналогично правильная ссылка https://its.1c.ru/db/v8311doc#bookmark:adm:TI000000698
Добавил команды для захвата и помещения объектов.
Сценарий захватить все/поместить все отрабатывает точно. Остальные должны, но я не проверял. В БДД я не могу.