Ticket #2655: handle-fix.diff

src/event.js

@@ -240 +240 @@
                 var val;
 
                 // Empty object is for triggered events with no data
-                event = jQuery.event.fix( event || window.event || {} ); 
+                event = arguments[0] = jQuery.event.fix( event || window.event ); 
 
                 // Namespaced event handlers
-                var parts = event.type.split(".");
-                event.type = parts[0];
+                var ns = event.type.split(".");
+                event.type = ns[0];
+                ns = ns[1];//namespace
 
-                var handlers = jQuery.data(this, "events") && jQuery.data(this, "events")[event.type], args = Array.prototype.slice.call( arguments, 1 );
-                args.unshift( event );
+                var handlers = jQuery.data(this, "events") && jQuery.data(this, "events")[event.type];
 
                 for ( var j in handlers ) {
                         var handler = handlers[j];
                         // Pass in a reference to the handler function itself
                         // So that we can later remove it
-                        args[0].handler = handler;
-                        args[0].data = handler.data;
+                        event.handler = handler;
+                        event.data = handler.data;
 
                         // Filter the functions by class
-                        if ( !parts[1] && !event.exclusive || handler.type == parts[1] ) {
-                                var ret = handler.apply( this, args );
+                        if ( !ns && !event.exclusive || handler.type == ns ) 
+                                val = handler.apply( this, arguments ) !== false && val;
+                }
 
-                                if ( val !== false )
-                                        val = ret;
-
-                                if ( ret === false ) {
-                                        event.preventDefault();
-                                        event.stopPropagation();
-                                }
-                        }
+                if ( val === false ) {//check after the loop
+                        event.preventDefault();
+                        event.stopPropagation();
                 }
 
                 // Clean up added properties in IE to prevent memory leak
@@ -279 +275 @@
         },
 
         fix: function(event) {
+                if( event._fixed_ )//already fixed
+                        return event;
+                
                 // store a copy of the original event object 
                 // and clone to set read-only properties
                 var originalEvent = event;
-                event = jQuery.extend({}, originalEvent);
+                event = jQuery.extend({ _fixed_:true }, originalEvent);
                 
                 // add preventDefault and stopPropagation since 
                 // they will not work on the clone