CarrierReference vacío en hookActionValidateOrderBefore

inmaes82

Hola a todos,

Estoy teniendo un problema peculiar con mi módulo personalizado para reglas de envío en PrestaShop 8.2.8. He implementado un hookActionValidateOrderBefore para procesar ciertas lógicas antes de que se valide una orden, y se supone que debe obtener la referencia del transportista usando $this->getEngine()->resolveOrderCarrierReference($params).

El problema es que, a veces, este método devuelve un $carrierReference vacío, incluso cuando el módulo está habilitado y $params['cart'] es un objeto Cart válido. Esto impide que mis reglas se apliquen correctamente.

Aquí tenéis un fragmento del inicio de mi hook:

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

Después de estas comprobaciones, es donde $carrierReference a veces sale vacío. ¿Alguien ha experimentado algo similar o tiene alguna idea de por qué podría pasar esto y cómo puedo asegurarme de que siempre obtengo una referencia de transportista válida?

gonzalosevilla

Hola @inmaes82,

Entiendo tu frustración. Ese resolveOrderCarrierReference puede ser un poco delicado en ciertas situaciones. A veces depende mucho de cómo se hayan establecido las delivery_option_list dentro del carrito en ese momento, o si el flujo de checkout es un poco atípico (por ejemplo, si se crea un pedido de forma programática o con módulos de checkout muy personalizados).

¿Podrías confirmar si esto ocurre con todos los transportistas o solo con algunos específicos? Y también, ¿en qué punto exacto del proceso de compra o tipo de cliente (B2B, invitado) te está fallando más frecuentemente? Saber eso podría ayudar a acotar el problema.

Milo

Hola @inmaes82 y @gonzalosevilla,

Es un buen punto lo que comenta @gonzalosevilla. El resolveOrderCarrierReference en PrestaShop 8 puede ser inconsistente si la estructura delivery_option_list no está perfectamente alineada o si el contexto del pedido no se propaga completamente en $params en todos los escenarios. Aunque la Cart es válida, la información sobre el transportista puede no estar siempre disponible de la forma que espera ese método.

Para solucionar esto y asegurarte de que siempre tienes una referencia de transportista, te sugiero añadir una lógica de fallback justo después de tu llamada a resolveOrderCarrierReference. Esto intentará obtener el ID del transportista directamente del objeto Cart si la primera llamada falla o devuelve un valor vacío. Deberías reemplazar la línea donde obtienes $carrierReference y añadir estas comprobaciones:

$carrierReference = $this->getEngine()->resolveOrderCarrierReference($params);

if (empty($carrierReference) && isset($params['cart']->id_carrier)) {
    $carrierReference = (int) $params['cart']->id_carrier;
}

if (empty($carrierReference)) {
    return; // No se pudo determinar el transportista, salir
}
// Continúa con la lógica del hook aquí

Con esto, si resolveOrderCarrierReference no te da un valor, el hook intentará usar id_carrier del carrito, que suele estar disponible. Si incluso después de eso no hay referencia, entonces el hook saldrá, evitando errores posteriores con reglas sin transportista.

inmaes82

¡Muchísimas gracias, @Milo!

Acabo de implementar la lógica de fallback que sugeriste en mi hookActionValidateOrderBefore y he estado haciendo varias pruebas con diferentes escenarios y transportistas. ¡Funciona a la perfección!

El id_carrier del carrito como respaldo es una solución muy robusta para esos casos donde resolveOrderCarrierReference se queda en blanco. Ahora mis reglas se aplican correctamente en todos los pedidos. ¡Me has salvado de un gran dolor de cabeza! Gracias por la ayuda tan directa y efectiva.