Opened 14 years ago
Closed 14 years ago
#3737 closed bug (fixed)
TypeError: ret[i].parentNode is null
Reported by: | malsup | Owned by: | flesler |
---|---|---|---|
Priority: | major | Milestone: | 1.3 |
Component: | core | Version: | |
Keywords: | Cc: | malsup | |
Blocked by: | Blocking: |
Description
TypeError occurred at line 1017 of 1.3b1 while using Taconite plugin to inject a script element into the document head.
This screenshot shows the error in Firebug: http://jquery.malsup.com/test13/test-taconite.png
Test page here: http://localhost:8888/malsup/test13/test-taconite-1.3b1.html
Same test using 1.2.6: http://localhost:8888/malsup/test13/test-taconite-1.2.6.html
I tried for quite a while to prepare a small test case but couldn't seem to get the same error to occur without using Taconite. :-(
Change History (7)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
need: | Review → Patch |
---|---|
Owner: | set to flesler |
The problem only occurs if a script DOM element is passed into jQuery.clean:
$(document.createElement("script")).appendTo("head");
The script processing at line 994 only happens if the incoming argument is a string; it needs to be moved outside that test ... er, somehow. :-)
comment:3 Changed 14 years ago by
Cc: | malsup added |
---|---|
need: | Patch → Commit |
Status: | new → assigned |
comment:6 Changed 14 years ago by
I created a simpler test case for this. The problem is in the assumption that a script element has a parent node. This is not true for script elements that have been created but not inserted into the DOM.
comment:7 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Fixed in SVN rev [6078].
Ugh, those links should point to:
http://jquery.malsup.com/test13/test-taconite-1.3b1.html and http://jquery.malsup.com/test13/test-taconite-1.2.6.html