Landing pull request 377. Check custom data != null(undefined), allows zero; Fixes...

More Details:
 - https://github.com/jquery/jquery/pull/377
 - http://bugs.jquery.com/ticket/9285

diff --git a/src/event.js b/src/event.js
index 2bed09046c8f6d4d4e3ba6a68995a2224e03e237..d2d57d676631a773a874951408ac2655926af53f 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -345,7 +345,7 @@ jQuery.event = {
                event.target = elem;
 
                // Clone any incoming data and prepend the event, creating the handler arg list
-               data = data ? jQuery.makeArray( data ) : [];
+               data = data != null ? jQuery.makeArray( data ) : [];
                data.unshift( event );
 
                var cur = elem,

diff --git a/test/unit/event.js b/test/unit/event.js
index 13877e019118b8032feaee70a3cef46e45627846..f71c7b29feff56b3948fe26c8ee246cf2a06c4d1 100644 (file)
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -14,6 +14,26 @@ test("null or undefined handler", function() {
        } catch (e) {}
 });
 
+test("bind(),live(),delegate() with non-null,defined data", function() {
+
+       expect(3);
+
+       var handler = function( event, data ) {
+                               equal( data, 0, "non-null, defined data (zero) is correctly passed" );
+                       };
+
+       jQuery("#foo").bind("foo", handler);
+       jQuery("#foo").live("foo", handler);
+       jQuery("div").delegate("#foo", "foo", handler);
+
+       jQuery("#foo").trigger("foo", 0);
+
+       jQuery("#foo").unbind("foo", handler);
+       jQuery("#foo").die("foo", handler);
+       jQuery("div").undelegate("#foo", "foo");
+
+});
+
 test("bind(), with data", function() {
        expect(4);
        var handler = function(event) {