#8670 closed bug (fixed)

Opened October 14, 2012 02:41PM UTC

Closed October 21, 2012 09:47PM UTC

Last modified October 21, 2012 09:50PM UTC

null reference when using effects

In the 1.8 version i used to pass null for the options parameter in show method e.g.

$(el).show("blind", null, 600, null);

now it says options is null here :

effect.complete = callback || options.complete;

line of code 15606

probably missing an if not null check

Attachments (0)

Change History (6)

Please address this as soon as possible. We have a lot of instances where we pass

null

for options into show() method.

$(el).show('drop', null, 500);

Changing them all to pass

undefined

or using

show(options)

overload shouldn't be a requirement.

$(el).show('drop', undefined, 500);

$(el).show({effect: 'drop', delay: 500});

It could be fixed with a change on one line from:

// catch (effect)
if ( options === undefined ) {
  options = {};
}

to this or equivalent:

// catch (effect)
if (options == null) {
  options = {};
}

owner:	→ gnarf
status:	open → assigned

resolution:	→ fixed
status:	assigned → closed

Effects: Allow 'null' for options - Fixes #8670 - null reference when using effects - Closes gh-783

Changeset: 8b76684a92d7340175284bb369426f0164673b0a

Sorry, this one was my fault - Fix should land in 1.9.1