search

oknosoft / windowbuilder

Окнософт:Заказ дилера
http://www.oknosoft.ru/zd/
Other
10 stars 19 forks source link

Метод для получения цвета по умолчанию #528

Closed senitskiy closed 5 years ago

senitskiy commented 5 years ago

Получения цвета вставки по умолчанию.

По умолчанию получаю цвет из реквизита вставки 1С "доступность цветов". В реквизите из списка "соответствие цветов" беру самый верхний, если цвета нет, то беру первый цвет из первой цвето-ценовой группы, если и там нет, ищу далее таким же образом доступный цвет.

unpete commented 5 years ago

Предлагаю регламент: pr на новый функционал, без одобренного issue не создаём. Вопрос не в том, хорошо ваше решение или плохо. Чтобы обеспечить качество проекта, я должен быть в курсе проблем. Для этого, надо смотреть не на ваш код, а на задачу, которая этот код породила.

senitskiy commented 5 years ago

ok!

senitskiy commented 5 years ago

Текущую задачу предлагаю здесь обсудить.

unpete commented 5 years ago

Рассказывайте, для чего и в какой момент потребовался цвет по умолчанию. Трудно поверить, что раньше не было возможности его задать.

senitskiy commented 5 years ago

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

unpete commented 5 years ago

Ок, а как цвет по умолчанию попадает в другие места? Где его задаёт технолог? Имеется в виду, цвет элемента в рисовалке, цвет параметрического изделия.

senitskiy commented 5 years ago

Сейчас при добавление аксессуара цвет берётся из умолчаний вставки. Можно сделать по аналогии, но с проверкой указанного цвета в "доступности цветов" данной вставки среди цветов и цвето-ценовых групп.

unpete commented 5 years ago

Ок, сегодня поковыряю код и попробую вспомнить, какие сценарии предполагались 6 лет назад. Должно быть компактно и единообразно.

senitskiy commented 5 years ago

Думаю, лучше совместить вышеуказанную логику с логикой из db_buyers_order.js/$p.DpBuyers_orderProductionRow и вынести в отдельный метод для дальнейшего использования.

unpete commented 5 years ago

Оказалось, что метод надо разместить в цветоценовых группах, а не во вставках, т.к. рекурсия. Для единообразия, задача технологу @A-Shimko, убрать из табчасти умолчаний вставок fake-параметры назначения цвета, проверить группы доступности цветов, при необходимости, передвинуть в них умолчательный цвет вверх списка.

A-Shimko commented 5 years ago

Не, ну все понятно...

unpete commented 5 years ago

Если что, @senitskiy подскажет - он инициатор изменений.