2012-04-03 19 views
6

Siguiendo las buenas jQuery Plugins/Authoring instrucciones que tengo una pequeña preguntajQuery - Plugin de opciones por defecto se extiende()

(function($){ 

    // Default Settings 
    var settings = { 
    var1: 50 
    , var2: 100 
    }; 

    var methods = { 
    init : function (options) { 
     console.log(settings); 
     settings = $.extend(options, settings); // Overwrite settings 
     console.log(settings); 
     return this; 
    } 
    , other_func: function() { 
     return this; 
    } 
    }; 

    $.fn.my_plugin = function (method) { 
    if (methods[method]) { 
     return methods[method].apply(this, Array.prototype.slice.call(arguments, 1)); 
    } else if (typeof method === 'object' || ! method) { 
     return methods.init.apply(this, arguments); 
    } else { 
     $.error('Method ' + method + ' does not exist on jQuery.my_plugin'); 
    }  
    }; 

})(jQuery); 

Si hago

>>> $('my_element').my_plugin({var3: 60}) 
Before Object { var2=100, var1=50} 
After Object { var3=60, var2=100, var1=50} 
[ my_element ] 

>>> $('my_element').my_plugin({var1: 60}) 
Before Object { var1=50, var2=100} 
After Object { var1=50, var2=100} 
[ my_element ] 

¿Por qué mi var1 no anulados?

Respuesta

19

Se mezclan el orden de los argumentos en su $.extend (objetivo debe ser en primer lugar), que debe ser:

settings = $.extend(settings, options); 

Ver this fiddle y la docs for $.extend()

Para para evitar confusiones, también puede ampliar su configuración con sus valores predeterminados, como este:

methods.init = function(options){ 

    var settings = $.extend({ 
    key1: 'default value for key 1', 
    key2: 'default value for key 2' 
    }, options); // <- if no/undefined options are passed extend will simply return the defaults 

    //here goes the rest 

}; 
+0

De hecho, este fue el orden, muchas gracias –

+1

Esto me ayudó a hacer mi [primer] (https://github.com/Glideh/jquery.particles.burst) plugin githubbed jquery :) –

4

Está sobrescribiendo sus valores predeterminados. Intenta crear una nueva variable para almacenar la configuración dentro del método init.

var defaults = { 
    var1: 50 
    , var2: 100 
    }; 

    var methods = { 
    init : function (options) { 
     console.log(defaults); 
     var settings = $.extend({},defaults,options || {}); 
     console.log(settings); 
     $(this).data("myPluginSettings",settings); 
     return this; 
    } 
    , other_func: function() { 
     console.log(this.data("myPluginSettings")); 
     return this; 
    } 
    }; 
+0

He invertido los parámetros. Gracias –

Cuestiones relacionadas