Make sure that .data() events don't bubble, per the discussion in 2e10af143b7eafb7142...

diff --git a/src/data.js b/src/data.js
index eb96b86..d38d47d 100644 (file)
--- a/src/data.js
+++ b/src/data.js
@@ -143,13 +143,18 @@ jQuery.fn.extend({
                        if ( data === undefined && this.length ) {
                                data = jQuery.data( this[0], key );
                        }
+
                        return data === undefined && parts[1] ?
                                this.data( parts[0] ) :
                                data;
+
                } else {
-                       return this.trigger("setData" + parts[1] + "!", [parts[0], value]).each(function() {
+                       return this.each(function() {
+                               var $this = jQuery( this ), args = [ parts[0], value ];
+
+                               $this.triggerHandler( "setData" + parts[1] + "!", args );
                                jQuery.data( this, key, value );
-                               jQuery.event.trigger( "changeData" + parts[1] + "!", [parts[0], value], this );
+                               $this.triggerHandler( "changeData" + parts[1] + "!", args );
                        });
                }
        },

diff --git a/test/unit/data.js b/test/unit/data.js
index 8a7a87b..872bbb7 100644 (file)
--- a/test/unit/data.js
+++ b/test/unit/data.js
@@ -70,7 +70,13 @@ test(".data()", function() {
 
 test(".data(String) and .data(String, Object)", function() {
        expect(27);
-       var div = jQuery("<div/>");
+       var parent = jQuery("<div><div></div></div>"),
+               div = parent.children();
+
+       parent
+               .bind("getData", function(){ ok( false, "getData bubbled." ) })
+               .bind("setData", function(){ ok( false, "setData bubbled." ) })
+               .bind("changeData", function(){ ok( false, "changeData bubbled." ) });
 
        ok( div.data("test") === undefined, "Check for no data exists" );