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

[BUG] @prizm-ui/components - [Regression] PrizmInputTextComponent передает null в модель при очистке #1423

Open
1 task done
AleksandrSibiakov opened this issue Mar 4, 2024 · 1 comment
Assignees
Labels
type:bug Something isn't working
Milestone

Comments

@AleksandrSibiakov
Copy link
Contributor

Библиотека

  • @prizm-ui/components

Компонент

PrizmInputTextComponent

Версия

Регрессия 3.2.0

Воспроизведение проблемы

Очистить текстовое поле через кнопку

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

С некоторого времени при очистке поля через кнопку в модель стало приходить значение null, что привело к появлению необработанных исключений в приложениях, где не делается проверка значений NULL-ish значений.

Укажу, что в нативном поведении контрола HTMLInput (и DefaultValueAccessor соответственно) никогда не приходит null
image

@AleksandrSibiakov
Copy link
Contributor Author

Хочу добавить, что для фикса возможно надо пересмотреть подход компонента к работе со значениями в целом.
В баге #1190 так же проблемы из за того что значение отправляется напрямую в модель FormControl (NgControl) - это должно делаться через ControlValueAccessor интерфейс, к которому указанный класс (PrizmInputTextComponent) не относится. ИМХО все манипуляции со значениями должны производиться через HTMLInput элемент и его события, которые будут зарегестрированы и обработаны соответствующим CVA, будь то стандартный DefaultValueAccessor или NgxMaskDirective или любой другой

@alexhawkins94 alexhawkins94 added the type:bug Something isn't working label Mar 4, 2024
@ZurabDev ZurabDev added this to the 4.1.0 milestone Mar 5, 2024
@ZurabDev ZurabDev self-assigned this Mar 13, 2024
@imonogarov imonogarov modified the milestones: 4.1.0, 4.7.0 May 24, 2024
@ZurabDev ZurabDev modified the milestones: 4.7.0, 4.5.2 Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants