form-field_type_checkbox setVal()

[belozer]

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

[belozer]

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

[belozer]

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

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

[awinogradov]

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

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

[belozer]

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

[rtemision]

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

[rtemision]

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

[belozer]

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

[belozer]

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