Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Commit eac3bfb

Browse files
imdarkmmalerba
authored andcommitted
fix(interimElement): added missing scope dispose to fix memory leak (#9710)
1 parent 38eaa52 commit eac3bfb

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/core/services/interimElement/interimElement.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -724,6 +724,11 @@ function InterimElementProvider() {
724724
if (options.$destroy) {
725725
// For $destroy, onRemove should be synchronous
726726
resolve(element);
727+
728+
if (!options.preserveScope && options.scope ) {
729+
// scope destroy should still be be done after the current digest is done
730+
action.then( function() { options.scope.$destroy(); });
731+
}
727732
} else {
728733
// Wait until transition-out is done
729734
action.then(function () {

0 commit comments

Comments
 (0)