IE "Invalid Argument" $('table').attr('non-existent')

[jdsharp]

IE 6 throws an "Invalid Argument" error when attempting to get an attribute from a table element. Functions as desired with a div (returns undefined). Fix attached. (Is broken in 1.1.3.1, 1.1.4, 1.2)

[jdsharp]

Test case

[jdsharp]

Changes at line 860

[jdsharp]

Some more research reveled that this only occurs when the attribute in question contains a ':'. Updated patch as follows:

// Fixes a getAttribute 'Invalid Exception' error when getting an attribute // with a ':' in it on a table element when the attribute does not exist // http://www.andrewdupont.net/2007/01/10/code-hasattribute-for-ie/ if ( jQuery.browser.msie ) {

if ( /href|src/.test(name) && !jQuery.isXMLDoc(elem) )

return elem.getAttribute( name, 2 );

if ( name.indexOf(':') > -1 )

return ( elem.attributes[name] != undefined ? elem.getAttribute(name) : undefined );

} return elem.getAttribute( name );

[digitalspaghetti]

Replying to jdsharp:

Some more research reveled that this only occurs when the attribute in question contains a ':'. Updated patch as follows: // Fixes a getAttribute 'Invalid Exception' error when getting an attribute // with a ':' in it on a table element when the attribute does not exist // http://www.andrewdupont.net/2007/01/10/code-hasattribute-for-ie/ if ( jQuery.browser.msie ) { if ( /href|src/.test(name) && !jQuery.isXMLDoc(elem) ) return elem.getAttribute( name, 2 ); if ( name.indexOf(':') > -1 ) return ( elem.attributes[name] != undefined ? elem.getAttribute(name) : undefined ); } return elem.getAttribute( name );

Tried the above patch, does not fix the error in IE6, however this did fix a problem I was having in Safari on Windows XP. I was trying to use hover() and hoverIntent() on an unordered list, and neither worked - however this fix solved the problem

[jdsharp]

This bug is still present in 1.2.2 and the same patch still applies.

[flesler]

Tested this with the trunk version, and it works fine. This was probably fixed with the recent massive modifications to .attr().

I tested on IE6.

[jdsharp]

This issue still existed in 1.2.6, I've created a patch against trunk

[jdsharp]

Patch against 5711