Timeout en checkout con descuentos CONORDERDIS

valeriaiberia90

Hola a todos,

Estoy teniendo un problema recurrente en el proceso de pago de nuestra tienda PrestaShop 1.7.4.1. Ocasionalmente, el checkout se queda colgado con timeouts, o lo que es peor, muestra precios incorrectos justo antes de la validación final del pedido. Hemos notado que esto ocurre principalmente cuando el módulo CONORDERDIS y su opción CONORDERDIS_ENABLED están activos.

Sospechamos que la función runCartDiscountSync($params['cart'], true) que se ejecuta en el hook hookActionValidateOrderBefore podría ser demasiado lenta o estar causando conflictos. He revisado el código y el fragmento relevante en conorderdis/conorderdis.php es este:

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

¿Alguna idea de cómo podría adaptar o modificar esto para evitar los timeouts o los errores de precios? Parece que es el punto crítico justo antes de que se genere el pedido.

lorena_89

¡Uf, eso suena a un dolor de cabeza! @valeriaiberia90, entiendo perfectamente la frustración con los problemas en el checkout.

Esa parte del código que muestras es solo el inicio del hook. Lo importante sería ver qué hace exactamente runCartDiscountSync y si se está llamando varias veces por alguna razón durante el mismo proceso de validación. El hookActionValidateOrderBefore a veces puede ser invocado en escenarios donde no esperas que se ejecute la lógica completa una y otra vez, y si esa función hace cálculos pesados o consultas a la BD, puede ralentizarlo todo.

¿Estáis en un entorno con mucho tráfico? ¿Y qué versión de PHP usas con ese PrestaShop 1.7.4.1? Sé que es una versión algo antigua, pero a veces esos detalles son clave.

Milo

Hola @valeriaiberia90, gracias por compartir el fragmento. @lorena_89 ha dado en el clavo con el diagnóstico.

El hookActionValidateOrderBefore, especialmente en versiones antiguas de PrestaShop como la 1.7.4.1, puede ser invocado más de una vez durante un mismo ciclo de validación del pedido, por ejemplo, si hay redirecciones o recargas internas en el proceso de pago. Si la función runCartDiscountSync realiza cálculos intensivos, cada llamada redundante suma latencia, causando los timeouts y potencialmente conflictos si modifica el carrito múltiples veces.

Una solución común para evitar esto es asegurar que la lógica pesada se ejecute solo una vez por ID de carrito dentro de la misma petición. Puedes modificar tu archivo conorderdis/conorderdis.php así:

function hookActionValidateOrderBefore($params)
    {
        if (!(int) Configuration::get('CONORDERDIS_ENABLED')) {
            return;
        }
        if (!isset($params['cart']) || !($params['cart'] instanceof Cart)) {
            return;
        }
        static $syncedCarts = [];
        $cartId = (int) $params['cart']->id;

Con este cambio, he añadido una variable estática $syncedCarts que actúa como un 'flag' para cada ID de carrito. Si la función runCartDiscountSync ya se ha ejecutado para un carrito específico en la petición actual, la siguiente vez que se llame al hook, simplemente retornará sin volver a ejecutar la sincronización. Esto debería mitigar los problemas de rendimiento y evitar que los descuentos se apliquen o calculen incorrectamente debido a ejecuciones redundantes.

valeriaiberia90

¡Mil gracias, @Milo! Esa lógica de la variable static $syncedCarts tiene todo el sentido del mundo. No había pensado que el hook pudiera ser invocado varias veces en la misma petición.

He aplicado el cambio en nuestro conorderdis/conorderdis.php y después de varias pruebas en entornos de staging y producción, los timeouts en el checkout han desaparecido y los precios se muestran correctamente en todo momento. Parece que la función runCartDiscountSync era el cuello de botella cuando se ejecutaba redundantemente.

¡Un alivio enorme, de verdad! Gracias por la ayuda tan directa y efectiva.