John (b696633c) at 27 Feb 10:30
v2.0.6
John (24abc2cf) at 27 Feb 10:29
Synchronisation des statuts: ajout d'un watch dog pour détecter les...
Pour info, Campagnodon 2.0.5 installé en prod à l'instant. @alexandre est prévenu. Je garde un œil cette après-midi.
Ok, je me le note.
Je suis plutôt confiant. C'est toi qui voit. Peut être plutôt le faire en début de semaine, au cas où.
Testé Campagnodon v. 2.0.5 sur https://dev3-france.attac.org (SPIP 4.2.9, php 8.2.7) avec succès. Merci ! Je suis pour déployer en prod (et peux m'en occuper), mais peux tout à fait comprendre que tu préfères verrouiller un état qu'on sait fonctionnel (Campagnodon v. 2.0.2).
Y'a un tag v2.0.5 posé. Ça a l'air de fonctionner.
John (88bbe6e2) at 08 Feb 14:53
v2.0.5: Fix déclaration table spip_campagnodon_campagnes
John (c2c0de35) at 08 Feb 14:53
v2.0.5: Fix déclaration table spip_campagnodon_campagnes
Merci à toi, j'aurais pu également y penser avant !
Haha, bien vu. Ça m'apprendra à aller trop vite... Je m'en charge.
je me demande s'il ne faudrait pas également appliquer le changement au type 'campagnodon_campagnes' (https://code.globenet.org/attacfr/campagnodon/-/blob/d2514e96af031e45dee1095edde0c10a502a6372/base/campagnodon.php#L126)
🫣
"contre intuitif", le mot est faible ^^' J'ai passé des heures à lire, et relire la doc pour ce genre de subtilité.
@pierre-gabriel , j'ai tagué une version v2.0.4. Ça a l'air de fonctionner, même si je ne sais pas trop quels sont les impacts de la modif. Je te laisse voir si ça vaut le coup de l'installer en dev et/ou en prod.
PS: pour installer, faire un checkout sur le tag v2.0.4 (en prenant soin de le faire avec le bon user système), puis afficher la liste des plugins pour que SPIP prenne en compte la modif.
John (c9b6b416) at 08 Feb 13:21
v2.0.4: Fix déclaration table spip_campagnodon_transactions
John (d2514e96) at 08 Feb 13:21
v2.0.4: Fix déclaration table spip_campagnodon_transactions
En l'occurrence, c'est même contre intuitif (ou alors j'ai loupé une étape). La qualité de la doc SPIP n'aide pas vraiment au développement.
Rah, SPIP et les trucs implicites... Il n'y a aucune documentation claire sur quand il faut mettre le "s" ou pas. Je corrige, et fais quelques tests pour m'assurer que ça ne casse rien.
Salut @john, Thierry s'est indirectement penché sur le module « Campagnodon » (nous sommes en train de tester le fonctionnement du site Attac France sous SPIP 4.2), il m'a fait ce retour :
« La table spip_campagnodon_transactions est déclarée avec 'type' => 'campagnodon_transactions' donc ?exec=campagnodon_transactions devrait afficher une transaction et pas une liste de transactions. Avec 'type' => 'campagnodon_transaction', il ne devrait plus y avoir de warning (ou alors changer le nom de la page pour ne pas avoir exec = type). »
Au besoin, je peux voir avec lui pour plus de détail.
Ce qui n'enlève rien à ta remarque concernant le module « Livraison » et la nécessité de vérifier la présence de la clé avant traitement
Merci bien, je m'en occupe.