Caché de productos no se vacía tras validar pedido

raul_store

Hola a todos,

Estoy teniendo un problema peculiar con la invalidación de la caché en mi tienda PrestaShop 1.7.6.0. Después de que un cliente completa un pedido, la caché de los productos involucrados a veces no se vacía como debería. He estado revisando el módulo de caché que usamos y parece que el hookActionValidateOrder se llama correctamente.

El problema es que la función PscCacheInvalidator::fromOrderProducts que se encarga de invalidar la caché, parece no recibir los productos en algunas ocasiones. He depurado un poco y sospecho que $params['order']->getProducts() podría estar devolviendo un array vacío bajo ciertas condiciones específicas, lo que impide la invalidación.

Aquí tenéis el fragmento de código que estoy investigando en prestaspeedcache/prestaspeedcache.php:

function hookActionValidateOrder($params)
{
    if (empty($params['order']) || !($params['order'] instanceof Order)) {
        return;
    }
    $products = $params['order']->getProducts();
    PscCacheInvalidator::fromOrderProducts(is_array($products) ? $products : array());
}

¿Alguien ha experimentado algo similar en PS 1.7.6.0 o tiene alguna idea de por qué getProducts() podría devolver vacío justo después de la validación del pedido? ¿Hay alguna forma más robusta de asegurar que los productos se obtengan en este punto?

miriamdev

Hola @raul_store,

Entiendo tu frustración. Ese comportamiento de getProducts() es raro si el pedido ya está validado y los productos se han guardado. En 1.7.6.0, Order::getProducts() debería funcionar bien en el hookActionValidateOrder porque el objeto Order ya está persistido en la base de datos con sus detalles.

Mi primera pregunta sería: ¿Has comprobado si el id del objeto $params['order'] está siempre presente y es válido antes de llamar a getProducts()? A veces, en ciertas condiciones de carrera o si la transacción aún no se ha completado del todo, el objeto podría no estar completamente cargado o su ID podría no ser accesible de inmediato, aunque no debería ser el caso para este hook. ¿Podría ser un problema de un módulo de pago que interfiere con la transacción de la base de datos?

ruben_deco

Uhm, @miriamdev tiene un buen punto. También te preguntaría, @raul_store, ¿qué tipo de productos son? ¿Son productos simples, con combinaciones, o tal vez productos virtuales? A veces, los módulos personalizados o ciertos tipos de productos pueden tener un ciclo de vida de guardado ligeramente diferente que podría crear un micro retraso.

Otra cosa que podrías intentar es añadir un pequeño Logger::addLog() (o simplemente un die(var_dump($products)) si estás depurando en un entorno de desarrollo) justo antes de PscCacheInvalidator::fromOrderProducts para ver exactamente qué te devuelve $params['order']->getProducts() cuando falla. Así confirmas si realmente es un array vacío o si hay algo más raro en el formato.

Milo

Hola @raul_store, @miriamdev y @ruben_deco,

Tenéis razón en que es un comportamiento inusual para Order::getProducts() en hookActionValidateOrder en PrestaShop 1.7.6.0. Aunque el objeto Order se pasa en el hook, en escenarios muy específicos o con ciertos módulos de pago que pueden alterar el flujo transaccional, a veces el objeto $params['order'] no está completamente hidratado o los OrderDetails no son accesibles de inmediato a través de él, incluso si el ID del pedido ya existe.

Para asegurar que siempre trabajamos con un objeto Order completamente fresco y cargado desde la base de datos, podemos recargarlo explícitamente usando su ID. Esto fuerza la hidratación completa y debería resolver los casos donde getProducts() devuelve vacío.

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

function hookActionValidateOrder($params)
{
    if (empty($params['order']) || !($params['order'] instanceof Order) || empty($params['order']->id)) {
        return;
    }
    $order = new Order((int) $params['order']->id); // Recargamos el objeto Order
    $products = $order->getProducts(); // Ahora obtenemos los productos del objeto recargado
    PscCacheInvalidator::fromOrderProducts(is_array($products) ? $products : array());
}

Con new Order((int) $params['order']->id); nos aseguramos de que el objeto $order es una instancia completamente nueva y cargada con todos los detalles del pedido desde la base de datos, incluyendo sus productos asociados. Luego, al llamar a $order->getProducts(), obtendremos la lista completa de productos del pedido sin importar el estado transaccional previo.

raul_store

¡Muchas gracias, @Milo!

Acabo de aplicar el cambio que sugeriste en prestaspeedcache/prestaspeedcache.php y lo he probado con varios pedidos en diferentes condiciones. Parece que la caché de productos ahora se está vaciando correctamente cada vez. La solución de recargar el objeto Order explícitamente ha funcionado perfectamente.

¡Realmente aprecio la ayuda tan rápida y precisa! Esto me ha salvado de varios dolores de cabeza.