If no hook is provided, and a boolean property exists, use that to return an attribute-style value for boolean attributes. Fixes #9079.

1 files changed, 14 insertions, 7 deletions

diff --git a/src/attributes.js b/src/attributes.js
index 2da2e1639..9db69abcc 100644
--- a/src/attributes.js
+++ b/src/attributes.js
@@ -285,8 +285,7 @@ jQuery.extend({
 	
 	attrFix: {
 		// Always normalize to ensure hook usage
-		tabindex: "tabIndex",
-		readonly: "readOnly"
+		tabindex: "tabIndex"
 	},
 	
 	attr: function( elem, name, value, pass ) {
@@ -327,20 +326,25 @@ jQuery.extend({
 
 				// Set boolean attributes to the same name
 				if ( value === true && !rspecial.test( name ) ) {
-					value = name;
+					value = name.toLowerCase();
 				}
 
 				elem.setAttribute( name, "" + value );
 				return value;
 			}
 
+		} else if ( hooks && "get" in hooks && notxml ) {
+			return hooks.get( elem, name );
+
 		} else {
+			var boolProp = elem[ jQuery.propFix[ name ] || name ];
 
-			if ( hooks && "get" in hooks && notxml ) {
-				return hooks.get( elem, name );
+			if ( typeof boolProp === "boolean" ) {
+				return boolProp ?
+					name.toLowerCase() :
+					undefined;
 
 			} else {
-
 				ret = elem.getAttribute( name );
 
 				// Non-existent attributes return null, we normalize to undefined
@@ -399,7 +403,9 @@ jQuery.extend({
 		}
 	},
 	
-	propFix: {},
+	propFix: {
+		readonly: "readOnly"
+	},
 	
 	prop: function( elem, name, value ) {
 		var nType = elem.nodeType;
@@ -441,6 +447,7 @@ jQuery.extend({
 // IE6/7 do not support getting/setting some attributes with get/setAttribute
 if ( !jQuery.support.getSetAttribute ) {
 	jQuery.attrFix = jQuery.extend( jQuery.attrFix, {
+		readonly: "readOnly",
 		"for": "htmlFor",
 		"class": "className",
 		maxlength: "maxLength",