Ticket #7165 (closed bug: fixed)
wrap leaks objects in $.cache
| Reported by: | IgorMinar | Owned by: | |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.5 |
| Component: | manipulation | Version: | 1.4.2 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
If a node is created via wrap, it is not properly removed from the cache when it's no longer part of the DOM.
See attached test case.
Change History
comment:2 Changed 3 years ago by snover
- Priority changed from undecided to blocker
- Component changed from unfiled to manipulation
- Milestone changed from 1.4.2 to 1.4.5
comment:4 Changed 2 years ago by iliakan
Fixed the pull: https://github.com/jquery/jquery/pull/137
comment:5 Changed 2 years ago by Colin Snover
- Status changed from open to closed
- Resolution set to fixed
Fix #7717 and #7165. Thanks to dmethvin and iliakan for their help fixing these issues.
Changeset: faefbb1ad0b81e8001b582d06d5bd9c9236e62ce
comment:6 Changed 2 years ago by Colin Snover
Update cloneCopyEvent so that it does not create superfluous data objects when cloning elements. Exposes a new method, $.hasData. Fixes #7165. Thanks to DaveMethvin and iliakan for their help.
Changeset: f28c774f2cde004a33aefc28306d995c599d963b
comment:7 Changed 2 years ago by Colin Snover
Update cloneCopyEvent so that it does not create superfluous data objects when cloning elements. Exposes a new method, $.hasData. Fixes #7165. Thanks to DaveMethvin and iliakan for their help.
Changeset: f28c774f2cde004a33aefc28306d995c599d963b
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
test case showing the leak