Ticket #5056 Premiers pas Composer #5057

Open

marcimat wants to merge 14 commits from issue_5056_composer_road into master

marcimat commented 4 months ago

Owner

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_*

marcimat added 4 commits 4 months ago

528b8fddd4 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP

On prépare un path 'ecrire/src'

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

94a0b802ab Ajout des dépendances aux polyfill php 8.0 et 8.1

f702a0e3e4 On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.

(mais qui n’activerait pas cette extension ?)

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever spip_substr_manuelle() notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction init_charset() exécutée,
mais ça serait à revoir / déplacer.

06fa87d87e Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

JamesRezo added 6 commits 4 months ago

508b4654fb déplacement de l'objet Champ dans Spip\Core

9c8324e655 déplacement de l'objet Boucle dans Spip\Core

c6c5453d54 déplacement de l'objet Critere dans Spip\Core

b2e4976840 déplacement des autres objets issus du fichier ecrire/public/interfaces.php dans Spip\Core

c2b2f757d5 feat(sqlite): déplacement des classes Sqlite dans Spip\Base

e3fdd36093 feat(deps): Ajout de composer.lock dans git

JamesRezo added 1 commit 4 months ago

0da15ef683 Spip\Base -> Spip\Sql

JamesRezo added 1 commit 4 months ago

53c982a2d4 .gitattributes pour la commande composer archive

b_b commented 4 months ago

Owner

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

marcimat commented 4 months ago

Poster

Owner

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

JamesRezo added 1 commit 4 months ago

77c7a488e1 éviter un doublon dans .gitattributes

JamesRezo commented 4 months ago

Owner

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

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

JamesRezo commented 4 months ago

Owner

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

JamesRezo added 1 commit 4 months ago

4ee8e652b9 fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

marcimat added 1 commit 4 months ago

8e9db73da8 Normalize composer.json

marcimat added 15 commits 4 months ago

2532e9770b On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP

On prépare un path 'ecrire/src'

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

09627464dc Ajout des dépendances aux polyfill php 8.0 et 8.1

fbdd3c3e7e On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.

(mais qui n’activerait pas cette extension ?)

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever spip_substr_manuelle() notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction init_charset() exécutée,
mais ça serait à revoir / déplacer.

17ba03f8f6 Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

aec8bf44a9 déplacement de l'objet Champ dans Spip\Core

9c847722c8 déplacement de l'objet Boucle dans Spip\Core

d6b790e4df déplacement de l'objet Critere dans Spip\Core

e463ef57e3 déplacement des autres objets issus du fichier ecrire/public/interfaces.php dans Spip\Core

b5d8d622da feat(sqlite): déplacement des classes Sqlite dans Spip\Base

1540a31673 feat(deps): Ajout de composer.lock dans git

aae190f292 Spip\Base -> Spip\Sql

eae4de588d .gitattributes pour la commande composer archive

fc6836936b éviter un doublon dans .gitattributes

f9425ee065 fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

58e0db85ac Normalize composer.json

marcimat added 15 commits 4 months ago

63c3e2d188 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP

On prépare un path 'ecrire/src'

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

7d4c093cc6 Ajout des dépendances aux polyfill php 8.0 et 8.1

30fd65dfec On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.

(mais qui n’activerait pas cette extension ?)

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever spip_substr_manuelle() notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction init_charset() exécutée,
mais ça serait à revoir / déplacer.

9ec7494219 Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

2a388354bc déplacement de l'objet Champ dans Spip\Core

97d6d60ac1 déplacement de l'objet Boucle dans Spip\Core

d8e2fa5153 déplacement de l'objet Critere dans Spip\Core

49d88de52e déplacement des autres objets issus du fichier ecrire/public/interfaces.php dans Spip\Core

269ce9db59 feat(sqlite): déplacement des classes Sqlite dans Spip\Base

3710463efa feat(deps): Ajout de composer.lock dans git

aaa4d61dae Spip\Base -> Spip\Sql

33a894ee1f .gitattributes pour la commande composer archive

35d9014bbd éviter un doublon dans .gitattributes

c37db2684f fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

6779e3e539 Normalize composer.json

