]> source.dussan.org Git - jquery.git/commitdiff
Pass correct arg list to special._default. Thanks @mikaelkaron!
authorDave Methvin <dave.methvin@gmail.com>
Thu, 27 Oct 2011 20:02:54 +0000 (16:02 -0400)
committerDave Methvin <dave.methvin@gmail.com>
Thu, 27 Oct 2011 20:02:54 +0000 (16:02 -0400)
src/event.js
test/unit/event.js

index 3576bffcada92c4ba5f8fc20b698748455a7e861..dcbeb4b7f8005b95f712bb8bfe9a4d2d66276d99 100644 (file)
@@ -365,7 +365,7 @@ jQuery.event = {
                // If nobody prevented the default action, do it now
                if ( !event.isDefaultPrevented() ) {
 
-                       if ( (!special._default || special._default.call( elem.ownerDocument, event, data ) === false) &&
+                       if ( (!special._default || special._default.apply( elem.ownerDocument, data ) === false) &&
                                !(type === "click" && jQuery.nodeName( elem, "a" )) && jQuery.acceptData( elem ) ) {
 
                                // Call a native DOM method on the target with the same name name as the event.
index 35664b02246c0819f64a74332a983012902211ba..246bde32204ee9bc3c310aa8b06017ae1f3cddfc 100644 (file)
@@ -1753,8 +1753,8 @@ test("live with special events", function() {
                remove: function( handleObj ) {
                        ok( true, "Remove run." );
                },
-               _default: function( event ) {
-                       ok( true, "Default run." );
+               _default: function( event, arg ) {
+                       ok( event.type === "foo" && arg == 42, "Default run with correct args." );
                }
        };
 
@@ -1769,16 +1769,16 @@ test("live with special events", function() {
        });
 
        // Run: Handler 1, Handler 2, Default
-       jQuery("#liveSpan1").trigger("foo");
+       jQuery("#liveSpan1").trigger("foo", 42);
 
        // Run: Handler 1, Default
-       jQuery("#liveSpan1").trigger("foo.a");
+       jQuery("#liveSpan1").trigger("foo.a", 42);
 
        // Run: remove
        jQuery("#liveSpan1").die("foo.a");
 
        // Run: Handler 2, Default
-       jQuery("#liveSpan1").trigger("foo");
+       jQuery("#liveSpan1").trigger("foo", 42);
 
        // Run: remove, teardown
        jQuery("#liveSpan1").die("foo");