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

isEmpty() #169

Open
mightyaleksey opened this issue Oct 19, 2015 · 3 comments
Open

isEmpty() #169

mightyaleksey opened this issue Oct 19, 2015 · 3 comments

Comments

@mightyaleksey
Copy link
Contributor

Привет, хочу понять, почему isEmpty делает еще проверку на дефолтное значение модели?
https://github.com/bem/bem-mvc/blob/master/common.blocks/i-model/__field/i-model__field.js#L227-L229

На деле, я хочу сбросить значение модели, но не получается, так как текущее значение соответствует дефолтному и isEqual возвращает всегда true.

@collapsus
Copy link
Contributor

кажется логика такова: мы создаём пустую модель, и после этого для каждого поля isEmpty должно вернуть true
соответственно при наличии дефолтов в декларации появляется эта дополнительная проверка
в целом да, ситуация получается конфликтная

@mightyaleksey
Copy link
Contributor Author

Спасибо. Я чуть позже понял как это работает. Вначале у меня вызывало некоторое удивление, что поле модели при сбросе принимало значение по-умолчанию и при этом считалось не измененным.

Леша, а скажи пожалуйста, как вы работаете с инпутами? Меня интересует как вы передаете начальные состояния для контролов? От куда-то извне или используете какие-то правила на этапе декларации модели?

@collapsus
Copy link
Contributor

это некоторая боль, пока ещё не решённая, на этапе шаблонизации дописываем в value, и пишем биндинги, т. е. по сути копипастим

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

No branches or pull requests

2 participants