Fallos en checkout con hook conorderfee al validar pedido

enriqueonline70

¡Hola a todos!

Estoy teniendo un problema bastante crítico con mi tienda PrestaShop 8.0.1. Mis clientes están reportando fallos al finalizar la compra; el pedido no se llega a realizar. Revisando los logs y depurando, he podido acotar el problema a la ejecución del hook hookActionValidateOrderBefore del módulo conorderfee.

Específicamente, parece que el problema ocurre cuando se ejecuta $this->runCartFeeSync($params['cart'], true) dentro de este hook. Si las condiciones para el cálculo de la tarifa cambian en el último momento o hay algún problema transitorio, la validación del pedido falla completamente y la compra se interrumpe.

Entiendo que es importante que la tarifa sea correcta, pero no quiero que un fallo temporal en la sincronización detenga todo el proceso de pedido. ¿Cómo podría adaptar o modificar este fragmento de código para que la validación del pedido proceda incluso si la sincronización de la tarifa encuentra problemas, quizás loggeando el error pero sin bloquear la compra?

Aquí tenéis el fragmento del conorderfee/conorderfee.php que he identificado:

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

¿Alguna idea de cómo manejar esto de forma robusta?

miriam_store

Hola @enriqueonline70, entiendo perfectamente la frustración. Un fallo en el checkout es lo peor.

Ese hook actionValidateOrderBefore es muy delicado porque se ejecuta justo antes de crear el pedido. Cualquier error no manejado ahí puede abortar la transacción. Si el problema es con $this->runCartFeeSync(), necesitas asegurarte de que esa función no arroje una excepción sin ser capturada o, si lo hace, que no impida la continuación del proceso.

PrestaShop 8, con su base en Symfony, es más estricto con el manejo de errores. La idea es que si esa sincronización falla de forma 'transitoria', no debería ser fatal. ¿Has revisado si la función runCartFeeSync en sí misma tiene algún try-catch o si simplemente lanza excepciones que no se manejan en el hook?

Milo

Hola @enriqueonline70 y @miriam_store,

Efectivamente, como comenta Miriam, el hookActionValidateOrderBefore es un punto crítico. La lógica de conorderfee intenta asegurar la coherencia de la tarifa justo antes de la validación, pero si runCartFeeSync arroja una excepción y no se captura, detendrá el proceso de pedido.

Para permitir que la validación del pedido continúe incluso si la sincronización de la tarifa falla (por un error transitorio, por ejemplo), debes envolver la llamada a $this->runCartFeeSync() en un bloque try-catch. Esto capturará cualquier excepción y te permitirá decidir qué hacer con ella (como registrarla para depuración) sin detener el flujo principal.

Aquí te muestro cómo podrías modificar el archivo conorderfee/conorderfee.php para incluir este manejo de errores justo después de las comprobaciones iniciales que has pegado:

if (!isset($params['cart']) || !($params['cart'] instanceof Cart)) {
            return;
        }
        try {
            $this->runCartFeeSync($params['cart'], true);
        } catch (Throwable $e) {
            // Opcional: registrar el error para futura revisión sin detener el pedido.
            // PrestaShopLogger::addLog('Error CONORDERFEE: ' . $e->getMessage(), 3, null, 'Cart', $params['cart']->id, true);
        }

Explicación de los cambios:

Hemos añadido un bloque try que envuelve la llamada a $this->runCartFeeSync($params['cart'], true);.
El bloque catch (Throwable $e) capturará cualquier excepción o error fatal que ocurra durante la ejecución de runCartFeeSync. Throwable es más amplio que Exception y es recomendable para PHP 7+.
Dentro del bloque catch, puedes añadir una línea para registrar el error usando PrestaShopLogger::addLog(). Esto es crucial para poder investigar por qué falló la sincronización, pero no detendrá el proceso de pedido. Si no necesitas registrarlo, puedes dejar el bloque catch vacío o con un comentario.

Con este cambio, la validación del pedido continuará, y solo se registrará el problema con la tarifa si ocurre, en lugar de abortar la compra.

enriqueonline70

¡Muchas gracias @Milo! ¡Esa es la solución que necesitaba!

He aplicado el try-catch en el conorderfee/conorderfee.php tal como lo indicaste, justo después de las comprobaciones iniciales y envolviendo la llamada a runCartFeeSync. He realizado varias pruebas de checkout con diferentes escenarios, simulando fallos y todo parece funcionar correctamente ahora. Los pedidos se validan y, si hubiera algún problema con la tarifa, al menos no interrumpe la compra.

También he incluido la línea de PrestaShopLogger::addLog para tener un registro de los errores por si acaso. Es un alivio ver que las compras ya no se bloquean.

¡Gracias de nuevo por la ayuda tan directa y eficaz!