]> source.dussan.org Git - jquery.git/commitdiff
Ticket #6808. Changes data() so on plain objects, it uses a function to contain the...
authorInfinitiesLoop <dareed@microsoft.com>
Tue, 20 Jul 2010 21:53:36 +0000 (05:53 +0800)
committerJohn Resig <jeresig@gmail.com>
Thu, 22 Jul 2010 19:52:01 +0000 (03:52 +0800)
src/data.js
test/unit/data.js

index f7af18d0d526f9e43f8c90158621718077b26dca..6dcfb4c2278ca44ac595a2538ebd9e43b18f4ce6 100644 (file)
@@ -46,13 +46,30 @@ jQuery.extend({
                // Avoid generating a new cache unless none exists and we
                // want to manipulate it.
                if ( typeof name === "object" ) {
-                       cache[ id ] = jQuery.extend(true, {}, name);
+                       if ( isNode ) {
+                               cache[ id ] = jQuery.extend(true, {}, name);
+                       } else {
+                               cache[ id ] = function() {
+                                       return jQuery.extend(true, {}, name);
+                               }
+                       }
 
                } else if ( !cache[ id ] ) {
-                       cache[ id ] = {};
+                       if ( isNode ) {
+                               cache[ id ] = {};
+                       } else {
+                               var store = {};
+                               cache[ id ] = function() {
+                                       return store;
+                               }
+                       }
+                       
                }
 
                thisCache = cache[ id ];
+               if ( !isNode ) {
+                       thisCache = thisCache();
+               }
 
                // Prevent overriding the named cache with undefined values
                if ( data !== undefined ) {
@@ -71,8 +88,12 @@ jQuery.extend({
                        windowData :
                        elem;
 
-               var id = elem[ jQuery.expando ], cache = jQuery.cache,
-                       isNode = elem.nodeType, thisCache = isNode ? cache[ id ] : id;
+               var isNode = elem.nodeType,
+                       id = elem[ jQuery.expando ], cache = jQuery.cache;
+               if ( id && !isNode ) {
+                       id = id();
+               }
+               var thisCache = cache[ id ];
 
                // If we want to remove a specific section of the element's data
                if ( name ) {
index d5eb73b080ce31aa6c18e1faaeb009fadd3d4f7d..8a7a87b9d72bfdafb4e3f70895b84849b9686006 100644 (file)
@@ -1,13 +1,14 @@
 module("data");
 
 test("expando", function(){
-       expect(6);
+       expect(7);
        
        equals("expando" in jQuery, true, "jQuery is exposing the expando");
        
        var obj = {};
        jQuery.data(obj);
        equals( jQuery.expando in obj, true, "jQuery.data adds an expando to the object" );
+       equals( typeof obj[jQuery.expando], "function", "jQuery.data adds an expando to the object as a function" );
 
        obj = {};       
        jQuery.data(obj, 'test');
@@ -17,7 +18,7 @@ test("expando", function(){
        jQuery.data(obj, "foo", "bar");
        equals( jQuery.expando in obj, true, "jQuery.data added an expando to the object" );
        
-       var id = obj[jQuery.expando];
+       var id = obj[jQuery.expando]();
        equals( id in jQuery.cache, false, "jQuery.data did not add an entry to jQuery.cache" );
        
        equals( id.foo, "bar", "jQuery.data worked correctly" );
@@ -54,7 +55,7 @@ test("jQuery.data", function() {
        jQuery.data( obj, "prop", true );
 
        ok( obj[ jQuery.expando ], "Data is being stored on the object." );
-       ok( obj[ jQuery.expando ].prop, "Data is being stored on the object." );
+       ok( obj[ jQuery.expando ]().prop, "Data is being stored on the object." );
 
        equals( jQuery.data( obj, "prop" ), true, "Make sure the right value is retrieved." );
 });