From 7632b7454b8a918b5eb9ade199a6a842eae98571 Mon Sep 17 00:00:00 2001 From: Timmy Willison Date: Mon, 16 Mar 2015 14:11:38 -0400 Subject: [PATCH] Attributes: revert returning null for non-elements --- src/attributes/attr.js | 2 +- test/unit/attributes.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/attributes/attr.js b/src/attributes/attr.js index 201a2d2a6..3bb05a5a3 100644 --- a/src/attributes/attr.js +++ b/src/attributes/attr.js @@ -28,7 +28,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 1163ff4e1..ebc28a0e1 100644 --- 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 ) { -- 2.39.5