Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Реализовать проверку на наличие дубликатов среди идентификаторов элементов форм #7

Closed
AlmazNasibullin opened this issue Dec 17, 2021 · 1 comment · Fixed by #37
Closed

Реализовать проверку на наличие дубликатов среди идентификаторов элементов форм #7

AlmazNasibullin opened this issue Dec 17, 2021 · 1 comment · Fixed by #37
Assignees
@nikolay-martynov
Labels
enhancement New feature or request form Валидация модели формы
Milestone
0.3 для EDT 2022.2

Comments

@AlmazNasibullin
Copy link
Collaborator

Описание проблемы

При одновременной доработке одной формы двумя и более разработчиками может возникнуть ситуация, что появляются дубликаты среди идентификаторов элементов форм, для этого достаточно добавить каждому из разработчиков новый элемент формы. Наличие дубликатов среди идентификаторов элементов форм "ломает" форму в рантайме.

Описание решения проблемы

Предлагается реализовать проверку с возможность исправления для вышеописанной проблемы наличия дубликатов среди идентификаторов элементов форм.

Дополнительная информация

No response
@AlmazNasibullin AlmazNasibullin added the enhancement New feature or request label Dec 17, 2021
@marmyshev marmyshev added the form Валидация модели формы label Dec 17, 2021
@marmyshev
Copy link
Collaborator

Бывают ситуации когда идентификатора у элемента нет - это тоже ошибка. Нужно фиксить так же - генерить новый идентификатор

@nikolay-martynov nikolay-martynov mentioned this issue Jul 20, 2022
Closed
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Добавлены проверки на отсутсвие и неверное значение у элементов
2518009 
форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Добавлены проверки на дублирование значений идентификаторов у
03b8b87 
элементов форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Для проверки правильности идентификаторов элементов формы
17ba8b5 
добавлены полезные расширения
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Синхронизация тестов с изменениями API CheckTestBase
105f922
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Добавлена детальная справка для проверки корректности
3f43b74 
идентификаторов элементов форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Исправлен заголовок и javadoc в соответствии с требованиями
47e23b2 
checkstyle
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Методы необходимые для тестирования проверки корректности
3240aca 
элементов форм перенесены из общего кода DT непосредственно в класс
теста проерки
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Добавлена возможность назначить новое значение для незаданных и
7049800 
дублирующихся идентификаторов элеметов форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Уменьшен вывод неинтересной трассировочной информации в
bce8c7a 
исправлении неверных идентификаторов элементов форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Исправлено отсутствие обновления списка ошибок при удалении
eaf5ff5 
элемента формы с дублирующимся идентификатором
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Добавлено пакетное исправление идентификаторов элементов форм
c074423 
при оптимизации проекта
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Добавлены дополнительные тесты для проверки и исправления
81a46d3 
идентификаторов элементов форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Обновлён список изменений для отражения добавления проверки и
9bc8006 
исправления дублирующихся идентификаторов элементов форм
nikolay-martynov added a commit that referenced this issue Aug 5, 2022
@nikolay-martynov
fix(#7): Применены рекомендации статического анализатора кода
e54c741
@nikolay-martynov nikolay-martynov mentioned this issue Aug 5, 2022
Merged
7 tasks
nikolay-martynov added a commit that referenced this issue Aug 8, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
eecdb82 
ревью

- Исправлены опечатки в javadoc
- Конфигурирование через поля заменено на конфигурирование через
конструктор
nikolay-martynov added a commit that referenced this issue Aug 8, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
a2cc9f1 
ревью

- Исправлена работа с транзакциями
nikolay-martynov added a commit that referenced this issue Aug 8, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
e129917 
ревью

- Исправлена ошибка в реализации по умолчанию в быстрой проверке
делегирующей полной проверке
nikolay-martynov added a commit that referenced this issue Aug 9, 2022
@nikolay-martynov
fix(#7): Убрана лишняя трассировка в проверках идентификаторов элементов
98e1517 
форм
nikolay-martynov added a commit that referenced this issue Aug 9, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
7ae1fcc 
ревью

- Убран ненужный интерфейс
- Проверка редактируемости конфигурации заменена на проверку
редактируемости каждой конкретной формы
- Убраны лишние проверки на безопасность запуска очистки
- Исправлено лишнее повышение версии в истории изменений
- Убраны лишние методы, которые запутывают в коде и ухудшают анализ
- Добавлен отдельный модуль для внутренних сервисов
- Улучшено название группы настроек трассировки
- Код сделан чуть понятней
nikolay-martynov added a commit that referenced this issue Aug 10, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
11e0c85 
ревью

- Убрана диагностическая трассировка выполняемых операций
- Убраны диагностические проверки соответствия параметров контрактам
методов
nikolay-martynov added a commit that referenced this issue Aug 10, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
b39e293 
ревью

- Код стал быстрее и понятнее
nikolay-martynov added a commit that referenced this issue Aug 10, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
7c7e0a8 
ревью

- Код стал быстрее и понятнее
nikolay-martynov added a commit that referenced this issue Aug 15, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
3e01acb 
ревью

- Использование Java Stream API заменено на классические циклы в виду
1C-Company/v8-code-style#1094
nikolay-martynov added a commit that referenced this issue Aug 24, 2022
@nikolay-martynov
fix(#7): Исправления по результатам ревью
79b64d3 
- Убран интерфейс, не соответствующий принципам ООП
nikolay-martynov added a commit that referenced this issue Aug 24, 2022
@nikolay-martynov
fix(#7): Рефакторинг кода проверок идентификаторов форм по результатам
8c5d2a4 
ревью
nikolay-martynov added a commit that referenced this issue Aug 29, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
1eeb407 
ревью

- Исправлена опечатка в сообщении об ошибке конфигурации
- Добавленные интеграционные тесты проверок
nikolay-martynov added a commit that referenced this issue Aug 30, 2022
@nikolay-martynov
fix(#7): Исправления кода проверок идентификаторов форм по результатам
23a86a4 
ревью

- Общие методы вынесены в
com._1c.g5.v8.dt.form.service.FormIdentifierService
nikolay-martynov added a commit that referenced this issue Aug 30, 2022
@nikolay-martynov
fix(#7): Исправления по результатам ревью
86ac0d3 
- В тестах запускается один конкретный клинап, а не все
nikolay-martynov added a commit that referenced this issue Aug 31, 2022
@nikolay-martynov
Merge pull request #37 from 1C-Company/bugfix/7-duplicate-id
78d4223 
Bugfix/#7-duplicate-id
@MaksimDzyuba MaksimDzyuba added this to the 0.3 для EDT 2022.2 milestone Aug 31, 2022
@marmyshev marmyshev mentioned this issue Oct 6, 2022
Closed
@BarinovIN BarinovIN mentioned this issue Feb 20, 2023
Open
@agibalovsa agibalovsa mentioned this issue Mar 14, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nikolay-martynov nikolay-martynov

Labels
enhancement New feature or request form Валидация модели формы
Projects
None yet
Milestone
0.3 для EDT 2022.2
Development

Successfully merging a pull request may close this issue.

Bugfix/#7-duplicate-id 1C-Company/dt-project-checks
4 participants
@AlmazNasibullin @marmyshev @nikolay-martynov @MaksimDzyuba