Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

ns.Model.destroy не убивает все обрабочтики на себе #183

Closed
doochik opened this Issue · 6 comments

3 participants

@doochik
Owner

Может стоит сделать?
Хотя с новой логикой, что она не удаляет себя, а инвалидирует, не уверен, что это правильно.

@chestozo @edoroshenko

@edoroshenko
Owner

Сейчас сложилось так, что init у модели дёргается только один раз. Даже после destroy и setData он второй раз не вызовется. Если вешать обработчик на init, а снимать на destroy, получится, что при пересоздании модели с таким же ключом обработчик не перенавесится.

Тут 2 пути:

  • при destroy устанавливать модели статус none и при следующем set дёргать init. Тогда обработчики можно снимать на destroy
  • не снимать обработчики на destroy атоматом, а делать это осознанно на уровне приложения

Первый путь выглядит более здраво, но это нужно делать

@chestozo
Owner

Надо, чтобы init дёргался. Потому что destroy это должен быть настоящий destroy. Зачем нам полуdestroy?

@edoroshenko
Owner

halfDestroy )

@chestozo
Owner

Не, ну правда )

@edoroshenko
Owner

Да, Роман, ты прав, надо сделать

@edoroshenko edoroshenko added the pain label
@edoroshenko
Owner

делаем в #224

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.