Caché de productos no se limpia tras pedidos validados

carlacart71

¡Hola a todos!

Tengo un problema con la invalidación de caché de productos en mi PrestaShop 1.7.0.4. Parece que después de validar un pedido, la caché de algunos productos no se refresca correctamente con PscCacheInvalidator::fromOrderProducts.

He estado revisando el código del módulo de caché y la función hookActionValidateOrder en prestaspeedcache.php utiliza params['order']->getProducts(). Mi sospecha es que para ciertos tipos de productos o en algunas condiciones, este método devuelve un array vacío, lo que impide que la caché se invalide para esos artículos.

Aquí tenéis el fragmento de código relevante:

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 encontrado un comportamiento similar o sabe por qué getProducts() podría devolver un array vacío en un hookActionValidateOrder? Necesito una forma de asegurarme de que la caché se invalide siempre para todos los productos del pedido.

alberto_bici

¡Hola @carlacart71!

A mí me pasó algo parecido hace tiempo en una tienda 1.7.1.x. Lo que me extraña es que getProducts() debería devolver siempre algo si el pedido tiene productos. ¿Has comprobado si el objeto $params['order'] está completamente cargado en ese punto del hook? A veces, dependiendo de cómo se dispara el hook o el estado del pedido, el objeto Order podría no tener todos sus detalles precargados.

También me pregunto si los productos que te dan problemas son de algún tipo especial, como productos virtuales, packs o productos con combinaciones muy específicas. A veces, la lógica interna de PrestaShop para esos casos puede ser un poco diferente.

joaquin_deco

Estoy con @alberto_bici. Es raro que getProducts() dé vacío para productos que sí están en el pedido. Lo primero que haría es añadir un var_dump($products) justo después de $products = $params['order']->getProducts(); y ver qué devuelve en un pedido problemático.

Si realmente está vacío, podríamos necesitar una alternativa para obtener los productos del pedido. En PrestaShop 1.7.0.4, podrías intentar acceder directamente a la tabla ps_order_detail para asegurarte de que recuperas todos los IDs de producto asociados a ese id_order y luego pasar esa información al PscCacheInvalidator.

Milo

Hola @carlacart71, entiendo perfectamente el problema que describes. Es una situación que puede ocurrir en PrestaShop 1.7.0.4 si el objeto Order en el momento del hook ActionValidateOrder no ha cargado completamente todos los detalles de los productos, o si hay algún tipo de producto que getProducts() está omitiendo.

Para asegurar una invalidación de caché robusta, debemos garantizar que se obtienen todos los productos del pedido directamente de los detalles del pedido, en caso de que el método getProducts() devuelva un array vacío. La mejor manera de hacerlo es recurrir a la tabla ps_order_detail.

Aquí te propongo una modificación para el fragmento de código en prestaspeedcache/prestaspeedcache.php:

function hookActionValidateOrder($params)
    {
        if (empty($params['order']) || !($params['order'] instanceof Order)) {
            return;
        }
        $products = $params['order']->getProducts();
        if (empty($products) && $params['order']->id) {
            // Fallback: Obtener productos directamente desde los detalles del pedido
            $products = Db::getInstance()->executeS('SELECT od.id_product, od.id_product_attribute FROM `'._DB_PREFIX_.'order_detail` od WHERE od.id_order = '.(int)$params['order']->id);
        }

Explicación del cambio:

Mantengo la llamada original a $params['order']->getProducts() como primera opción.
Añado una condición if (empty($products) && $params['order']->id): Si $products está vacío y tenemos un id_order válido, significa que necesitamos una alternativa.
Dentro de esa condición, realizo una consulta directa a la base de datos (ps_order_detail) para obtener id_product y id_product_attribute de todos los elementos asociados al id_order actual. Esto garantiza que no se omita ningún producto.
Finalmente, se pasa el array $products (ya sea el original o el obtenido por fallback) a PscCacheInvalidator::fromOrderProducts.

Esto debería solucionar el problema de la invalidación de caché para esos productos específicos.

carlacart71

¡Muchísimas gracias, @Milo! Esa es una solución muy elegante y robusta. He implementado el código que sugeriste en prestaspeedcache.php en mi PrestaShop 1.7.0.4 y he hecho varias pruebas con pedidos de diferentes tipos de productos.

Ahora la caché se invalida correctamente para todos los productos sin excepción. El fallback directo a ps_order_detail fue clave. ¡Me has ahorrado mucho tiempo de depuración!