Hook ActionValidateOrderBefore y objeto Cart nulo

valeria9483

¡Hola a todos!

Estoy teniendo un problema bastante molesto con un módulo de pago en PrestaShop 9.0.3 (con PHP 8.2). El módulo debería aplicar un recargo o un descuento según la forma de pago, pero no lo hace de manera consistente.

He estado depurando y parece que el hook ActionValidateOrderBefore está retornando prematuramente en ciertos escenarios, especialmente cuando $params['cart'] no es una instancia válida de Cart. Esto provoca que la lógica para resolveOrderPaymentModule nunca se ejecute, y adiós descuentos/recargos.

Este es el fragmento de código problemático en paydisorfee/paydisorfee.php:

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

Mi pregunta es, ¿cómo se supone que debemos manejar esto en PS 9? ¿Hay alguna forma más robusta de asegurar que siempre tengamos un objeto Cart válido en este hook, o debo modificar esta comprobación? Me está volviendo loco que a veces funciona y a veces no.

manueldeco

Hola @valeria9483,

Entiendo tu frustración. En PrestaShop 9, con las últimas versiones de PHP, la tipificación y la consistencia de los objetos se ha vuelto más estricta. Ese return; es justo el culpable si $params['cart'] no es lo que esperas.

¿Te pasa con todos los métodos de pago o solo con alguno en particular? A veces, en flujos específicos de ciertos módulos de pago o en la revalidación de pedidos, el objeto cart no se pasa de la misma manera o directamente no se pasa como una instancia de Cart, sino como un array o incluso null. Es un clásico. La lógica de tu módulo necesita un Cart para aplicar los cálculos, así que si no lo tiene, se salta.

Habría que asegurar que siempre haya un objeto Cart para la lógica del módulo.

Milo

Hola @valeria9483 y @manueldeco,

Manuel tiene razón, este es un escenario común con la evolución de los hooks en PrestaShop y la estricta tipificación de PHP 8. El problema es que el hook ActionValidateOrderBefore no siempre garantiza que $params['cart'] sea una instancia de Cart en todas las circunstancias posibles (por ejemplo, en ciertos procesos de pago asíncronos o durante la revalidación de un pedido por parte del back office).

Para solucionarlo de forma robusta, si $params['cart'] no es una instancia de Cart, debemos intentar cargar el carrito desde el contexto global de PrestaShop. Si ni siquiera en el contexto hay un carrito válido, entonces sí, el módulo no tiene nada que procesar y puede retornar.

Aquí te dejo una modificación para el archivo paydisorfee/paydisorfee.php que debería resolver el problema:

function hookActionValidateOrderBefore($params)
    {
        if (!(int) Configuration::get('PAYDISORFEE_ENABLED')) {
            return;
        }
        if (!isset($params['cart']) || !($params['cart'] instanceof Cart)) {
            $params['cart'] = Context::getContext()->cart; // Intentamos obtener el carrito del contexto
            if (!($params['cart'] instanceof Cart)) { // Si sigue sin ser un Cart, salimos
                return;
            }

Con este cambio, primero verifica si $params['cart'] existe y es una instancia de Cart. Si no lo es, intenta recuperar el carrito del contexto actual. Solo si esa segunda opción también falla (es decir, no hay ningún Cart válido disponible), entonces el hook retorna sin hacer nada. De esta forma, aseguras que tu lógica interna siempre reciba un objeto Cart para aplicar los descuentos o recargos.

valeria9483

¡Increíble, @Milo! ¡Muchas gracias por la explicación tan clara y la solución!

He aplicado el cambio que sugeriste en paydisorfee/paydisorfee.php y he hecho varias pruebas con diferentes escenarios de pago y ¡funciona perfectamente! Ahora los descuentos y recargos se aplican de forma consistente en todos los casos, incluso en los que antes fallaban.

La verdad es que no se me había ocurrido tirar del Context::getContext()->cart de esa manera como fallback. ¡Salvado el día!

Gracias de nuevo a ti y a @manueldeco por vuestra ayuda.