TCPDF conflit – déclaration de constantes

  • Resolved Agent 3W

    (@agent3w)


    3 years, 12 months ago

    Bonjour,

    il y a un conflit avec le plugin de GLS.

    Le plugin n’est pas utilisable en l’état et plante le site. (C’est un soucis de librairie tcpdf)

    Origine du problème :
    – votre plugin fait appel à la librairie tcpdf
    – cette librairie utilise un fichier de configuration dont les constantes ne sont pas testées :
    fichier : colissimo-shipping-methods-for-woocommerce/lib/tcpdf/config/tcpdf_config.php

    => Il faut encadrer les déclarations de constante comme dans les autres fichiers de la librairie (vérifier que chaque constante n’existe pas avant de la déclarer)

    Pourquoi conflit ? car le plugin de GLS (déjà installé) utilise la même librairie avec le même problème. On a donc une double déclaration de variables identiques ! et donc l’erreur fatale PHP : CONSTANT ALREADY EXISTS.

    J’ai fait le même rapport à l’équipe de GLS.

    Merci pour votre aide.

    Bonne journée.
    Bien cordialement,
    Joffrey

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter Agent 3W

    (@agent3w)

    Quand on sécurise les constantes, ce les classes qui sont doublées…..

    Bref, je suggère de charger la dépendance uniquement si une classe ou une constante de celle-ci n’existe pas.

    Modifier toute la dépendance serait abusé du coup….

    Merci pour votre aide ??

    Bien cordialement,
    Joffrey

    Plugin Author Colissimo

    (@iscpcolissimo)

    Bonjour,

    pourriez vous remplacer le contenu du dossier wp-content/plugins/colissimo-shipping-methods-for-woocommerce/lib par celui que tu peux trouver à l’adresse suivante :

    https://we.tl/t-DjDRncfxCM

    ?a a fonctionné chez certains clients

    Cordialement.

    Thread Starter Agent 3W

    (@agent3w)

    Bonjour,

    ?a fonctionne !! Super merci.

    ?a va se passer comment à la prochaine mise à jour du coup ?

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘TCPDF conflit – déclaration de constantes’ is closed to new replies.