From 3ae5fbc16a60cf3f41a1c6166a65dd473e904eb1 Mon Sep 17 00:00:00 2001 From: David Serduke Date: Wed, 28 Nov 2007 22:23:40 +0000 Subject: [PATCH] Fixed #1701 by passing through the arguments as suggested. --- src/event.js | 2 +- test/unit/event.js | 8 ++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/event.js b/src/event.js index 6475cd9..d4a899b 100644 --- a/src/event.js +++ b/src/event.js @@ -333,7 +333,7 @@ jQuery.fn.extend({ event.preventDefault(); // and execute the function - return args[this.lastToggle].apply( this, [event] ) || false; + return args[this.lastToggle].apply( this, arguments ) || false; }); }, diff --git a/test/unit/event.js b/test/unit/event.js index 593c6da..47da9b6 100644 --- a/test/unit/event.js +++ b/test/unit/event.js @@ -172,7 +172,7 @@ test("trigger(event, [data], [fn])", function() { }); test("toggle(Function, Function)", function() { - expect(4); + expect(5); var count = 0, fn1 = function(e) { count++; }, fn2 = function(e) { count--; }, @@ -180,7 +180,11 @@ test("toggle(Function, Function)", function() { link = $('#mark'); link.click(preventDefault).click().toggle(fn1, fn2).click().click().click().click().click(); ok( count == 1, "Check for toggle(fn, fn)" ); - + + $("#firstp").toggle(function () { + equals(arguments.length, 4, "toggle correctly passes through additional triggered arguments, see #1701" ) + }, function() {}).trigger("click", [ 1, 2, 3 ]); + var first = 0; $("#simon1").one("click", function() { ok( true, "Execute event only once" ); -- 1.7.10.4