Made a number of spacing changes to bring the code more-inline with the jQuery Core...

[jquery.git] / src / data.js

diff --git a/src/data.js b/src/data.js
index 1d993fe..e3dc190 100644 (file)
--- a/src/data.js
+++ b/src/data.js
@@ -6,7 +6,19 @@ jQuery.extend({
        
        expando:expando,
 
+       // The following elements throw uncatchable exceptions if you
+       // attempt to add expando properties to them.
+       noData: {
+               "embed": true,
+               "object": true,
+               "applet": true
+       },
+
        data: function( elem, name, data ) {
+               if ( elem.nodeName && jQuery.noData[elem.nodeName.toLowerCase()] ) {
+                       return;
+               }
+
                elem = elem == window ?
                        windowData :
                        elem;
@@ -25,24 +37,31 @@ jQuery.extend({
 
                // Avoid generating a new cache unless none exists and we
                // want to manipulate it.
-               if ( cache[ id ] ) {
+               if ( typeof name === "object" ) {
+                       elem[ expando ] = id;
+                       thisCache = cache[ id ] = jQuery.extend(true, {}, name);
+               } else if ( cache[ id ] ) {
                        thisCache = cache[ id ];
                } else if ( typeof data === "undefined" ) {
                        thisCache = emptyObject;
                } else {
                        thisCache = cache[ id ] = {};
                }
-               
+
                // Prevent overriding the named cache with undefined values
                if ( data !== undefined ) {
                        elem[ expando ] = id;
                        thisCache[ name ] = data;
                }
-               
-               return name ? thisCache[ name ] : thisCache;
+
+               return typeof name === "string" ? thisCache[ name ] : thisCache;
        },
 
        removeData: function( elem, name ) {
+               if ( elem.nodeName && jQuery.noData[elem.nodeName.toLowerCase()] ) {
+                       return;
+               }
+
                elem = elem == window ?
                        windowData :
                        elem;
@@ -81,9 +100,14 @@ jQuery.extend({
 });
 
 jQuery.fn.extend({
-       data: function( key, value ){
+       data: function( key, value ) {
                if ( typeof key === "undefined" && this.length ) {
                        return jQuery.data( this[0] );
+
+               } else if ( typeof key === "object" ) {
+                       return this.each(function() {
+                               jQuery.data( this, key );
+                       });
                }
 
                var parts = key.split(".");
@@ -99,14 +123,14 @@ jQuery.fn.extend({
                                this.data( parts[0] ) :
                                data;
                } else {
-                       return this.trigger("setData" + parts[1] + "!", [parts[0], value]).each(function(){
+                       return this.trigger("setData" + parts[1] + "!", [parts[0], value]).each(function() {
                                jQuery.data( this, key, value );
                        });
                }
        },
 
-       removeData: function( key ){
-               return this.each(function(){
+       removeData: function( key ) {
+               return this.each(function() {
                        jQuery.removeData( this, key );
                });
        }