Ошибки в работе с командным интерфейсом (изменение видимости команд)

DmitryShvaika commented 3 years ago

Опишите ошибку

Edt может самостоятельно менять видимость команд даже когда командный интерфейс не редактировался. Например, достаточно добавить комментарий в модуль формы.

Как воспроизвести

ОшибкаВидимостиИсходная.zip

Загрузите dt-файл в пустую базу
Можно сохранить исходный Cf-файл, можно использовать файл из архива (выгружен из dt)
Загрузите проект в Edt
В модуле формы документа Реализация добавьте любой комментарий
Обновите базу
Выгрузите результирующий Cf-ник
Сравните его с исходным с полными настройками

Как результат видим изменение видимости команд (как нормальных, так и битых ссылок), и изменение порядка команд

Ожидаемое поведение

Командный интерфейс не должен изменяться при редактировании модуля. При изменении командного интерфейса изменение данных должно соответствовать изменениям, произведенным на форме.

Скриншоты

Дополнительно прилагаю видео - процесс изменения порядка команд интерфейса в тестовой базе

КомандаПрыгает

Окружение (пожалуйста укажите информацию):

OS: Windows 10
Версия 1C:EDT 2021.2.8
Версия платформы 1С 8.3.12

tretyakevich commented 3 years ago

Оригинальный файл XML-выгрузки: `

CommonCommand.ОбщаяКоманда2

            <Type>Auto</Type>
            <CommandGroup>FormCommandBarCreateBasedOn</CommandGroup>
            <Index>1</Index>
            <Visible>
                <xr:Common>false</xr:Common>
            </Visible>
        </Item>`

Выгрузка EDT: `Item>

Document.Реализация.StandardCommand.CreateBasedOn

            <Type>Auto</Type>
            <CommandGroup>FormCommandBarCreateBasedOn</CommandGroup>
        </Item>`

Не сериализовано одно свойство (Visible). Раз оно выгрузилось - то false для Visible в платформе не считается default-ым значением. Нужно проверить на чистой форме. В EDT оно явно считается default-ым (поскольку нет в выгрузке). В зависимости от результата тестирования считать проблему либо EDT-й, либо платформенной.
Как и предполагалось - редактирование модуля не меняет модель формы. Проблема именно в выгрузке в XML-формат платформы. Причина обновления формы при изменении модуля - ограниченная функциональность платформы по работе с XML-файлами (изменение модуля требует публикации всей формы).

tretyakevich commented 3 years ago

Проверил 1-й пункт выше. Visible false не является дефолтным значением и в нормальных условиях выставляется только в случае снятия галки 'Видимость'. Однако в предоставленной конфигурации галка стоит, и в выгрузке есть эта запись. Нужно выяснить, какое состояние индицируется таким образом.

Проверил - явно неадекватное поведение платформы на выгрузке. При снятии флага автоположения (и вне зависимости от положения/записи/незаписи индекса) начинает писать Visible=false.

(Ниже все варианты для снятого автоположения):

В результате при наличии включенной автовидимости - пишет Visible=false (автовидимость не пишет, так как дефолтное значение). При этом показывает дефолтную галку видимости включенной.
При выключенной автовидимости - пишет только defaultVisibility=false. При этом галка видимости включена.
При снятой галке видимости (и, естественно, снятой автовидимости) - пишет visibility=false и defaultVisibility=false (единственный ожидаемый результат).

Ожидаемое поведение (которое, собственно, и воспроизводится EDT):

Отключено автоположение - никак не трогаем visibility и defaultVisibility. Пишем индексы
Выключение автовидимости - пишем defaultVisibility=false
Выключение видимости - пишем visibility=false и defaultVisibility=false

Планируемые действия:

Репорт проблемы выгрузки в платформу для исправления проблемы в платформе
Временный патч в EDT для повторения текущего поведения платформы (до исправления проблемы в платформе)

tretyakevich commented 3 years ago

Дополнительно прилагаю видео - процесс изменения порядка команд интерфейса в тестовой базе

@DmitryShvaika Данная проблема не относится к проблеме с неожиданным изменением данных при публикации в ИБ, и, соответственно, не является критической. Соответственно в рамках данной Issue и приоритетного процесса анализа/исправления критических проблема исправляться не будет.

Просьба оформить ее отдельной ошибкой.

DmitryShvaika commented 3 years ago

Спасибо, завел: https://github.com/1C-Company/1c-edt-issues/issues/55

MaksimDzyuba commented 2 years ago

Дополнительно прилагаю видео - процесс изменения порядка команд интерфейса в тестовой базе

@DmitryShvaika Данная проблема не относится к проблеме с неожиданным изменением данных при публикации в ИБ, и, соответственно, не является критической. Соответственно в рамках данной Issue и приоритетного процесса анализа/исправления критических проблема исправляться не будет.

Просьба оформить ее отдельной ошибкой.

снял с задачи, что она критичная

marmyshev commented 1 year ago

@tretyakevich уточни, бага исправлена полностью в 2021.3 или частично? Нужно либо перенести майлстоун или снять, либо закрыть задачу

1C-Company / 1c-edt-issues