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

Erreur fatale : instanciation de ConorderfeeEngine

mathildehub

Bonjour à tous,

Je rencontre un problème assez bloquant avec un module sur ma boutique PrestaShop 8.2.0 (PHP 8.0). J'ai une erreur fatale dès que la méthode getEngine() est appelée pour instancier ConorderfeeEngine. Il semblerait que le constructeur de ConorderfeeEngine échoue ou qu'il utilise des propriétés du module qui ne sont pas encore initialisées, alors qu'elles sont transmises via $this.

Voici le fragment de code en question dans conorderfee/conorderfee.php :

function getEngine()
    {
        if (!$this->engine) {
            $this->engine = new ConorderfeeEngine($this);
        }

        return $this->engine;
    }

Quand cette ligne new ConorderfeeEngine($this) s'exécute, ça plante complètement. Avez-vous déjà rencontré ce type de comportement où le contexte du module n'est pas prêt au moment de l'instanciation de son propre moteur ? Comment puis-je adapter ce snippet pour que l'engine soit correctement initialisé sans erreur ?

mathieu_deco

Salut @mathildehub,

Je vois le problème. C'est classique quand un objet enfant dépend fortement de l'état de son parent au moment de sa création. Sur quelle version exacte de PHP tournes-tu avec PrestaShop 8.2.0 ? Parfois, des comportements subtils liés à l'initialisation des $this peuvent varier légèrement. Est-ce que le constructeur de ConorderfeeEngine essaie d'accéder à des propriétés comme $module->context ou $module->id immédiatement après avoir reçu $this ? Le problème est probablement là.

marionvendeur52

Oui, @mathieu_deco a raison. Le $this passé au constructeur de ConorderfeeEngine est bien l'instance du module principal conorderfee. Si, dans le constructeur de ConorderfeeEngine, il tente d'accéder à $module->context ou à d'autres propriétés comme $module->configuration et que ces dernières ne sont pas encore pleinement établies par le cycle de vie de PrestaShop au moment de cet appel, ça peut effectivement provoquer une erreur fatale. C'est une question de timing d'initialisation. J'ai eu un problème similaire avec un autre module de paiement qui nécessitait le Context complet avant d'être instancié.

Milo

Bonjour @mathildehub,

Je comprends tout à fait la situation, c'est un piège courant. Lorsque ConorderfeeEngine est instancié via new ConorderfeeEngine($this), son constructeur s'attend à ce que l'objet module ($this) soit déjà dans un état stable et que toutes ses propriétés essentielles, comme $this->context, soient bien définies. Si le contexte n'est pas encore entièrement chargé ou si le module n'a pas fini son propre processus d'initialisation lors de l'appel à getEngine(), cela peut entraîner l'accès à des propriétés nulles ou non définies, provoquant l'erreur fatale que tu décris.

Pour résoudre ce problème de manière robuste, il faut s'assurer que le contexte du module est bien disponible avant de passer l'instance du module à son moteur. Voici une modification à apporter à votre méthode getEngine() dans conorderfee/conorderfee.php :

function getEngine()
    {
        if (!$this->engine) {
            // Assurer que le contexte du module est disponible
            if (!isset($this->context) || !($this->context instanceof \Context)) {
                $this->context = \Context::getContext();
            }
            $this->engine = new ConorderfeeEngine($this);
        }
        return $this->engine;

Ce petit ajout garantit que $this->context est toujours une instance valide de \Context avant que ConorderfeeEngine n'essaie d'y accéder. Cela stabilise l'initialisation du moteur et évite les erreurs dues à des propriétés non initialisées.

Pour des fonctionnalités plus avancées et une gestion plus poussée des frais de commande conditionnels, il est souvent plus simple et plus maintenable d'utiliser un module dédié comme Frais conditionnels de commande qui gère ces subtilités d'initialisation et de contexte nativement.

mathildehub

Merci beaucoup @Milo ! C'est exactement ce qu'il fallait. J'ai appliqué la modification à mon conorderfee/conorderfee.php, et l'erreur fatale a disparu. Le module s'initialise maintenant sans problème et l'engine fonctionne comme prévu. Je garde en tête la suggestion du module dédié pour l'avenir, c'est une excellente approche pour la stabilité. Merci encore pour l'aide précieuse !