Skip to main content

Bug Tracker

Side navigation

#3601 closed bug (duplicate)

Opened November 13, 2008 04:05PM UTC

Closed November 17, 2008 03:05PM UTC

Last modified March 15, 2012 12:02AM UTC

Mismatch parameters count passed to jQuery.easing.linear() or jQuery.easing.swing()

Reported by: DanielKatz Owned by:
Priority: minor Milestone: 1.3
Component: effects Version: 1.2.6
Keywords: easing, parameters Cc: DanielKatz, flesler
Blocked by: Blocking:
Description

In some cases mismatch parameters count passed to jQuery.easing.linear() or jQuery.easing.swing() and causing an exception.

Test case is provided.

(at the line: this.pos = jQuery.easing[this.options.easing || (jQuery.easing.swing ? "swing" : "linear")](this.state, n, 0, 1, this.options.duration);)

Attachments (3)
  • Ticket #3601.patch (0.5 KB) - added by DanielKatz November 17, 2008 03:28AM UTC.

    this fixes the case where the callback function is defined in another iframe

  • jquery_bug_test_case.htm (1.5 KB) - added by DanielKatz November 13, 2008 04:06PM UTC.

    bug TestCase

  • jquerybug.rar (29.4 KB) - added by DanielKatz November 13, 2008 05:30PM UTC.

    Simpler test case

Change History (7)

Changed November 13, 2008 04:24PM UTC by flesler comment:1

cc: → DanielKatz, flesler
component: unfilledfx
need: ReviewTest Case
owner: flesler
priority: blockerminor

The test case is really not a good one. The whole iframe+string manipulation can lead to unreal situations.

can you make one that reproduces the error, with a simple html file with no iframes and stuff like that ?

Changed November 13, 2008 04:38PM UTC by DanielKatz comment:2

Replying to [comment:1 flesler]:

The test case is really not a good one. The whole iframe+string manipulation can lead to unreal situations. can you make one that reproduces the error, with a simple html file with no iframes and stuff like that ?

I'l try. but i think this the only case. and it's obviously bug in jQuery because this exception occures on every browser...

And logicaly the iframe doesn't make any difference.

Changed November 13, 2008 05:30PM UTC by DanielKatz comment:3

Replying to [comment:1 flesler]:

The test case is really not a good one. The whole iframe+string manipulation can lead to unreal situations. can you make one that reproduces the error, with a simple html file with no iframes and stuff like that ?

I've wrote slightly simpler test case it's attached to the ticket.

Changed November 13, 2008 05:52PM UTC by flesler comment:4

need: Test CaseReview

Ok then, so the problem is related to iframes.

That's something you don't mention anywhere in the whole ticket.

Changed November 13, 2008 05:57PM UTC by DanielKatz comment:5

Replying to [comment:4 flesler]:

Ok then, so the problem is related to iframes. That's something you don't mention anywhere in the whole ticket.

Yes it is. And if I'd know how to update the description I would. sorry.

Changed November 13, 2008 06:32PM UTC by DanielKatz comment:6

Replying to [ticket:3601 DanielKatz]:

In some cases mismatch parameters count passed to jQuery.easing.linear() or jQuery.easing.swing() and causing an exception. Test case is provided. (at the line: this.pos = jQuery.easing[this.options.easing || (jQuery.easing.swing ? "swing" : "linear")](this.state, n, 0, 1, this.options.duration);)

Allegedly the bug appears only the code executes in context of onload event of IFrame.

Changed November 17, 2008 03:05PM UTC by john comment:7

resolution: → duplicate
status: newclosed

I'm going to close this as a duplicate of #3618 - because we really should fix .constructor use everywhere in jQuery.