Browse Source

Fixes extend settings in some modules using old format

pull/2158/merge
jlukic 9 years ago
parent
commit
9bfe781f50
3 changed files with 10 additions and 4 deletions
  1. 4
      src/definitions/behaviors/colorize.js
  2. 4
      src/definitions/behaviors/visit.js
  3. 6
      src/definitions/modules/shape.js

4
src/definitions/behaviors/colorize.js

@ -15,7 +15,9 @@
$.fn.colorize = function(parameters) { $.fn.colorize = function(parameters) {
var var
settings = $.extend(true, {}, $.fn.colorize.settings, parameters),
settings = ( $.isPlainObject(parameters) )
? $.extend(true, {}, $.fn.colorize.settings, parameters)
: $.extend({}, $.fn.colorize.settings),
// hoist arguments // hoist arguments
moduleArguments = arguments || false moduleArguments = arguments || false
; ;

4
src/definitions/behaviors/visit.js

@ -31,7 +31,9 @@ $.visit = $.fn.visit = function(parameters) {
$allModules $allModules
.each(function() { .each(function() {
var var
settings = $.extend(true, {}, $.fn.visit.settings, parameters),
settings = ( $.isPlainObject(parameters) )
? $.extend(true, {}, $.fn.visit.settings, parameters)
: $.extend({}, $.fn.visit.settings),
error = settings.error, error = settings.error,
namespace = settings.namespace, namespace = settings.namespace,

6
src/definitions/modules/shape.js

@ -37,8 +37,10 @@ $.fn.shape = function(parameters) {
$allModules $allModules
.each(function() { .each(function() {
var var
moduleSelector = $allModules.selector || '',
settings = $.extend(true, {}, $.fn.shape.settings, parameters),
moduleSelector = $allModules.selector || '',
settings = ( $.isPlainObject(parameters) )
? $.extend(true, {}, $.fn.shape.settings, parameters)
: $.extend({}, $.fn.shape.settings),
// internal aliases // internal aliases
namespace = settings.namespace, namespace = settings.namespace,

Loading…
Cancel
Save