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

Mise à jour de la méthode inconsistency() pour PS9 et PHP 8.2

mathilde_59

Bonjour à tous,

Je suis en train de migrer une boutique sous PrestaShop 9.1.4, avec PHP 8.2, et je rencontre un problème avec la méthode inconsistency() dans le fichier myordergridpro/myordergridpro.php. Le code actuel ne gère pas les typehints stricts de PHP 8.2 et je pense qu'il y a un souci de compatibilité avec la façon dont Symfony gère les événements et les services.

Actuellement, la logique derrière cette fonction déclenche des requêtes très lourdes sur plus de 200 000 commandes sans une bonne indexation, ce qui fait exploser le temps d'exécution et la limite mémoire de PHP. Je cherche à moderniser cette partie.

Voici le fragment de code incriminé :

function inconsistency()
    {
        return (new myordergridproUpdate())->inconsistency();
    }

Comment puis-je adapter ce snippet pour qu'il soit propre aux standards de PS9/Symfony et PHP 8.2, notamment en ajoutant des typehints stricts et en pensant à une meilleure intégration événementielle ?

julienpanier14

Salut @mathilde_59,

Je vois le problème. Le new myordergridproUpdate() en plein milieu, c'est typique des anciennes pratiques et ça pose un souci avec l'injection de dépendances et le conteneur de services de Symfony. En PS9, il faut vraiment éviter d'instancier des objets comme ça directement.

Est-ce que cette fonction inconsistency() est appelée comme un hook spécifique, ou c'est juste une fonction utilitaire dans ton module ? Le contexte d'appel est important pour savoir comment l'intégrer proprement.

quentinfr12

Je suis d'accord avec @julienpanier14. La première chose à faire serait de transformer myordergridproUpdate en un service Symfony pour qu'il puisse être injecté. Et pour la méthode inconsistency() elle-même, il faut absolument lui ajouter un type de retour strict pour PHP 8.2, comme un array ou un bool en fonction de ce qu'elle est censée retourner (une liste d'incohérences, ou juste un statut).

Sans ça, tu vas continuer à avoir des problèmes de performance et de stabilité avec les versions récentes de PHP.

Milo

Bonjour @mathilde_59,

Vos observations sont très pertinentes. Le pattern new MyClass() est en effet à proscrire dans un environnement Symfony/PrestaShop 9, car il contourne le conteneur de services et rend la gestion des dépendances compliquée. Quant à l'absence de typehints, PHP 8.2 est beaucoup plus strict et cela peut provoquer des erreurs inattendues.

Pour la compatibilité avec les souscripteurs d'événements Symfony, l'idée n'est pas de transformer cette fonction en souscripteur directement, mais de faire en sorte que la logique myordergridproUpdate soit un service injecté, et que sa méthode inconsistency() puisse être appelée par un service ou un contrôleur qui, lui, pourrait être déclenché par un événement.

Voici une version adaptée de votre snippet avec un typehint de retour strict, en supposant que inconsistency() renvoie un tableau d'incohérences (vide si aucune) :

function inconsistency(): array // Typehint strict PHP 8.2 pour le rapport d'incohérence
{
    // Dans PrestaShop 9, l'instanciation directe comme 'new myordergridproUpdate()'
    // devrait idéalement être remplacée par l'injection de dépendances via un service.
    // La logique métier doit être accessible par les souscripteurs d'événements Symfony.
    return (array) (new myordergridproUpdate())->inconsistency();
}

Cela résout le problème du typehint direct. Pour une intégration complète et maintenable dans PS9, il serait préférable d'encapsuler cette logique dans un module dédié qui respecte l'architecture Symfony. Vous pouvez trouver des solutions comme automatiser order grid – colonnes sur la boutique qui gèrent ces aspects de manière robuste.

mathilde_59

Merci beaucoup @Milo pour cette explication claire et le code corrigé ! C'est exactement ce qu'il me fallait pour démarrer. J'ai appliqué le typehint et ajouté le cast, et ça passe déjà mieux les vérifications de PHP 8.2. Le commentaire sur l'injection de dépendances est très pertinent, je vais regarder comment refactoriser myordergridproUpdate en tant que service pour une meilleure intégration Symfony. C'est un excellent point de départ pour la performance et la stabilité. Merci encore pour votre aide précieuse !