0

Tengo la siguiente función:

var config = service.getConfiguration();
switch (config.type) {
  case 'pattern':
    $wrapper.find('.pattern-library li').removeClass('selected');
    $wrapper.find('.' + config.extra.replace(' ', '.')).addClass('selected');
    $wrapper.find('.btn-picker-pattern').css({'background-color': config.color});
  break;
  case 'color':
    $wrapper.find('.btn-picker-color').css({'background-color': config.color});
  break;
  case 'image':
    $wrapper.find('#image_position_chzn').remove();
    $imagePositionSelector.val(config.extra).removeClass('chzn-done').chosen();
  break;
}
$('body').trigger('menu-configuration-update');
};

y para arreglar un error que contiene, necesito agregar esta condición:

if ( !$.isEmptyObject(config.extra) ) { 
  $wrapper.find( '.' + config.extra.replace(' ', '.') ).addClass('selected');

Entonces pienso que seria mejor trasladar todo a if o sino una manera de poder insertar la condición en mi función actual. Como podría hacerlo?

8
  • Donde te da el error? Y que error te da? Commented el 5 abr. 2019 a las 12:42
  • entiendo que ese if lo tenes que agregar en, case 'pattern' y case 'image', correcto? Commented el 5 abr. 2019 a las 13:11
  • Por lo que entiendo de tu problema, la condición del if es independiente de las del switch, por lo que podría ir antes o después de este Commented el 5 abr. 2019 a las 13:22
  • @SaúlHormazábal gracias por tu respuesta, debajo deje el comentario de lo que me sucede actualmente. Commented el 5 abr. 2019 a las 13:27
  • @FabrizioMigotto gracias por tu respuesta, debajo deje el comentario de lo que me sucede actualmente Commented el 5 abr. 2019 a las 13:27

1 respuesta 1

1

Al utilizar ese if resolví mi error de "Cannot read replace property", pero Ahora hace que no quede seleccionada la clase que es elegida como fondo en mi menu y ese seria mi problema ahora, entonces como podria hacerlo correctamente?

var config = service.getConfiguration();

            switch (config.type) {
                case 'pattern':
                    $wrapper.find('.pattern-library li').removeClass('selected');

                    if (!$.isEmptyObject(config.extra)) { 
                        $wrapper.find('.' + config.extra.replace(' ', '.')).addClass('selected');
                    }

                    $wrapper.find('.btn-picker-pattern').css({'background-color': config.color});
                    break;
                case 'color':
                    $wrapper.find('.btn-picker-color').css({'background-color': config.color});
                    break;
                case 'image':
                    $wrapper.find('#image_position_chzn').remove();
                    $imagePositionSelector.val(config.extra).removeClass('chzn-done').chosen();
                    break;
            }
            $('body').trigger('menu-configuration-update');
        };
2

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.