Frontmatter
| id | 1764 |
| title | component.Base: _controller => store the instance |
| state | Closed |
| labels | enhancement |
| assignees | tobiu |
| createdAt | Apr 13, 2021, 11:06 PM |
| updatedAt | Apr 13, 2021, 11:07 PM |
| githubUrl | https://github.com/neomjs/neo/issues/1764 |
| author | tobiu |
| commentsCount | 0 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Apr 13, 2021, 11:07 PM |
right now _controller contains the controller id and beforeGetController() resolves the instance.
we can remove this lookup and store the instance directly (same way as _model).
component.Base: destroy() will destroy the controller anyway, so there should not be any memory leaks.