Error en herramienta de simulación de carrito

inmahub64

Hola, estoy teniendo un problema con la herramienta de simulación de un módulo de reglas de envío que estoy usando en PrestaShop 9.1.3 con PHP 8.2.

A veces, al intentar simular, me devuelve un error JSON genérico como ['success' => false, 'error' => 'Mensaje de error']. Investigando un poco, parece que el fallo ocurre cuando Validate::isLoadedObject($cart) devuelve false.

Esto pasa si Tools::getValue('id_cart') no es un ID de carrito válido o si $this->context->cart no se ha cargado correctamente por alguna razón. No entiendo por qué new Cart($idCart) no siempre devuelve un objeto válido si id_cart es un número.

Aquí tenéis el fragmento de código donde sucede en shipmethodrules/controllers/front/simulate.php:

function postProcess()
    {
        if (!$this->context->employee || !(int) $this->context->employee->id) {
            $this->ajaxDie(json_encode(['success' => false, 'error' => 'Unauthorized']));
        }
        $idCart = (int) Tools::getValue('id_cart');
        $cart = $idCart > 0 ? new Cart($idCart) : $this->context->cart;
        if (!Validate::isLoadedObject($cart)) {

¿Alguna idea de cómo puedo hacer esta parte más robusta para asegurar que el carrito siempre sea válido antes de continuar?

manueldeco

Buenas @inmahub64, entiendo perfectamente lo que comentas. En PS9, con las validaciones estrictas y PHP 8.x, estas cosas saltan más. Me ha pasado algo similar al trabajar con carritos en el front-office.

Tal como lo tienes, si id_cart es un número mayor que 0, new Cart($idCart) siempre se ejecutará. El problema es que new Cart($idCart) siempre crea una instancia de la clase Cart, incluso si el ID no existe en la base de datos. Si el id_cart no existe, ese objeto Cart estará vacío y Validate::isLoadedObject($cart) devolverá false, que es lo que te está pasando.

¿Estás seguro de que el id_cart que se está pasando es siempre un carrito existente? A veces, las sesiones caducan o se pasan IDs de prueba que no existen y ahí es donde falla.

Milo

Hola @inmahub64 y @manueldeco,

@manueldeco lo ha clavado. El problema no es tanto que Validate::isLoadedObject($cart) falle, sino que el $cart se construye de una manera que puede ser un objeto Cart vacío si el id_cart no existe en la base de datos, incluso si es un número válido. El constructor de Cart no valida la existencia al instanciar.

La solución es asegurarse de que, si se proporciona un id_cart, este no solo sea un número, sino que también corresponda a un objeto Cart cargado correctamente. Podrías modificar esa línea así para que el objeto $cart solo se cree si realmente existe un carrito con ese ID, o si no, usar el carrito de contexto:

function postProcess()
    {
        if (!$this->context->employee || !(int) $this->context->employee->id) {
            $this->ajaxDie(json_encode(['success' => false, 'error' => 'Unauthorized']));
        }
        $idCart = (int) Tools::getValue('id_cart');
        $cart = ($idCart > 0 && Validate::isLoadedObject(new Cart($idCart))) ? new Cart($idCart) : $this->context->cart;
        if (!Validate::isLoadedObject($cart)) {

Con este cambio, solo se intentará cargar el new Cart($idCart) si idCart es positivo Y si ese ID de carrito existe y se puede cargar. Si no, se usará $this->context->cart como fallback. Esto debería solucionar los errores JSON por carritos no cargados.

Para una gestión más avanzada de reglas de envío y para no tener que tocar el código del módulo (lo cual no es ideal si vienen actualizaciones), te recomiendo echar un vistazo a soluciones dedicadas que te permitan ocultar transportistas en el carrito o aplicar tarifas condicionales de forma más robusta y sin overrides manuales.

inmahub64

¡Increíble @Milo! Esa modificación en la línea del new Cart($idCart) resuelve el problema perfectamente. Tenía sentido lo que decía @manueldeco, pero no sabía cómo implementarlo bien sin que me generara conflictos. Con el cambio que has sugerido, el simulador ya no lanza ese error JSON cuando le pasamos un id_cart que no existe o es inválido en mi PrestaShop 9.1.3.

Muchísimas gracias por la ayuda tan rápida y eficaz. Y gracias por la recomendación del módulo, lo tendré muy en cuenta para futuras mejoras y para no tener que estar parcheando código de terceros.