Hook ValidateOrderBefore no procesa carrito en módulo

samueldeco

Hola a todos,

Estoy teniendo un quebradero de cabeza con mi módulo de pago personalizado en PrestaShop 1.7.7.5. El problema es que los descuentos o recargos asociados a ciertos métodos de pago no se aplican correctamente, y al depurar, veo que mi hookActionValidateOrderBefore dentro del módulo paydisorfee se detiene antes de tiempo. Parece que el $params['cart'] no se está pasando como una instancia de Cart o directamente no está.

Aquí tenéis el fragmento de código donde se produce el corte:

function hookActionValidateOrderBefore($params)
    {
        if (!(int) Configuration::get('PAYDISORFEE_ENABLED')) {
            return;
        }
        if (!isset($params['cart']) || !($params['cart'] instanceof Cart)) {
            return;
        }

Está en paydisorfee/paydisorfee.php. Básicamente, si el carrito no está bien, el hook sale y no se aplica ninguna lógica posterior para los recargos. ¿Alguien ha experimentado algo similar en 1.7.7.5 con módulos de pago personalizados? ¿Hay alguna forma de asegurar que el carrito siempre llegue bien a este hook o de manejar esta situación de otra manera?

lorena24

Hola @samueldeco,

Entiendo tu frustración. Ese return prematuro puede ser un fastidio. La verdad es que ese if que comprueba $params['cart'] es bastante estándar y crítico. Si el carrito no es una instancia de Cart o no existe, es señal de que algo más arriba en el flujo de tu módulo de pago no está pasando el objeto Cart correctamente a la función validateOrder de PrestaShop, que es la que dispara el hook.

¿Podrías confirmar qué versión de PHP estás usando? A veces, con versiones más antiguas de 1.7 y PHP 7.1-7.3, algunos módulos personalizados tienen comportamientos raros si no siguen al pie de la letra cómo se deben pasar los parámetros. Asegúrate de que tu módulo de pago esté llamando a PaymentModule::validateOrder() de la forma esperada, con todos los parámetros correctos, especialmente el objeto Cart.

enriqueonline70

Buenas @samueldeco y @lorena24,

A mí me pasó algo parecido con un módulo de transferencia bancaria que habíamos personalizado un poco. En mi caso, el problema no era que faltara el carrito, sino que en ciertas circunstancias (creo que cuando el usuario volvía de una pasarela externa y se producía algún tipo de redirección), el $params['cart'] llegaba, pero no era un objeto Cart válido, era otra cosa o un array vacío por alguna razón.

Revisad cómo vuestro módulo de pago invoca PaymentModule::validateOrder(). A veces, un error en los parámetros que le pasáis ahí o en el flujo de la finalización del pedido puede hacer que el $params['cart'] no sea lo que espera el hook. No creo que el problema esté en el if del hook, sino en cómo se genera y se pasa el carrito antes de que ese hook se ejecute. Es como si el plato principal no llegara a la mesa del chef, el chef no puede cocinar, ¿no? 😃

Milo

Hola @samueldeco, @lorena24 y @enriqueonline70,

Gracias por la descripción detallada. El problema que reporta @samueldeco es un clásico con algunos módulos de pago en PrestaShop 1.7.7.5, especialmente aquellos que no manejan explícitamente el objeto Cart o que tienen flujos de pago complejos.

Efectivamente, la condición if (!isset($params['cart']) || !($params['cart'] instanceof Cart)) está ahí por una razón fundamental: el hook hookActionValidateOrderBefore necesita un objeto Cart válido para operar correctamente. Si no lo recibe, el módulo asume que no hay un contexto de pedido válido y se aborta.

La razón por la que esto ocurre es que tu módulo de pago personalizado, al llamar a PaymentModule::validateOrder(), no está pasando el objeto Cart correctamente en el array $params o este se está perdiendo/corrompiendo antes de que se ejecute el hook. Para solucionarlo dentro del contexto del hook de paydisorfee, podemos añadir una lógica de respaldo para asegurarnos de que el carrito siempre esté disponible, obteniéndolo del Context global si no se pasa correctamente.

Aquí tienes una modificación que puedes aplicar en tu archivo paydisorfee/paydisorfee.php:

function hookActionValidateOrderBefore($params)
    {
        if (!(int) Configuration::get('PAYDISORFEE_ENABLED')) {
            return;
        }
        // Aseguramos que $params['cart'] esté siempre disponible y sea una instancia de Cart.
        // Si no está o no es válido, intentamos obtenerlo del contexto global de PrestaShop.
        if (!isset($params['cart']) || !($params['cart'] instanceof Cart)) {
            if (isset(Context::getContext()->cart) && Context::getContext()->cart instanceof Cart) {
                $params['cart'] = Context::getContext()->cart;

Con este cambio, si por alguna razón el objeto Cart no llega en $params, el hook intentará recuperarlo del Context global, lo cual es una práctica segura en PrestaShop 1.7 para asegurar que la lógica del carrito pueda continuar. Pruébalo y cuéntanos si se soluciona.

samueldeco

¡Increíble @Milo!

Acabo de aplicar tu modificación en paydisorfee/paydisorfee.php y ¡funciona a la perfección! Los recargos se están aplicando ahora sin problemas y el hook ya no se detiene prematuramente. Tenía sentido que el problema no estuviera en el if en sí, sino en el flujo previo, y tu solución de respaldo desde el Context es justo lo que necesitaba para blindarlo.

Muchísimas gracias por la ayuda tan rápida y precisa. Me has ahorrado horas de depuración. ¡Un saludo!