wrap leaks objects in $.cache

Reported by:	IgorMinar	Owned by:
Priority:	blocker	Milestone:	1.5
Component:	manipulation	Version:	1.4.2
Keywords:		Cc:
Blocked by:		Blocking:

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.

Attachments (1)

temp.html (0.8 KB) - added by IgorMinar October 12, 2010 05:38PM UTC.

test case showing the leak

Change History (8)

Changed October 14, 2010 05:17AM UTC by snover comment:1

status:	new → open

live test case

Changed October 23, 2010 11:05PM UTC by snover comment:2

component:	unfiled → manipulation
milestone:	1.4.2 → 1.4.5
priority:	undecided → blocker

Changed December 18, 2010 07:29PM UTC by iliakan comment:3

Fix: https://github.com/jquery/jquery/pull/135

Changed December 19, 2010 12:04AM UTC by iliakan comment:4

Fixed the pull: https://github.com/jquery/jquery/pull/137

Changed December 23, 2010 12:21AM UTC by Colin Snover comment:5

resolution:	→ fixed
status:	open → closed

Fix #7717 and #7165. Thanks to dmethvin and iliakan for their help fixing these issues.

Changeset: faefbb1ad0b81e8001b582d06d5bd9c9236e62ce

Changed December 23, 2010 12:21AM UTC by Colin Snover comment:6

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

Changed December 24, 2010 12:05PM UTC by Colin Snover comment:7