Attributes: revert returning null for non-elements

diff --git a/src/attributes/attr.js b/src/attributes/attr.js
index 716f4fe25a44f7a54168abc1516c736fa3db6202..f5c8dd598f83716b82605e629e3c0264689667a7 100644 (file)
--- a/src/attributes/attr.js
+++ b/src/attributes/attr.js
@@ -31,7 +31,7 @@ jQuery.extend({
 
                // don't get/set attributes on text, comment and attribute nodes
                if ( !elem || nType === 3 || nType === 8 || nType === 2 ) {
-                       return null;
+                       return;
                }
 
                // Fallback to prop when attributes are not supported

diff --git a/test/unit/attributes.js b/test/unit/attributes.js
index ebe171bd9b7af6c2a7fadb2592e8a3b9bb0b8dc2..0ac1345e696f00b01fc4af8273e05ff27a1e0405 100644 (file)
--- a/test/unit/attributes.js
+++ b/test/unit/attributes.js
@@ -371,7 +371,7 @@ test( "attr(String, Object)", function() {
        jQuery.each( [ commentNode, textNode, attributeNode ], function( i, elem ) {
                var $elem = jQuery( elem );
                $elem.attr( "nonexisting", "foo" );
-               strictEqual( $elem.attr("nonexisting"), null, "attr(name, value) works correctly on comment and text nodes (bug #7500)." );
+               strictEqual( $elem.attr("nonexisting"), undefined, "attr(name, value) works correctly on comment and text nodes (bug #7500)." );
        });
 
        jQuery.each( [ window, document, obj, "#firstp" ], function( i, elem ) {