Skip to main content

Bug Tracker

Side navigation

#2140 closed bug (fixed)

Opened January 09, 2008 08:05PM UTC

Closed January 14, 2008 06:46PM UTC

jQuery.event.trigger bug in Opera when using extra parameter

Reported by: scottgonzalez Owned by:
Priority: major Milestone: 1.2.2
Component: event Version: 1.2.1
Keywords: event trigger opera Cc:
Blocked by: Blocking:

When the extra parameter is used in jQuery.event.trigger and there is no default event for the type and the element does not have "handle" data, the event fails in Opera.

The problem occurs because val is undefined when this line is executed:

var ret = extra.apply( elem, data.concat( val ) );

This change fixes the problem:

var ret = extra.apply( elem, data.concat( val || false ) );

I'm not too familiar with the internal event code, so I'm not sure if that is an appropriate default value. I tried using null, but that causes the error as well.

I came across this problem using the Droppable plugin with a callback (drop, over and out all cause the problem).

Attachments (0)
Change History (2)

Changed January 11, 2008 06:55PM UTC by scott.gonzal comment:1

Oops, the WikiFormatting mangled the code, that should have been:

var ret = extra.apply( elem, data.concat( val || false ) );

Perhaps a cleaner solution is:

var ret = extra.apply( elem, val !== undefined ? data.concat( val ) : data );

Changed January 14, 2008 06:46PM UTC by john comment:2

resolution: → fixed
status: newclosed

Fixed in SVN rev [4440].