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

Potential Memoryleak #62

Closed
oweitman opened this issue Mar 16, 2020 · 0 comments · Fixed by ioBroker/create-adapter#437
Closed

Potential Memoryleak #62

oweitman opened this issue Mar 16, 2020 · 0 comments · Fixed by ioBroker/create-adapter#437

Comments

@oweitman
Copy link
Contributor

oweitman commented Mar 16, 2020

https://github.com/ioBroker/ioBroker.template/blob/9ab757f8f72d429c38bf9ea33529d83260073ca4/JavaScriptVIS/widgets/template/js/template.js#L47

If the widget is recreated or redrawn by vis the previous bounds are not automaticly deleted by vis/canJS and leads to a memory leak
redraw/recreation is triggered for example if you use binding.
the bound states should manually unbound with the unbind command or should
added to the data('bound') property of the widget. data('bindHandler') should also set
example: iframe widget of the basic set https://github.com/ioBroker/ioBroker.vis/blob/2a08ee6da626a65b9d0b42b8679563e74272bfc6/www/widgets/basic.html#L380

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant