Objeto Cart nulo en hookActionFilterPaymentOptions

marta_tienda30

Hola a todos,

Estoy teniendo un problema recurrente con un módulo (conordervalid.php) en PrestaShop 1.7.6.7. El método hookActionFilterPaymentOptions a veces se interrumpe prematuramente porque el objeto $cart no es una instancia Cart válida o Validate::isLoadedObject($cart) falla.

Aquí tenéis el fragmento de código donde se intenta obtener el carrito:

function hookActionFilterPaymentOptions(&$params)
    {
        if (!(int) Configuration::get('CONORDERVALID_ENABLED')) {
            return;
        }
        $cart = isset($params['cart']) && $params['cart'] instanceof Cart
            ? $params['cart']
            : (isset($this->context->cart) && $this->context->cart instanceof Cart ? $this->context->cart : null);

El problema es que, incluso después de esa asignación, a veces $cart sigue siendo nulo o un objeto Cart vacío que no está 'cargado' correctamente, y las opciones de pago no se filtran como deberían. ¿Alguien ha experimentado algo similar o sabe cómo asegurar que el carrito siempre sea un objeto válido y cargado en este hook?

lucas_tech

Hola @marta_tienda30,

Entiendo tu frustración. En PS 1.7.x, especialmente en versiones más antiguas como la 1.7.6.7 con PHP 7.x, el context->cart puede ser un poco volátil en ciertos puntos del ciclo de vida de la página o en llamadas AJAX.

Lo que veo en tu código es que ya intentas asignar el carrito desde $params o $this->context->cart. Sin embargo, la clave está en el Validate::isLoadedObject($cart). Aunque el $cart sea una instancia de Cart, si no se ha cargado explícitamente desde la base de datos (por ejemplo, con new Cart($id_cart)), no pasará esa validación. ¿Podrías confirmar si estás viendo algún patrón, como que falle solo para usuarios invitados o en el primer acceso al checkout?

ignacioes

Buenas @marta_tienda30,

A mí me pasó algo parecido con un módulo de descuentos, donde las reglas no se aplicaban porque el carrito parecía 'vacío' para el hook. Mi solución fue siempre añadir una comprobación estricta después de obtener el objeto del carrito. Es muy común que el context->cart esté inicializado pero no necesariamente cargado con los datos del usuario o de la sesión actual si no se maneja bien la sesión.

Creo que la clave es forzar una validación de objeto cargado antes de cualquier lógica del módulo. Si no está cargado, no hay nada que filtrar, ¿verdad? Es una buena práctica para evitar errores tontos.

Milo

Hola @marta_tienda30,

Los compañeros @lucas_tech y @ignacioes han dado con el punto clave. El hookActionFilterPaymentOptions se dispara en un momento donde el context->cart puede existir como objeto Cart (es decir, $cart instanceof Cart es true), pero no estar 'cargado' con los datos persistentes de la sesión o la base de datos (es decir, Validate::isLoadedObject($cart) es false).

Para asegurar que la lógica de tu módulo siempre trabaje con un carrito válido y completamente cargado, debes añadir esa comprobación explícita justo después de intentar obtener el carrito. Si el carrito no está cargado, el hook debe terminar su ejecución para evitar problemas.

Aquí tienes la modificación que deberías aplicar en tu archivo conordervalid/conordervalid.php:

$cart = isset($params['cart']) && $params['cart'] instanceof Cart
            ? $params['cart']
            : (isset($this->context->cart) && $this->context->cart instanceof Cart ? $this->context->cart : null);

        if (!Validate::isLoadedObject($cart)) {
            return; // Salir si el carrito no es válido o no está cargado.
        }
        // Aquí continúa el resto de la lógica del hook con un $cart asegurado

Con este cambio, te aseguras de que cualquier operación posterior en el hook siempre se realice sobre un objeto Cart que ha sido correctamente instanciado y cargado con sus datos desde la base de datos, lo cual es fundamental para el filtrado de opciones de pago.

marta_tienda30

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

He añadido la línea if (!Validate::isLoadedObject($cart)) { return; } justo después de la asignación del carrito en conordervalid/conordervalid.php y el problema ha desaparecido por completo. Ahora el módulo se comporta de forma consistente y los filtros de pago se aplican siempre correctamente.

Me estaba volviendo loca con esto. Es un detalle tan pequeño pero tan importante para la fiabilidad. ¡Gracias de nuevo a ti y a la comunidad por vuestra ayuda!