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
Fix eventService memory leaks #9752
Fix eventService memory leaks #9752
Conversation
…nto v8/bugfix/umbEditorHeader-memoryleak
tinymce.service.js also appears to be leaking via events, leaving tinymce content in memory, but services cant use $scope. |
src/Umbraco.Web.UI.Client/src/common/services/editor.service.js
Outdated
Show resolved
Hide resolved
...Web.UI.Client/src/common/directives/components/content/umbvariantcontenteditors.directive.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @nzdev, I'd noticed memory spikes a while back when opening multiple infinite editors, this should help alleviate that issue. I've reverted the changes in the services as it's not required due to the lifespan of those object.
Prerequisites
If there's an existing issue for this PR then this fixes N/A
Description
What?
Fix memory leaks by unbinding event from $rootscope.
Why?
Memory leaks in the browser when events are not unbound from the $rootscope.
How to test?
Ensure memory freed when navigating the backoffice ui.