Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

Problème avec hook ActionValidateOrderBefore et règles de livraison

julienshop

Bonjour à tous,

Je rencontre un problème assez frustrant avec mon module de règles de livraison personnalisées sur PrestaShop 1.7.3.3. Il semble que le hook ActionValidateOrderBefore ne fonctionne pas comme prévu et que nos règles de livraison ne soient pas appliquées du tout lors de la passation de commande.

En débugant, j'ai l'impression que le hook se termine prématurément, soit parce que le $params['cart'] n'est pas toujours une instance de Cart dans certains contextes, soit que getEngine()->resolveOrderCarrierReference($params) renvoie une chaîne vide de manière inattendue.

Voici le début de la fonction qui pose problème dans mon fichier shipmethodrules/shipmethodrules.php:

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

Est-ce que quelqu'un a déjà rencontré ce type de comportement sur cette version de PrestaShop ? Comment puis-je m'assurer que le panier est toujours valide et que le hook ne se quitte pas prématurément ?

leatech52

Salut @julienshop,

Je vois le souci avec cette vérification de Cart instance. C'est une vieille version de PrestaShop, et les objets Cart peuvent être un peu capricieux dans certains hooks si le contexte n'est pas parfaitement initialisé.

Peux-tu confirmer si cela se produit pour tous les transporteurs ou uniquement pour des transporteurs spécifiques ? Et utilises-tu le mode multiboutique ou des groupes de clients B2B ? Cela pourrait influencer la manière dont le panier est construit ou validé avant ce hook.

juliette_fr

Hello @julienshop,

Effectivement, la vérification !($params['cart'] instanceof Cart) est un point sensible. Avant ce return, as-tu essayé de faire un var_dump($params['cart']) pour voir ce qu'il contient réellement lorsque le problème se produit ? Parfois, ce n'est pas un objet Cart mais un null ou un tableau vide, ce qui justifie le return mais ne résout pas le problème de fond.

Il faudrait aussi vérifier si un autre module ne modifierait pas le panier ou le contexte avant que ton hook ne soit appelé. C'est du PrestaShop 1.7.3.3, c'est l'époque où les conflits de modules étaient monnaie courante. 🙂

Milo

Bonjour @julienshop, @leatech52, @juliette_fr,

Le problème que vous décrivez est un classique sur les versions plus anciennes de PrestaShop 1.7, notamment avec des hooks comme ActionValidateOrderBefore. Le contexte du panier n'est pas toujours entièrement hydraté ou l'objet Cart peut ne pas être directement accessible ou être d'un type inattendu si d'autres modules ou des overrides s'en mêlent juste avant.

La ligne !($params['cart'] instanceof Cart) est correcte pour valider l'objet, mais si le panier est validé par un processus en arrière-plan ou par un module de paiement tiers qui ne passe pas un objet Cart entièrement formé, cela peut poser problème.

Pour rendre la vérification plus robuste et éviter les retours prématurés lorsque le panier est techniquement là mais pas en tant qu'instance Cart directe, on peut ajouter une vérification supplémentaire ou s'assurer que l'objet est bien chargé. Voici une version légèrement modifiée qui s'assure que le panier est un objet et tente de le charger si ce n'est pas le cas, ou au moins de ne pas bloquer si un ID de panier est présent.

function hookActionValidateOrderBefore($params)
    {
        if (!(int) Configuration::get('SHIPMETHODRULES_ENABLED')) {
            return;
        }
        if (!isset($params['cart']) || !Validate::isLoadedObject($params['cart'])) {
            // Tente de charger l'objet Cart si seulement l'ID est passé
            if (isset($params['cart']->id) && $params['cart']->id > 0) {
                $params['cart'] = new Cart((int)$params['cart']->id);
            } else {

Ici, j'ai remplacé !($params['cart'] instanceof Cart) par !Validate::isLoadedObject($params['cart']) qui est plus souple et j'ai ajouté une petite logique pour recharger le panier si nous n'avons que son ID, ce qui arrive parfois. Cela permet de s'assurer que l'objet Cart est bien utilisable.

Pour une gestion plus avancée et maintenable de vos règles de livraison sans avoir à modifier directement le code des modules, je vous recommande d'explorer un module advanced shipping rules qui gère ces scénarios de manière plus robuste et centralisée.

julienshop

Merci beaucoup @Milo !

J'ai appliqué la modification que vous avez suggérée dans mon fichier shipmethodrules/shipmethodrules.php, et ça a résolu mon problème. Maintenant, les règles de livraison sont correctement appliquées lors de la validation de commande. Le $params['cart'] est bien géré, même dans les cas un peu tordus.

J'envisage aussi votre suggestion pour le module dédié, c'est clairement plus propre pour la maintenance à long terme. Encore merci pour l'aide rapide et efficace !