Hook displayProductFooter no se ejecuta

hugoonline

¡Hola a todos!

Estoy teniendo un problema con un módulo que uso para mostrar una tabla de combinaciones en la ficha de producto. En PrestaShop 8.2.3, la función hookdisplayProductFooter simplemente no se ejecuta, y he estado depurando el código y parece que hay una comprobación de versión que lo está bloqueando.

El problema es que la función self::isPs17() devuelve true en mi PrestaShop 8.2.3 (lo cual es correcto, porque PS8 se considera 'PS17+' en esa lógica), pero luego hay un return anticipado que impide que renderCombinationsList se ejecute, incluso cuando la configuración CFG_WHERE está en WHERE_FOOTER.

Aquí tenéis el fragmento de código del archivo mycombtabfree/mycombtabfree.php que creo que es el culpable:

function hookdisplayProductFooter($params)
    {
        if (self::isPs17()) {
            return;
        }
        if ((int) Configuration::get(self::CFG_WHERE) !== self::WHERE_FOOTER) {
            return;
        }

        return $this->renderCombinationsList($params);
    }

¿Alguna idea de cómo puedo modificar esto para que funcione correctamente en PrestaShop 8.2.3? Me trae de cabeza.

hugodev

Hola @hugoonline,

Entiendo perfectamente tu frustración. Ese tipo de lógica para la compatibilidad de versiones es un clásico problema en módulos antiguos. La función isPs17() fue introducida para detectar versiones de PrestaShop 1.7.0.0 y superiores. Si el desarrollador original la usó con un return; justo después, es porque probablemente quería que ese hook no se ejecutara en PS 1.7+ o que usara una lógica diferente.

Dado que PrestaShop 8.2.3 se considera 'PS17' a efectos de esa comprobación, el if (self::isPs17()) { return; } efectivamente está matando la ejecución del hook antes de que llegue a renderCombinationsList.

¿Estás usando el tema Classic o alguno derivado? A veces, la ubicación de los hooks también puede influir si la plantilla no los llama correctamente, pero por lo que describes, parece un problema directo del código del módulo.

Milo

Saludos @hugoonline,

@hugodev ha dado en el clavo. La lógica de self::isPs17() en ese módulo está diseñada para hacer que el hook no haga nada en PrestaShop 1.7 y versiones posteriores (incluido PS 8.x). Esto probablemente se hizo para evitar conflictos o porque el módulo tenía una implementación diferente para versiones más antiguas (PS 1.6 o anteriores).

Para que renderCombinationsList se ejecute en tu PrestaShop 8.2.3, necesitas eliminar o invertir esa comprobación. Si el objetivo es que el hook funcione en PS 1.7+ y no solo en las versiones anteriores a PS 1.7, la solución más sencilla es eliminar esa línea de control de versión. Te recomiendo hacer esto en una anulación del módulo si es posible, o directamente en el archivo si no hay otra opción y tienes una copia de seguridad.

Aquí tienes cómo deberías modificar el código en mycombtabfree/mycombtabfree.php:

function hookdisplayProductFooter($params)
    {
        // Eliminada la comprobación isPs17() para permitir la ejecución en PS 1.7+
        // if (self::isPs17()) { return; }

        if ((int) Configuration::get(self::CFG_WHERE) !== self::WHERE_FOOTER) {
            return;
        }

        return $this->renderCombinationsList($params);
    }

Con esta modificación, el hook ya no saldrá prematuramente en PrestaShop 8.2.3 y debería llamar a renderCombinationsList como esperas, siempre que CFG_WHERE esté configurado correctamente.

hugoonline

¡Increíble, @Milo! ¡Muchísimas gracias!

He aplicado el cambio que sugeriste en mycombtabfree/mycombtabfree.php, comentando la línea if (self::isPs17()) { return; } y ahora el hook displayProductFooter se ejecuta perfectamente en mi PrestaShop 8.2.3. La tabla de combinaciones se renderiza donde debería.

No sé cómo no se me ocurrió que esa comprobación estaba diseñada para desactivar la funcionalidad en versiones más nuevas, y no para activarla. ¡Error de lógica por mi parte!

¡Gracias de nuevo por la ayuda tan rápida y precisa!