X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fevent.js;h=5b77e8c51e262b5170e3bd036217cb258f81c9f3;hb=25f9974cee5874d10df32f2d16fb985181e0c077;hp=d0cacf6b582b174cab4540573e5566b14232aade;hpb=5d033dba02fc1ffe1398cdaf0e9d402fc75e13f0;p=jquery.git diff --git a/src/event.js b/src/event.js index d0cacf6..5b77e8c 100644 --- a/src/event.js +++ b/src/event.js @@ -41,17 +41,10 @@ jQuery.event = { // Init the element's event structure var events = jQuery.data(elem, "events") || jQuery.data(elem, "events", {}), handle = jQuery.data(elem, "handle") || jQuery.data(elem, "handle", function(){ - // returned undefined or false - var val; - // Handle the second event of a trigger and when // an event is called after a page has unloaded - if ( typeof jQuery == "undefined" || jQuery.event.triggered ) - return val; - - val = jQuery.event.handle.apply(arguments.callee.elem, arguments); - - return val; + if ( typeof jQuery != "undefined" && !jQuery.event.triggered ) + return jQuery.event.handle.apply(arguments.callee.elem, arguments); }); // Add elem as a property of the handle function // This is to prevent a memory leak with non-native @@ -167,7 +160,7 @@ jQuery.event = { trigger: function(type, data, elem, donative, extra) { // Clone the incoming data, if any - data = jQuery.makeArray(data || []); + data = jQuery.makeArray(data); if ( type.indexOf("!") >= 0 ) { type = type.slice(0, -1); @@ -199,9 +192,10 @@ jQuery.event = { if ( exclusive ) data[0].exclusive = true; - // Trigger the event - if ( jQuery.isFunction( jQuery.data(elem, "handle") ) ) - val = jQuery.data(elem, "handle").apply( elem, data ); + // Trigger the event, it is assumed that "handle" is a function + var handle = jQuery.data(elem, "handle"); + if ( handle ) + val = handle.apply( elem, data ); // Handle triggering native .onfoo handlers (and on links since we don't call .click() for links) if ( (!fn || (jQuery.nodeName(elem, 'a') && type == "click")) && elem["on"+type] && elem["on"+type].apply( elem, data ) === false ) @@ -237,28 +231,29 @@ jQuery.event = { handle: function(event) { // returned undefined or false - var val; + var val, ret, namespace, all, handlers; - // 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]; + namespace = event.type.split("."); + event.type = namespace[0]; + namespace = namespace[1]; + all = !namespace && !event.exclusive; //cache this now, all = true means, any handler - var handlers = jQuery.data(this, "events") && jQuery.data(this, "events")[event.type], args = Array.prototype.slice.call( arguments, 1 ); - args.unshift( event ); + handlers = ( 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; // Filter the functions by class - if ( !parts[1] && !event.exclusive || handler.type == parts[1] ) { - var ret = handler.apply( this, args ); + if ( all || handler.type == namespace ) { + // Pass in a reference to the handler function itself + // So that we can later remove it + event.handler = handler; + event.data = handler.data; + + ret = handler.apply( this, arguments ); if ( val !== false ) val = ret; @@ -301,6 +296,9 @@ jQuery.event = { originalEvent.cancelBubble = true; }; + // Fix timeStamp + event.timeStamp = event.timeStamp || +new Date; + // Fix target property, if necessary if ( !event.target ) event.target = event.srcElement || document; // Fixes #1925 where srcElement might not be defined either