null reference when using effects

[0cis32f]

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

[ikarlik]

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 = {};
}

[maciejmrozinski]

​https://github.com/jquery/jquery-ui/pull/783

[Corey Frang]

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

Changeset: 8b76684a92d7340175284bb369426f0164673b0a

[Corey Frang]

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