Skip to content
Permalink
Browse files

Landing pull request 377. Check custom data != null(undefined), allow…

…s zero; Fixes #9285.

More Details:
 - #377
 - http://bugs.jquery.com/ticket/9285
  • Loading branch information...
rwaldron authored and timmywil committed May 16, 2011
1 parent a5cf257 commit c17f589ec99e8309e813a4081eed47f39a0c6120
Showing with 21 additions and 1 deletion.
  1. +1 −1 src/event.js
  2. +20 −0 test/unit/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,
@@ -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) {

0 comments on commit c17f589

Please sign in to comment.
You can’t perform that action at this time.