Upgrade PHP 8.5#2258
Conversation
Mopolo
left a comment
Mopolo
left a comment
There was a problem hiding this comment.
Un peu long à lire mais pas si compliqué en vrai. La majorité des modifs sont simples.
| @@ -42,6 +42,7 @@ public function iParseThePdfContent(): void | |||
| public function thePageOfThePdfShouldContain(string $page, string $expectedContent): void | |||
| { | |||
| $pageContent = $this->pdfPages[$page] ?? null; | |||
| $pageContent = str_replace("\t", " ", $pageContent); | |||
There was a problem hiding this comment.
À cause de la maj de la lib pdf je suppose ?
There was a problem hiding this comment.
En fait, je pensai qu'en faisant comme ça je n'aurai pas eu à intervenir sur les scenarii mais j'ai du quand même les corriger.
Est-ce que j'enlève ce code et je les corriges les tabulations ?
There was a problem hiding this comment.
Les tabulations c'est qu'il y en a dans les fichiers pdf ?
S'il faut quand même modifier les scénarios pourquoi corriger oui.
Faudra juste qu'on check bien que les factures sont correctement générées.
There was a problem hiding this comment.
En fait, smalot/pdfparser nous permet de parser les PDF pour les tests fonctionnels.
Donc la tabulation n'a pas d'importance, mais je vais faire ça propre.
On n'a pas touché à la génération des pdf.
# Conflicts: # sources/Afup/Association/Cotisations.php
stakovicz
force-pushed
the
feat-upgrade-php-8.5
branch
from
b0adb0d to
84be794
Compare
stakovicz
force-pushed
the
feat-upgrade-php-8.5
branch
from
9e2a4f5 to
7e52eb9
Compare
There was a problem hiding this comment.
Il faut aussi l'enlever du .gitignore du coup : https://github.com/afup/web/blob/master/.gitignore#L58
2 participants
On met à jour PHP 8.5.
J'ai essayé de faire des commits atomic, mais c'est pas évident.
J'ai passé un coup de Rector et j'ai du adapter quelques tests à cause de la montée de version de
smalot/pdfparser