marcimat added 1 commit 4 months ago

9ec464645b Plus besoin de inc/chiffrer avec l’autoloader

JamesRezo added 1 commit 4 months ago

9f92366478 Déplacement des class d'itérateurs dans Spip\Core\Iterateur

JamesRezo added 3 commits 4 months ago

35e8fb1e96 Déplacement de la classe Spip_Css_Vars_Collection

b625b3f2e2 Déplacement de la classe SPIP_Traductions_Description

d2bd257272 Déplacement de la classe Bouton

JamesRezo added 1 commit 4 months ago

d50214a77d Remplacement de idna_convert.class.php par sa lib d'origine

JamesRezo added 1 commit 3 months ago

d6c1e775c1 uopdate composer

marcimat added 22 commits 3 months ago

a7d4d77cd8 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP

On prépare un path 'ecrire/src'

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

e694103645 Ajout des dépendances aux polyfill php 8.0 et 8.1

fc84ff2723 On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.

(mais qui n’activerait pas cette extension ?)

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever spip_substr_manuelle() notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction init_charset() exécutée,
mais ça serait à revoir / déplacer.

0e2336bf13 Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

320bae7318 déplacement de l'objet Champ dans Spip\Core

de4256fd55 déplacement de l'objet Boucle dans Spip\Core

5e996b657a déplacement de l'objet Critere dans Spip\Core

dd492e6541 déplacement des autres objets issus du fichier ecrire/public/interfaces.php dans Spip\Core

74a89d36be feat(sqlite): déplacement des classes Sqlite dans Spip\Base

4544693339 feat(deps): Ajout de composer.lock dans git

912cc1a040 Spip\Base -> Spip\Sql

1cd4b6f58e .gitattributes pour la commande composer archive

ec7430e927 éviter un doublon dans .gitattributes

ba9c60c145 fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

bf00f04741 Normalize composer.json

91ae672533 Plus besoin de inc/chiffrer avec l’autoloader

4f8878a08a Déplacement des class d'itérateurs dans Spip\Core\Iterateur

7b11285d15 Déplacement de la classe Spip_Css_Vars_Collection

17c0f3e543 Déplacement de la classe SPIP_Traductions_Description

c5f0b31986 Déplacement de la classe Bouton

da768caeb9 Remplacement de idna_convert.class.php par sa lib d'origine

9f8ade6bde uopdate composer

JLuc referenced this pull request 2 months ago

Introduire Composer et son autoloader dans SPIP #5056

rastapopoulos approved these changes 2 months ago

rastapopoulos left a comment

Owner

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")

b_b approved these changes 2 months ago

b_b left a comment

Owner

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

b_b commented 2 months ago

Owner

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

marcimat added 22 commits 1 month ago

3b2b76e785 On charge l’autoloader de composer.json aux 3 points d’entrées de SPIP

On prépare un path 'ecrire/src'

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

e7d4aad48c Ajout des dépendances aux polyfill php 8.0 et 8.1

fa6d20a08b On charge et utilise le polyfill-mbstring. Les fonctions mb_ sont donc toujours disponibles.

(mais qui n’activerait pas cette extension ?)

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever spip_substr_manuelle() notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction init_charset() exécutée,
mais ça serait à revoir / déplacer.

64cd2e7a9a Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

9bd5d1f99f déplacement de l'objet Champ dans Spip\Core

ddf0e419e9 déplacement de l'objet Boucle dans Spip\Core

9cdea5ef78 déplacement de l'objet Critere dans Spip\Core

af97182149 déplacement des autres objets issus du fichier ecrire/public/interfaces.php dans Spip\Core

62c961199a feat(sqlite): déplacement des classes Sqlite dans Spip\Base

72d26b3602 feat(deps): Ajout de composer.lock dans git

461a2321ed Spip\Base -> Spip\Sql

1c20529e6f .gitattributes pour la commande composer archive

f12b72a878 éviter un doublon dans .gitattributes

f407d08933 fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

d479102d8c Normalize composer.json

f2dd9488a7 Plus besoin de inc/chiffrer avec l’autoloader

1c9712757c Déplacement des class d'itérateurs dans Spip\Core\Iterateur

