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.

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

очистка незаполненных строк состава заполнения #536

Merged
merged 2 commits into from
Aug 5, 2019

Conversation

rnpoddor
Copy link
Contributor

@rnpoddor rnpoddor commented Aug 3, 2019

Случаются ошибки отсутствия стеклопакета в изделии по причине пустой строки в составе заполнения. Менеджер после редактирования состава заполнения оставляет пустую строку, у которой не выбрана вставка, в результате в спецификацию стеклопакет не выписывается.

@unpete
Copy link
Member

unpete commented Aug 3, 2019

Почему решили разместить в форме а не в dataobj или калькуляторе спецификации?

@rnpoddor
Copy link
Contributor Author

rnpoddor commented Aug 4, 2019

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

Решение через dataobj так и не нашел.

@rnpoddor
Copy link
Contributor Author

rnpoddor commented Aug 4, 2019

Пустые строки в glass_specification не влияют на формирование формулы и расчета толщины.

@rnpoddor
Copy link
Contributor Author

rnpoddor commented Aug 4, 2019

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

@unpete
Copy link
Member

unpete commented Aug 5, 2019

PR приму. Бизнес-логику в формах вполне можно терпеть.
Добавил задачу oknosoft/metadata.js#442
Нужна унификация, которая позволит выкинуть из клиентских приложений разную чушь про вставки, номенклатуру, подразделения и прочие явные реквизиты и переложить контроль на платформу.

@unpete unpete merged commit a888a72 into oknosoft:master Aug 5, 2019
@rnpoddor rnpoddor deleted the feature2 branch August 5, 2019 07:11
unpete added a commit that referenced this pull request Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants