]> source.dussan.org Git - jquery.git/commitdiff
Event: remove guard for falsy handler argument of jQuery#on method
authorOleg Gaidarenko <markelog@gmail.com>
Thu, 30 Apr 2015 17:05:18 +0000 (20:05 +0300)
committerOleg Gaidarenko <markelog@gmail.com>
Sun, 3 May 2015 10:28:27 +0000 (13:28 +0300)
Since we don't have this in off method and its a common perception
that this is a rudiment code

Ref gh-2248
Closes gh-2249

src/event.js
test/unit/event.js

index 42bace43d23cb479204bd4abd424dddb4e45959e..ee0af6179076abe3efb8acbd8746997e77186136 100644 (file)
@@ -819,8 +819,6 @@ jQuery.fn.extend({
                }
                if ( fn === false ) {
                        fn = returnFalse;
-               } else if ( !fn ) {
-                       return this;
                }
 
                if ( one === 1 ) {
index fef9e05fb9657b9f733bbeb2fb4c2133db3dab13..95f6f4b33aa0edf29329935f7035cf3a4e89c9c8 100644 (file)
@@ -5,20 +5,6 @@ module( "event", {
        teardown: moduleTeardown
 });
 
-test("null or undefined handler", function() {
-       expect(2);
-       // Supports Fixes bug #7229
-       try {
-               jQuery("#firstp").on( "click", null );
-               ok(true, "Passing a null handler will not throw an exception");
-       } catch ( e ) {}
-
-       try {
-               jQuery("#firstp").on( "click", undefined );
-               ok(true, "Passing an undefined handler will not throw an exception");
-       } catch ( e ) {}
-});
-
 test("on() with non-null,defined data", function() {
 
        expect(2);