LearnNewsExamplesServices
Frontmatter
id1768
titlecomponent.Base: destroy() => model.removeBindings()
stateClosed
labels
enhancement
assigneestobiu
createdAtApr 14, 2021, 12:40 PM
updatedAtApr 14, 2021, 12:52 PM
githubUrlhttps://github.com/neomjs/neo/issues/1768
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtApr 14, 2021, 12:52 PM

component.Base: destroy() => model.removeBindings()

Closed v8.1.0 enhancement
tobiu
tobiu commented on Apr 14, 2021, 12:40 PM

we can reduce the amount of calls by probably a lot in case we check for a bind config first.

also, we can simplify the parent model access (same way as for controllers).

tobiu added the enhancement label on Apr 14, 2021, 12:40 PM
tobiu assigned to @tobiu on Apr 14, 2021, 12:40 PM
tobiu referenced in commit b80b05b - "component.Base: destroy() => model.removeBindings() #1768" on Apr 14, 2021, 12:44 PM
tobiu referenced in commit c3b343d - "#1768 we need to call removeBindings() on the closest parent model unrelated of destroying an own model." on Apr 14, 2021, 12:50 PM
tobiu closed this issue on Apr 14, 2021, 12:52 PM