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

Не destroy-ить от-destroy-енный вид #616

Open
chestozo opened this issue Aug 12, 2016 · 4 comments
Open

Не destroy-ить от-destroy-енный вид #616

chestozo opened this issue Aug 12, 2016 · 4 comments
Labels

Comments

@chestozo
Copy link
Member

Кейс:

  • есть вид-коллекция
  • вызывается у элемента коллекции destroy
  • это приводит к тому, что вид-коллекция дополнительно вызывает destroy на уже уничтоженном элементе
  • повторно вызывается обработчик ns-view-destroyed, в котором уже нельзя обратиться к моделями вида, к примеру

Предложение:

  • в destroy проверять флаг destroyed у вида и ничего не делать, если вид уже уничтожен.
@chestozo
Copy link
Member Author

@vitkarpov @Katochimoto

@vitkarpov
Copy link
Member

вызывается у элемента коллекции destroy —> это приводит к тому, что вид-коллекция дополнительно вызывает destroy на уже уничтоженном элементе

Не понял вот этот переход. Можешь пояснить более подробнее? И, кстати, кто вызывает destroy отдельного вида?

@chestozo
Copy link
Member Author

вызывается у элемента коллекции destroy —> это приводит к тому, что вид-коллекция дополнительно вызывает destroy на уже уничтоженном элементе

Не понял вот этот переход. Можешь пояснить более подробнее?

Я пока не вычленил чёткий кейс, но, показалось, что надо прикрыть возможность 2 раза вызвать destroy на виде.

кто вызывает destroy отдельного вида?

в данном случае я руками вызываю, но у нас в логах есть случай, когда это делает кто-то другой )

@vitkarpov
Copy link
Member

надо прикрыть возможность 2 раза вызвать destroy на виде.

Ну да, согласен. Это кажется правильным в любом случае.

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

2 participants