Módulo MyQuill siempre 'inconsistente'

valeria9483

Tengo un problema extraño con mi módulo personalizado MyQuill en PrestaShop 1.7.5.0. Funciona perfectamente, todas sus funciones son correctas y hace lo que debe, pero en el Gestor de Módulos siempre aparece con el aviso de 'inconsistente'.

Revisando el código, he visto que en el archivo myquill/myquill.php hay un método llamado inconsistency() que, por alguna razón, siempre devuelve true. Supongo que esto es lo que está causando el falso positivo, pero no estoy seguro de cómo debería ser esta función o si debería eliminarla.

Aquí tenéis el fragmento de código:

function inconsistency($ret = null)
    {
        return true;
    }

¿Alguien sabe si esto es un error del módulo o si hay alguna forma correcta de manejar esta función para que no me dé ese aviso constante? Me gustaría que el módulo apareciera como 'OK' si realmente no tiene ningún problema.

beatrizseller

Hola @valeria9483, eso es un clásico de PrestaShop con módulos mal terminados o que se han portado de versiones muy antiguas. La función inconsistency() está ahí para que el propio PrestaShop pueda validar si al módulo le falta algo o no está bien configurado. Si devuelve true, está diciendo 'sí, soy inconsistente', por eso te sale el aviso.

¿El módulo lo has desarrollado tú o lo has comprado? A veces, los desarrolladores dejan esa función así durante el desarrollo y luego se olvidan de quitarla o implementarla correctamente. Si el módulo te funciona bien, es casi seguro que puedes modificar esa línea.

paula94

Totalmente de acuerdo con @beatrizseller. En PrestaShop 1.7.5.0, el gestor de módulos usa esa función para sus comprobaciones internas. Si el módulo está realmente bien y no quieres que salga el aviso, lo lógico es que esa función devuelva false.

He visto casos donde los desarrolladores la usan para comprobar, por ejemplo, si un archivo de configuración existe o si una tabla de base de datos está creada. Si no necesitas ninguna de esas comprobaciones y el módulo está completo, un return false; debería bastar. ¿Qué otras cosas hace tu módulo MyQuill? Es para ver si habría algo que se debería validar ahí.

Milo

Hola @valeria9483,

Los compañeros están en lo cierto. En PrestaShop 1.7.x, el método inconsistency() dentro de la clase principal del módulo (en este caso, myquill.php) es utilizado por el Gestor de Módulos para determinar su estado de 'salud'. Si la función devuelve true, PrestaShop interpreta que el módulo tiene alguna inconsistencia, incluso si funcionalmente todo parece estar bien para ti.

Dado que confirmas que tu módulo MyQuill funciona perfectamente y no presenta problemas, lo más probable es que esa función se haya quedado así por error o se usara para alguna comprobación temporal que ya no es necesaria.

La solución es simple: haz que la función devuelva false. Con esto, le estás indicando a PrestaShop que el módulo NO presenta inconsistencias.

Aquí tienes el fragmento de código modificado:

function inconsistency($ret = null)
    {
        // Si el módulo funciona correctamente y no hay inconsistencias reales,
        // devolvemos 'false' para que el Gestor de Módulos lo marque como 'OK'.
        return false;
    }

Después de guardar el cambio en myquill/myquill.php, actualiza la página del Gestor de Módulos. Deberías ver que el aviso de 'inconsistente' ha desaparecido y ahora tu módulo MyQuill se muestra como 'OK'.

valeria9483

¡Funciona perfectamente, @Milo! Muchas gracias por la aclaración y por el código. He hecho el cambio y, tal como dijiste, el aviso ha desaparecido y ahora el módulo aparece como 'OK'. Me quitas un dolor de cabeza. ¡Un saludo a todos!