Side navigation
Ticket #3096: event-extra.patch
File event-extra.patch, 1.9 KB (added by scottgonzalez, June 27, 2008 08:40PM UTC)
Index: test/unit/event.js
===================================================================
--- test/unit/event.js (revision 5753)
+++ test/unit/event.js (working copy)
@@ -182,7 +182,7 @@
});
test("trigger(event, [data], [fn])", function() {
- expect(67);
+ expect(72);
var handler = function(event, a, b, c) {
equals( event.type, "click", "check passed data" );
@@ -192,14 +192,16 @@
return "test";
};
- var handler2 = function(a, b, c) {
+ var handler2 = function(event, a, b, c) {
+ equals( event.type, "click", "check passed data" );
equals( a, 1, "check passed data" );
equals( b, "2", "check passed data" );
equals( c, "abc", "check passed data" );
return false;
};
- var handler3 = function(a, b, c, v) {
+ var handler3 = function(event, a, b, c, v) {
+ equals( event.type, "click", "check passed data" );
equals( a, 1, "check passed data" );
equals( b, "2", "check passed data" );
equals( c, "abc", "check passed data" );
@@ -207,7 +209,8 @@
return "newVal";
};
- var handler4 = function(a, b, c, v) {
+ var handler4 = function(event, a, b, c, v) {
+ equals( event.type, "click", "check passed data" );
equals( a, 1, "check passed data" );
equals( b, "2", "check passed data" );
equals( c, "abc", "check passed data" );
Index: src/event.js
===================================================================
--- src/event.js (revision 5753)
+++ src/event.js (working copy)
@@ -206,10 +206,6 @@
if ( (!fn || (jQuery.nodeName(elem, 'a') && type == "click")) && elem["on"+type] && elem["on"+type].apply( elem, data ) === false )
val = false;
- // Extra functions don't get the custom event object
- if ( event )
- data.shift();
-
// Handle triggering of extra function
if ( extra && jQuery.isFunction( extra ) ) {
// call the extra function and tack the current return value on the end for possible inspection
Download in other formats:
Original Format
File event-extra.patch, 1.9 KB (added by scottgonzalez, June 27, 2008 08:40PM UTC)
Index: test/unit/event.js
===================================================================
--- test/unit/event.js (revision 5753)
+++ test/unit/event.js (working copy)
@@ -182,7 +182,7 @@
});
test("trigger(event, [data], [fn])", function() {
- expect(67);
+ expect(72);
var handler = function(event, a, b, c) {
equals( event.type, "click", "check passed data" );
@@ -192,14 +192,16 @@
return "test";
};
- var handler2 = function(a, b, c) {
+ var handler2 = function(event, a, b, c) {
+ equals( event.type, "click", "check passed data" );
equals( a, 1, "check passed data" );
equals( b, "2", "check passed data" );
equals( c, "abc", "check passed data" );
return false;
};
- var handler3 = function(a, b, c, v) {
+ var handler3 = function(event, a, b, c, v) {
+ equals( event.type, "click", "check passed data" );
equals( a, 1, "check passed data" );
equals( b, "2", "check passed data" );
equals( c, "abc", "check passed data" );
@@ -207,7 +209,8 @@
return "newVal";
};
- var handler4 = function(a, b, c, v) {
+ var handler4 = function(event, a, b, c, v) {
+ equals( event.type, "click", "check passed data" );
equals( a, 1, "check passed data" );
equals( b, "2", "check passed data" );
equals( c, "abc", "check passed data" );
Index: src/event.js
===================================================================
--- src/event.js (revision 5753)
+++ src/event.js (working copy)
@@ -206,10 +206,6 @@
if ( (!fn || (jQuery.nodeName(elem, 'a') && type == "click")) && elem["on"+type] && elem["on"+type].apply( elem, data ) === false )
val = false;
- // Extra functions don't get the custom event object
- if ( event )
- data.shift();
-
// Handle triggering of extra function
if ( extra && jQuery.isFunction( extra ) ) {
// call the extra function and tack the current return value on the end for possible inspection