Adding in the change recommended by Alexander Farkas to fix some bugs in the change...

[jquery.git] / src / event.js

diff --git a/src/event.js b/src/event.js
index 9abf838..820b2b0 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -57,6 +57,12 @@ jQuery.event = {
                        handle = jQuery.data( elem, "handle", eventHandle );
                }
 
+               // If no handle is found then we must be trying to bind to one of the
+               // banned noData elements
+               if ( !handle ) {
+                       return;
+               }
+
                // Add elem as a property of the handle function
                // This is to prevent a memory leak with non-native
                // event in IE.
@@ -422,6 +428,8 @@ jQuery.event = {
                                jQuery.extend( proxy, data || {} );
 
                                proxy.guid += data.selector + data.live; 
+                               data.liveProxy = proxy;
+
                                jQuery.event.add( this, data.live, liveHandler, data ); 
                                
                        },
@@ -604,6 +612,9 @@ jQuery.event.special.submit = {
                                        return trigger( "submit", this, arguments );
                                }
                        });
+
+               } else {
+                       return false;
                }
        },
 
@@ -659,9 +670,9 @@ function testChange( e ) {
                        jQuery.data( elem, "_change_data", val );
                }
 
-               if ( elem.type !== "select" && (data != null || val) ) {
+               if ( data != null || val ) {
                        e.type = "change";
-                       return jQuery.event.trigger( e, arguments[1], this );
+                       return jQuery.event.trigger( e, arguments[1], elem );
                }
 }
 
@@ -752,12 +763,10 @@ jQuery.each(["bind", "one"], function( i, name ) {
                        for ( var key in type ) {
                                this[ name ](key, data, type[key], fn);
                        }
-
                        return this;
                }
                
                if ( jQuery.isFunction( data ) ) {
-                       thisObject = fn;
                        fn = data;
                        data = undefined;
                }
@@ -767,17 +776,11 @@ jQuery.each(["bind", "one"], function( i, name ) {
                        return fn.apply( this, arguments );
                }) : fn;
 
-               if ( type === "unload" && name !== "one" ) {
-                       this.one( type, data, fn, thisObject );
-
-               } else {
-                       // Deprecated: Please don't expect an empty jQuery set to bind to document
-                       (!this.selector && !this.context ? jQuery(document) : this).each(function() {
+               return type === "unload" && name !== "one" ?
+                       this.one( type, data, fn ) :
+                       this.each(function() {
                                jQuery.event.add( this, type, handler, data );
                        });
-               }
-
-               return this;
        };
 });
 
@@ -791,20 +794,14 @@ jQuery.fn.extend({
                        return this;
                }
 
-               // Deprecated: Please don't expect an empty jQuery set to bind to document
-               (!this.selector && !this.context ? jQuery(document) : this).each(function() {
+               return this.each(function() {
                        jQuery.event.remove( this, type, fn );
                });
-
-               return this;
        },
        trigger: function( type, data ) {
-               // Deprecated: Please don't expect an empty jQuery set to bind to document
-               (!this.selector && !this.context ? jQuery(document) : this).each(function() {
+               return this.each(function() {
                        jQuery.event.trigger( type, data, this );
                });
-
-               return this;
        },
 
        triggerHandler: function( type, data ) {
@@ -864,7 +861,7 @@ jQuery.fn.extend({
 
 function liveHandler( event ) {
        var stop = true, elems = [], selectors = [], args = arguments,
-               related, match, fn, elem, j, i, data,
+               related, match, fn, elem, j, i, l, data,
                live = jQuery.extend({}, jQuery.data( this, "events" ).live);
 
        for ( j in live ) {
@@ -920,7 +917,7 @@ function liveConvert( type, selector ) {
        return ["live", type, selector.replace(/\./g, "`").replace(/ /g, "&")].join(".");
 }
 
-jQuery.each( ("blur focus load resize scroll unload click dblclick " +
+jQuery.each( ("blur focus focusin focusout load resize scroll unload click dblclick " +
        "mousedown mouseup mousemove mouseover mouseout mouseenter mouseleave " +
        "change select submit keydown keypress keyup error").split(" "), function( i, name ) {
 
@@ -929,8 +926,8 @@ jQuery.each( ("blur focus load resize scroll unload click dblclick " +
                return fn ? this.bind( name, fn ) : this.trigger( name );
        };
 
-       if ( jQuery.fnAttr ) {
-               jQuery.fnAttr[ name ] = true;
+       if ( jQuery.attrFn ) {
+               jQuery.attrFn[ name ] = true;
        }
 });