8a047edfcc Déplacement de la classe Spip_Css_Vars_Collection

9ed375d284 Déplacement de la classe SPIP_Traductions_Description

2bb328cbce Déplacement de la classe Bouton

97f2d0e3e2 Remplacement de idna_convert.class.php par sa lib d'origine

8161944a46 uopdate composer

marcimat added 14 commits 4 weeks ago

4803f9332a feat(autoloader): Charger l’autoloader de Composer aux 3 points d’entrées de SPIP

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

f0cc2b7b17 build: Ajouter des dépendances aux polyfill php 8.0 et 8.1

ee366ca9ed refactor: Charger et utiliser le polyfill-mbstring.

Les fonctions mb_ sont donc toujours disponibles ; mais qui n’activerait pas cette extension ?

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever `spip_substr_manuelle()` notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction `init_charset()` exécutée, mais ça serait à revoir / déplacer.

bc191c39d6 refactor: Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

df4642f95e refactor: Déplacer les classes de ecrire/public dans Spip\Core

69a59172e7 feat(sqlite): Déplacer des classes Sqlite dans Spip\Sql

7185f1a599 feat(deps): Fournir composer.lock

09497c914c build: .gitattributes pour la commande composer archive

f7ccd40fcb fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

d86b5f44cd refactor: Plus besoin de `inc/chiffrer` avec l’autoloader

007cee4301 refactor: Déplacer d’autres classes dans Spip\ (dont les itérateurs)

c069f49f86 style: Normaliser composer.json

54e2442d1a build: Remplacer idna_convert.class.php par sa lib d'origine

2f3fe67b10 build: Update composer

marcimat commented 4 weeks ago

Poster

Owner

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.

marcimat added 14 commits 4 weeks ago

4803f9332a feat(autoloader): Charger l’autoloader de Composer aux 3 points d’entrées de SPIP

À partir de cet instant, SPIP nécessite d’avoir exécuté `composer install` ou `composer update`
pour fonctionner.

f0cc2b7b17 build: Ajouter des dépendances aux polyfill php 8.0 et 8.1

ee366ca9ed refactor: Charger et utiliser le polyfill-mbstring.

Les fonctions mb_ sont donc toujours disponibles ; mais qui n’activerait pas cette extension ?

Le polyfill ne prend en compte que l’utf8 (et ascii).

- On adapte nos fonctions en conséquence, ce qui permet d’enlever `spip_substr_manuelle()` notamment.
- On ne teste pas la présence de pcre non plus, il est compilé toujours avec PHP
- On déplace du code d’exécution dans une fonction `init_charset()` exécutée, mais ça serait à revoir / déplacer.

bc191c39d6 refactor: Utiliser str_contains lorsque possible (fournie par le polyfill pour php 7.4)

df4642f95e refactor: Déplacer les classes de ecrire/public dans Spip\Core

69a59172e7 feat(sqlite): Déplacer des classes Sqlite dans Spip\Sql

7185f1a599 feat(deps): Fournir composer.lock

09497c914c build: .gitattributes pour la commande composer archive

f7ccd40fcb fix(archives): modification de composer.json, .gitignore et .gitattributes pour que les commandes git archive et composer archive fonctionnent

d86b5f44cd refactor: Plus besoin de `inc/chiffrer` avec l’autoloader

007cee4301 refactor: Déplacer d’autres classes dans Spip\ (dont les itérateurs)

c069f49f86 style: Normaliser composer.json

54e2442d1a build: Remplacer idna_convert.class.php par sa lib d'origine

2f3fe67b10 build: Update composer

marcimat commented 4 weeks ago

Poster

Owner

Ç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 ?

goldstein requested review from rastapopoulos 3 weeks ago

goldstein requested review from b_b 3 weeks ago

b_b approved these changes 3 weeks ago

rastapopoulos approved these changes 3 weeks ago

Reviewers

b_b approved these changes 3 weeks ago

rastapopoulos approved these changes 3 weeks ago

This pull request has changes conflicting with the target branch.

ecrire/public/references.php

ecrire/req/sqlite_generique.php

No Milestone

No project

No Assignees

4 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.