Fix #10299. Add a propHook for oldIE link href.

appendto · Nov 2, 2012 · 240b141 · 240b141
1 parent 9c2f000
commit 240b141
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/attributes.js b/src/attributes.js
@@ -562,6 +562,7 @@ if ( !getSetAttribute ) {
 
 
 // Some attributes require a special call on IE
+// http://msdn.microsoft.com/en-us/library/ms536429%28VS.85%29.aspx
 if ( !jQuery.support.hrefNormalized ) {
 	jQuery.each([ "href", "src", "width", "height" ], function( i, name ) {
 		jQuery.attrHooks[ name ] = jQuery.extend( jQuery.attrHooks[ name ], {
@@ -571,6 +572,13 @@ if ( !jQuery.support.hrefNormalized ) {
 			}
 		});
 	});
+
+	// link's href property should get the full normalized URL (#10299)
+	jQuery.propHooks.href = {
+		get: function( elem, name ) {
+			return elem.getAttribute( name, 4 );
+		}
+	};
 }
 
 if ( !jQuery.support.style ) {

diff --git a/test/unit/attributes.js b/test/unit/attributes.js
@@ -54,7 +54,7 @@ test( "jQuery.propFix integrity test", function() {
 });
 
 test( "attr(String)", function() {
-	expect( 46 );
+	expect( 47 );
 
 	equal( jQuery("#text1").attr("type"), "text", "Check for type attribute" );
 	equal( jQuery("#radio1").attr("type"), "radio", "Check for type attribute" );
@@ -92,6 +92,8 @@ test( "attr(String)", function() {
 		"href": "#5"
 	}).appendTo("#qunit-fixture");
 	equal( jQuery("#tAnchor5").attr("href"), "#5", "Check for non-absolute href (an anchor)" );
+	jQuery("<a id='tAnchor6' href='#5' />").appendTo("#qunit-fixture");
+	equal( jQuery("#tAnchor5").prop("href"), jQuery("#tAnchor6").prop("href"), "Check for absolute href prop on an anchor" );
 
 	// list attribute is readonly by default in browsers that support it
 	jQuery("#list-test").attr( "list", "datalist" );