Standardize on using double-quotes for string literals.

[jquery.git] / src / attributes.js

diff --git a/src/attributes.js b/src/attributes.js
index 0509781..787c6bb 100644 (file)
--- a/src/attributes.js
+++ b/src/attributes.js
@@ -137,19 +137,19 @@ jQuery.fn.extend({
 
                // Typecast once if the value is a number
                if ( typeof value === "number" ) {
-                       value += '';
+                       value += "";
                }
 
                var val = value;
 
-               return this.each(function(){
+               return this.each(function() {
                        if ( jQuery.isFunction(value) ) {
                                val = value.call(this);
 
                                // Typecast each time if the value is a Function and the appended
                                // value is therefore different each time.
                                if ( typeof val === "number" ) {
-                                       val += '';
+                                       val += "";
                                }
                        }
 
@@ -163,7 +163,7 @@ jQuery.fn.extend({
                        } else if ( jQuery.nodeName( this, "select" ) ) {
                                var values = jQuery.makeArray(val);
 
-                               jQuery( "option", this ).each(function(){
+                               jQuery( "option", this ).each(function() {
                                        this.selected = jQuery.inArray( this.value || this.text, values ) >= 0;
                                });
 
@@ -210,10 +210,12 @@ jQuery.each({
                        this.className = this.className || classNames === false ? "" : jQuery.data( this, "__className__" ) || "";
                }
        }
-}, function(name, fn){
-       jQuery.fn[ name ] = function(val, state){
+}, function( name, fn ) {
+       jQuery.fn[ name ] = function( val, state ) {
                if ( jQuery.isFunction( val ) ) {
-                       return this.each(function() { jQuery(this)[ name ]( val.call(this), state ); });
+                       return this.each(function() {
+                               jQuery(this)[ name ]( val.call(this), state );
+                       });
                }
 
                return this.each( fn, arguments );
@@ -232,13 +234,13 @@ jQuery.extend({
                offset: true
        },
                
-       attr: function( elem, name, value ) {
+       attr: function( elem, name, value, pass ) {
                // don't set attributes on text and comment nodes
                if ( !elem || elem.nodeType === 3 || elem.nodeType === 8 ) {
                        return undefined;
                }
 
-               if ( name in jQuery.attrFn && value !== undefined ) {
+               if ( pass && name in jQuery.attrFn ) {
                        return jQuery(elem)[name](value);
                }
 
@@ -268,14 +270,7 @@ jQuery.extend({
                                                throw "type property can't be changed";
                                        }
 
-                                       // browsers index elements by id/name on forms, give priority to attributes.
-                                       if( jQuery.nodeName( elem, "form" ) ) {
-                                               // convert the value to a string (all browsers do this but IE) see #1070
-                                               elem.setAttribute( name, "" + value );
-
-                                       } else {
-                                               elem[ name ] = value;
-                                       }
+                                       elem[ name ] = value;
                                }
 
                                // browsers index elements by id/name on forms, give priority to attributes.