Effects: Fix show/hide/toggle test for using core functions. Fixed: #5456 - show...

diff --git a/ui/jquery.effects.core.js b/ui/jquery.effects.core.js
index 801b57e4c7c0da6bee96cc147350c7d46a37b34c..632e0e9325f17fd6a39191d86f85aef6268786f3 100644 (file)
--- a/ui/jquery.effects.core.js
+++ b/ui/jquery.effects.core.js
@@ -455,7 +455,7 @@ $.fn.extend({
 
        _show: $.fn.show,
        show: function(speed) {
-               if (!speed || typeof speed == 'number' || $.fx.speeds[speed]) {
+               if (!speed || typeof speed == 'number' || $.fx.speeds[speed] || !$.effects[speed] ) {
                        return this._show.apply(this, arguments);
                } else {
                        var args = _normalizeArguments.apply(this, arguments);
@@ -466,7 +466,7 @@ $.fn.extend({
 
        _hide: $.fn.hide,
        hide: function(speed) {
-               if (!speed || typeof speed == 'number' || $.fx.speeds[speed]) {
+               if (!speed || typeof speed == 'number' || $.fx.speeds[speed] || !$.effects[speed] ) {
                        return this._hide.apply(this, arguments);
                } else {
                        var args = _normalizeArguments.apply(this, arguments);
@@ -475,10 +475,10 @@ $.fn.extend({
                }
        },
 
-       // jQuery core overloads toggle and create _toggle
+       // jQuery core overloads toggle and creates _toggle
        __toggle: $.fn.toggle,
        toggle: function(speed) {
-               if (!speed || typeof speed == 'number' || $.fx.speeds[speed] ||
+               if (!speed || typeof speed == 'number' || $.fx.speeds[speed] || !$.effects[speed]  ||
                        typeof speed == 'boolean' || $.isFunction(speed)) {
                        return this.__toggle.apply(this, arguments);
                } else {