Event: remove guard for falsy handler argument of jQuery#on method

(cherry-picked from fac67a984268ef8f7de952666fda6d8d32754f5f)

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

diff --git a/src/event.js b/src/event.js
index 55ee4ea1470c101554da9750e0706801262a0a8e..87ae3a65b17ee14cee072cd685cf284fef982863 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -1002,8 +1002,6 @@ jQuery.fn.extend({
                }
                if ( fn === false ) {
                        fn = returnFalse;
-               } else if ( !fn ) {
-                       return this;
                }
 
                if ( one === 1 ) {

diff --git a/test/unit/event.js b/test/unit/event.js
index 3854a2133ba9c5afbc2e309172cc390a0c8ff4f4..dbfb2cd533272ed674ce5a559c538e30c6cf6aef 100644 (file)
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -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);