Ticket #5056 Premiers pas Composer #5057

On charge ici l’autoloader de Composer (vendor/autoload.php), ainsi que des petites dépendances (polyfill php 8.0 8.1 et mbstring).

On fait évoluer le code

sur certains strpos() en passant.
sur la présence de mb_*

@JamesRezo je vois un double /plugins/ export-ignore dans 53c982a2d4 ;)

Je me questionne aussi sur le /config/ vu qu’on a un fichier ecran_securite.php dedans pour le moment ?

@JamesRezo je vois un double /plugins/ export-ignore dans 53c982a2d4 ;)

bien vu ! c'est corrigé, merci :-)

Je me questionne aussi sur le /config/ vu qu’on a un fichier ecran_securite.php dedans pour le moment ?

C'est bon comme ça : .gitattributes exclut /config et composer.json archive.exclude le réintègre

ah je me rappelle, j'avais rien relu ni approuvé car la page de ce diff avait fait exploser mon navigateur (et bon là j'ai pas tout pu lire vu la quantité et même ya pas tout "Certains fichiers n'ont pas été affichés car il y a eu trop de fichiers modifiés dans ce diff")

Même si gitea nous force à afficher le diff avant de pouvoir valider un PR, j'y suis arrivé :)

J'ai bien envie de cliquer sur le bouton de fusion... :)

J’ai rebasé, squashé un peu plus, up les libs composer, et j’ai tenté de mettre des entête à la https://www.conventionalcommits.org/fr/v1.0.0/

Il manque encore les entrées de changelog
Je me demande s’il faudrait finalement pas attendre (ou nécessiter) PHP 8.0 mini : j’ai peur que certaines libs ne soient pas entièrement compatibles sur la plage PHP 7.4 à PHP 8.1 (notamment sur la correction des deprecated)

Ceci dit je n’ai pas vu de problème apparent là pour le moment.

Ça fait plusieurs fois avec Gitea là que je dois pousser mes commits (push --force) plusieurs fois : il fait comme si tout fonctionnait la première fois (on voit les notifs ici, et sur le bot shiraz), mais… pouf c’est toujours les anciens commits qui sont dans la branche pourtant.

Je dois push une seconde fois pour que ça passe.

Quelle est donc cette fourberie de machinerie ?

On aura quelques menus soucis du coup avec les plugins qui utilisaient les "classes" du compilo, qui sont maintenant dans un namepsace : 

ex saisies/balise/saisie.php Line 207 : Class "Champ" not found

https://git.spip.net/spip-contrib-extensions/saisies/src/branch/master/balise/saisie.php#L207

Au lieu de new Champ il faudra new \Spip\Core\Balise (ou Champ (alias), mais Balise sera plus clair pour tout le monde je crois).

- $c = new Champ;
+ $c = new \Spip\Core\Balise;

Il faudrait d’ailleurs trouver un meilleur namespace pour ces éléments du compilateur…

Spip\Template\Node ?
Spip\Compiler\Node ?
Spip\Parser\Node ?
Spip\Engine\Parser\Node ?

Quelques usages sur la zone

Dans les dépots de spip-contrib-extensions, on trouve

Champ

saisies/balise/saisie.php:              $b = new Champ;
tipafriend/balise/tipafriend.php:       $t = new Champ;

Texte

article_pdf/article_pdf_options.php:    $texte = new Texte;
article_pdf/article_pdf_options.php:                    $texte = new Texte;
article_pdf/article_pdf_options.php:    $texte = new Texte;
article_pdf/article_pdf_options.php:                    $texte = new Texte;
blocsdepliables/blocsdepliables_fonctions.php:  $texte = new Texte;
couteau_suisse/outils/blocs_fonctions.php:      $texte = new Texte; $texte->type='texte'; $texte->texte='visible';
saisies/balise/saisie.php:              $s = new Texte;
tipafriend/balise/tipafriend.php:       $t = new Texte;
tipafriend/balise/tipafriend.php:       $t = new Texte;

Contexte

adaptive_images/adaptive_images_options.php:                    $lieu = new Contexte();

Hum, du coup il faudra tester la version de SPIP pour appeler le bon code ? Pas moyen d'avoir un autre mécanisme de compatiblitéé ?

tu dois pouvoir faire un class_exists("\Spip\Core\Balise") pour savoir si tu dois instancier la nouvelle classe ou l'ancienne

Il faut probablement ajouter disallow: /vendor (et d'autres ?) au robots.txt.html n'est il pas ?

Bon, on peut dire que c'est bon maintenant, amha il est temps de merger :)

Ticket #5056 Premiers pas Composer #5057

Quelques usages sur la zone

Reviewers