[plugin] plxMyShop 1.0.4

cpalo

En relisant le fil, j'ai vu que j'avais déjà signalé cette erreur et que cela avait été corrigé avec la 1.04.. Je vais aller voir sur le github la modification en question. Peut etre une piste

cpalo

j'ai vu ce défaut avec la 5.8.10 sans l'avoir remarqué avec la 5.8.9 par contre

Avec PluXml 5.8.8 plxMyShop 0.13.2 fonctionne correctement.
Avec PluXml 5.8.9 Nous avons l'erreur deprecated (cf mon autre post)
PluXml 5.8.10 corrige cette erreur mais provoque l'erreur indiquée ci-dessus 'undefined variable empechant le choix d'un template de page pour les produits)

gcyrillus-nomade

bonsoir,

est-il possible d'avoir un zip de tes test avec plxMyShop (sans le fichier users.xml) et nous indiquer les versions de php utilisées? C'est un serveur, apache? j'imagine
Cdt

cpalo

Bonsoir
Je pars pour quelques jours( retour lundi)
Mais je vais préparer ça.. et mettre en ligne...cela dependra de ma connexion internet. Sinon au plus tard mardi.. ce qui me permettra de bien refaire le point sur mes differents tests
En local c'est xampp
et en ligne c'est Nuxit
j'utilise O2Switch pour WordPress
Cordialement

cpalo

Avec le plugin plxMyAllArchives, il y avait le même message d'erreur.
Avec ton correctif, tout est rentré dans l'ordre pour ce plugin.
Il faut donc que je creuse un peu plus pour corriger plxMyShop

cpalo

J'ai voulu réutiliser la version de plxMyShop correspondant dans ce post ( la 1.0.4) qui fonctionnait mais sans Paypal.
Sauf que maintenant j'ai ce message d'erreur
La difference c'est le passage en Php 8

Parse error: syntax error, unexpected variable "$this" in F:\htdocs\dvpts-PluXml-5.8.8\plxMyShop-5.8.4-1.0.2\plugins\plxMyShop\plxMyShop.php on line 146

Et la ligne en question:

if(!empty($_SESSION[__CLASS__]['prods']) OR !empty($_COOKIE[__CLASS__])){#[$this->dLang] #dLang == '' ici (Est chargé par plxMotorConstruct)

gcyrillus-nomade

Bonjour,

Cela ressemble à un commentaire, mais avec la syntaxe #[kekchose] PHP8 cherche à interpreter ce qu'il y a entre les crochets .
Voir https://www.php.net/manual/fr/language.attributes.overview.php - https://www.php.net/manual/fr/language.attributes.syntax.php

Les attributs peuvent donc être considérés comme un langage de configuration intégré directement dans le code.

Donc à priori, changer le # en // et virer les crochets si tu tiens à garder ce commentaire.
cdt

cpalo

Bonjour,
Conclusion de tous ces essais avant de repartir sur un nouveau post:
plxMyShop 1.04 (Stripe mais pas Paypal) compatible jusqu'à PluXml 5.8.4 et Php 7.4
plxMyShop 0.13.2 (Paypal) compatible jusqu'à PluXml 5.8.8 et Php 8.2
( à partir de PluXml 5.8.9, dans l'édition des produits les templates de pages ne s'affichent plus)

gcyrillus-nomade

Bonsoir,

est-il possible d'avoir un zip de tes test avec plxMyShop (sans le fichier users.xml) et nous indiquer les versions de php utilisées? C'est un serveur, apache? j'imagine
Cdt

Je ne connais pas vraiment ce plugin.
Nous ne sommes pas très nombreux sur le forum et j'ai de beaucoup moins de temps disponible pour aider dans les mois à venir et je ne vois pas grand monde réagir aux soucis des uns et des autres.Alors si tu peut fournir des archives de PluXml toute prête à tester avec le plugin déjà configuré avec quelques pages et qui te fait les misères que tu décrit, je pourrais à partir de la essayer de déboguer et comprendre/comparer les différences entre versions et selon les versions de PHP que tu indiques sans avoir à appendre à utiliser ce plugin et y mettre quelques produits.N'étant pas développeur de métier (aucune formation dans le domaine) , j'ai plutôt tendance à me tenir éloigner de ce qui peut toucher au e-commerce, à tord peut-être.

SudWebDesign bien plus expert que moi dans le domaine semble aux abonnés absent pour le moment

Cdt

cpalo

Bonsoir,
Je te remercie de toute l'aide que tu apportes. Je suis plus integrateur web que developpeur Php. J'ai donc remis les mains dans le cambouis( y compris avec wordpress).
Je n'avais pas oublié ta demande.. Durant ces quelques jours c'est ce que j'ai fait, et testé toutes les versions, et j'ai reperé à quelle version ces bugs se situaient pour comparer.
J'ai preparé deux sites en lignes pour tester et montrer,
Je vais préparer ces zip... tu me laisses quelques jours...

Cordialement

gcyrillus-nomade

Pas de soucis

gcyrillus-nomade

Bonsoir, finalement , ce n'etait pas trés long, le temps de regarder les warnings, puis de passer en version 5.8.9 et de corriger les autres warnings.

En gros: devant la class des plugins concernés : ajout d'un commentaire de configuration #[\AllowDynamicProperties] (voir https://forum.alsacreations.com/topic-20-90415-1.html ) qui m'a été utile pour simplement autorisé ce type de chose au lieu de redeclaré tout en amont dans la class.

Pour la version 1.0.4 commentaire à probleme réecrit en //[$this->dLang] #dLang == '' ici (pluXml 5.8.3) au lieu de #[$this->dLang].etc...

Selon la version du plugin (ligne 871 ou 1230 ) la variable $this->aProds[$lang] declaré en $this->aProds[$lang]=false; et a mettre en tableau vide car elle sera un tableau $this->aProds[$lang]=array(); .

Par contre, pas vu de soucis avec les deux versions pour le select des template de pages statiques. tout semble fonctionné.

Cdt

cpalo

Grand Merci..!
Je fais des modifs dans les templates, css et options.... et je partage
Cordialement

gcyrillus-nomade

J'ai trouvé une autre modif à faire sur le fichier editionProduitAdmin.php a la ligne 52 ou 56 (selon tes deux versions) :
C'est à propos de la fonction trim().
modifier : $content[$lang] = trim($plxPlugin->getFileProduct($id,$lang)); en $content[$lang] = trim((string)$plxPlugin->getFileProduct($id,$lang)); de façon à éviter un message de l’interpréteur.

cpalo

Merci
C'est noté