#5669 closed bug (fixed)
isPlainObject( null/undefined/window ) on IE
Reported by: | Robert Katić | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 1.4 |
Component: | core | Version: | 1.4a1 |
Keywords: | isPlanarObject | Cc: | |
Blocked by: | Blocking: |
Description (last modified by )
On IE:
jQuery.isPlainObject( null ); // throws "'null.nodeType' is null or not an object"
jQuery.isPlainObject( undefined ); // throws "'undefined.nodeType' is null or not an object"
jQuery.isPlainObject( window ); // returns true
Patch NOTE: With this patch isPlanarObject will work with windows too, until an global (window) constructor is not defined.
Change History (8)
comment:1 follow-up: 2 Changed 13 years ago by
Description: | modified (diff) |
---|---|
Summary: | isPlanarObject( null/undefined/window ) on IE → isPlainObject( null/undefined/window ) on IE |
comment:2 Changed 13 years ago by
Replying to dmethvin:
Corrected the spelling of the method name; the patch looks like a good idea.
Thanks, but there is still an isPlanarObject at the end (ups). Too much 3D programming for this week I guess :).
comment:3 Changed 13 years ago by
I made another mistake. On first two cases (null and undefined), isPlainObjects returns true instead of throwing an error. Anyway the patch is fixing that.
comment:6 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:7 Changed 13 years ago by
That commit wasn't enough. IE 8 was still failing so I landed some more checks: http://github.com/jquery/jquery/commit/67b3be24e1215609cca524ee8158fc7e6e80db93
comment:8 Changed 13 years ago by
This latest fix also resolves problems with passing certain ActiveX objects... specifically "Msxml2.XSLTemplate" -- see #6488
Corrected the spelling of the method name; the patch looks like a good idea.