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

form-field_type_checkbox setVal() #225

Open
belozer opened this issue Sep 18, 2017 · 8 comments
Open

form-field_type_checkbox setVal() #225

belozer opened this issue Sep 18, 2017 · 8 comments
Labels

Comments

@belozer
Copy link
Member

belozer commented Sep 18, 2017

При попытке вызвать setVal() будет ошибка, т.к. у checkbox нет этого api.

@belozer belozer added the bug label Sep 18, 2017
@belozer belozer changed the title form-field_type_checkbox form-field_type_checkbox setVal() Sep 18, 2017
@belozer
Copy link
Member Author

belozer commented Sep 21, 2017

Возможно стоит доопределить checkbox этим методом, но не думаю что это хорошая идея. Другого выхода пока не вижу (локально у себя так сделали).

@belozer belozer closed this as completed Sep 28, 2017
@belozer belozer reopened this Sep 28, 2017
@belozer
Copy link
Member Author

belozer commented Sep 28, 2017

https://github.com/bem-contrib/bem-forms/blob/v1.0.0/common.blocks/form-field/_type/form-field_type_checkbox.browser.js#L27

здесь дёргается вызов базового метода, что пытается сделать установку значения на контроле

@awinogradov
Copy link
Member

Возможно стоит доопределить checkbox этим методом, но не думаю что это хорошая идея.

Звучит вполне разумно, почему нет?

@belozer
Copy link
Member Author

belozer commented Nov 7, 2017

@awinogradov не очень люблю делать переопределение поведения библиотек в других библиотеках)

@rtemision
Copy link

С attach та же история.

@rtemision
Copy link

Но для него, я полагаю, надо исключение делать на невыполнение setVal().

@belozer
Copy link
Member Author

belozer commented Feb 21, 2018

Возможно стоит немного изменить логику с setVal, для удобного (без дублирования логики) переопределения в модификаторах (form-field_type_checkbox, form-field_type_attach).

@belozer
Copy link
Member Author

belozer commented Feb 21, 2018

Таким образом не нужно вмешиваться в логику базовых блоков.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants