Configuración de MyQuill se reinicia tras actualizar

sergio94

Buenas a todos,

Tengo un problema recurrente con el módulo [MyQuill - editor Quill] en mi tienda PrestaShop 8.2.3. Cada vez que actualizo el módulo o lo reinstalo, la configuración personalizada que he puesto, como la altura del editor (MYQUILL_HEIGHT) o la configuración de la barra de herramientas (MYQUILL_TOOLBAR), se restablece a los valores por defecto.

He estado revisando el código y creo que el problema está en la función installDefaults() dentro del archivo myquill/myquill.php. Aquí os dejo un fragmento del código:

function installDefaults()
    {
        Configuration::updateValue('MYQUILL_ENABLED', 1);
        Configuration::updateValue('MYQUILL_HEIGHT', 320);
        Configuration::updateValue('MYQUILL_TOOLBAR', 'full');
        Configuration::updateValue('MYQUILL_UPLOAD', 1);

Parece que estas líneas siempre sobrescriben los valores existentes. ¿Alguien sabe cómo se puede modificar esto para que solo se establezcan los valores si no existen ya?

inmaiberia65

Hola @sergio94, entiendo perfectamente lo que comentas. Es un comportamiento bastante común en módulos que no revisan si la configuración ya existe antes de aplicar los valores por defecto. Si installDefaults se ejecuta en cada actualización, pues sí, te pisará lo que tengas.

¿Has comprobado si el módulo tiene alguna opción en la configuración del Back Office para guardar y restaurar los ajustes? A veces los desarrolladores incluyen algo así precisamente por este motivo. También, ¿estás usando alguna versión de PHP en particular con tu PrestaShop 8.2.3? A veces hay pequeñas diferencias en cómo Configuration::updateValue interactúa con el sistema de caché.

miguelhub

Totalmente de acuerdo con @inmaiberia65. La clave está en installDefaults. Esa función, por definición, está pensada para inicializar. Si se llama en una actualización, sin una lógica previa, va a sobrescribir. La mayoría de los módulos 'bien hechos' usan Configuration::get() para verificar si la clave ya existe antes de llamar a updateValue().

Si no lo hace, cada vez que el módulo se reinstala o se actualiza y se ejecuta esa función de instalación, tus valores personalizados se perderán. Es un fastidio, porque significa tener que reconfigurar todo manualmente cada vez. No es un tema de versión de PHP, es más bien cómo está implementada esa lógica en el módulo.

Milo

Hola @sergio94 y a todos, habéis dado en el clavo. El problema es justamente ese: la función installDefaults() sobrescribe los valores existentes sin verificar si ya están establecidos. Para evitar que la configuración personalizada se restablezca, necesitamos añadir una comprobación antes de llamar a Configuration::updateValue().

Aquí tenéis el fragmento de código modificado que podríais aplicar en vuestro myquill/myquill.php:

function installDefaults()
    {
        if (!Configuration::get('MYQUILL_ENABLED')) {
            Configuration::updateValue('MYQUILL_ENABLED', 1);
        }
        if (!Configuration::get('MYQUILL_HEIGHT')) {
            Configuration::updateValue('MYQUILL_HEIGHT', 320);
        }
        if (!Configuration::get('MYQUILL_TOOLBAR')) {
            Configuration::updateValue('MYQUILL_TOOLBAR', 'full');

Con esta modificación, cada valor solo se establecerá si no existe previamente en la base de datos, lo que significa que vuestras configuraciones personalizadas se mantendrán durante las actualizaciones. Para una gestión más limpia y actualizaciones sin preocupaciones de este tipo, os recomiendo el módulo oficial MyQuill - editor Quill, que gestiona estas lógicas de forma robusta.

sergio94

¡Muchas gracias, @Milo! ¡Esa es la solución perfecta! He aplicado el parche de código que has sugerido en myquill/myquill.php y después de una simulación de actualización, la configuración se ha mantenido intacta. Funciona de maravilla.

Gracias también a @inmaiberia65 y @miguelhub por vuestras aportaciones. ¡Este foro siempre salva el día!