]> source.dussan.org Git - jquery.git/commitdiff
You can now overwrite values returned from .data() with .bind("getData") - returning...
authorJohn Resig <jeresig@gmail.com>
Sun, 3 Feb 2008 18:43:04 +0000 (18:43 +0000)
committerJohn Resig <jeresig@gmail.com>
Sun, 3 Feb 2008 18:43:04 +0000 (18:43 +0000)
src/core.js
test/unit/core.js

index 6997f89073707d483df2deb4f53a36341b649e5d..efc654fb409444addbdc760824f90a2c878eb9a4 100644 (file)
@@ -480,16 +480,19 @@ jQuery.fn = jQuery.prototype = {
 
        data: function( key, value ){
                var parts = key.split(".");
+               parts[1] = parts[1] ? "." + parts[1] : "";
 
                if ( value == null ) {
-                       if ( this.length ) {
-                               var data = jQuery.data( this[0], key );
-                               return data == null ?
-                                       jQuery.data( this[0], parts[0] ) :
-                                       data;
-                       }
+                       var data = this.triggerHandler("getData" + parts[1] + "!", [parts[0]]);
+                       
+                       if ( data == undefined && this.length )
+                               data = jQuery.data( this[0], key );
+
+                       return data == null && parts[1] ?
+                               this.data( parts[0] ) :
+                               data;
                } else
-                       return this.trigger("setData" + (parts[1] ? "." + parts[1] : "") + "!", [parts[0], value]).each(function(){
+                       return this.trigger("setData" + parts[1] + "!", [parts[0], value]).each(function(){
                                jQuery.data( this, key, value );
                        });
        },
index 3ca1e9510c2cfa96d6948ba406b037030a710b2a..3cb90aa7886b7be41fa13f487b7764151164faa5 100644 (file)
@@ -1398,7 +1398,7 @@ test("$.data", function() {
 });
 
 test(".data()", function() {
-       expect(11);
+       expect(16);
        var div = $("#foo");
        ok( div.data("test") == undefined, "Check for no data exists" );
        div.data("test", "success");
@@ -1406,25 +1406,41 @@ test(".data()", function() {
        div.data("test", "overwritten");
        ok( div.data("test") == "overwritten", "Check for overwritten data" );
 
-       var hits = {test:0};
+       var hits = {test:0}, gets = {test:0};
 
        div
                .bind("setData",function(e,key,value){ hits[key] += value; })
                .bind("setData.foo",function(e,key,value){ hits[key] += value; })
+               .bind("getData",function(e,key){ gets[key] += 1; })
+               .bind("getData.foo",function(e,key){ gets[key] += 3; });
 
        div.data("test.foo", 2);
        ok( div.data("test") == "overwritten", "Check for original data" );
        ok( div.data("test.foo") == 2, "Check for namespaced data" );
        ok( div.data("test.bar") == "overwritten", "Check for unmatched namespace" );
-       ok( hits.test == 2, "Check triggered functions" );
+       equals( hits.test, 2, "Check triggered setter functions" );
+       equals( gets.test, 5, "Check triggered getter functions" );
 
        hits.test = 0;
+       gets.test = 0;
 
        div.data("test", 1);
        ok( div.data("test") == 1, "Check for original data" );
        ok( div.data("test.foo") == 2, "Check for namespaced data" );
        ok( div.data("test.bar") == 1, "Check for unmatched namespace" );
-       ok( hits.test == 1, "Check triggered functions" );
+       equals( hits.test, 1, "Check triggered setter functions" );
+       equals( gets.test, 5, "Check triggered getter functions" );
+
+       hits.test = 0;
+       gets.test = 0;
+
+       div
+               .bind("getData",function(e,key){ return key + "root"; })
+               .bind("getData.foo",function(e,key){ return key + "foo"; });
+
+       ok( div.data("test") == "testroot", "Check for original data" );
+       ok( div.data("test.foo") == "testfoo", "Check for namespaced data" );
+       ok( div.data("test.bar") == "testroot", "Check for unmatched namespace" );
 });
 
 test("$.removeData", function() {