Code pour PHP >= 7.4 #4991

Merged
marcimat merged 20 commits from issue_4990_raptor_php_74 into master 1 week ago
marcimat commented 1 week ago
Owner

Ticket #4990 : On adapte certaines parties du code avec une syntaxe fonctionnant à partir de PHP 7.4.

C’est toujours mieux que rien.

Pour cela j’ai utilisé Rector, mais il faut un peu le surveiller quand même, au vu de notre code d’origine bien loin des standards.

Ticket #4990 : On adapte certaines parties du code avec une syntaxe fonctionnant à partir de PHP 7.4. C’est toujours mieux que rien. Pour cela j’ai utilisé Rector, mais il faut un peu le surveiller quand même, au vu de notre code d’origine bien loin des standards.
marcimat added 20 commits 1 week ago
JamesRezo approved these changes 1 week ago
b_b commented 1 week ago
Owner

Dans 33f37523d4 le changement de syntaxe pour la valeur de $spip_version_code c'est voulu ?

Dans https://git.spip.net/spip/spip/commit/33f37523d47b7ac610b8f1ff6aaa65b07e67842d le changement de syntaxe pour la valeur de `$spip_version_code` c'est voulu ?
Poster
Owner

Dans 33f37523d4 le changement de syntaxe pour la valeur de $spip_version_code c'est voulu ?

Oui, mais la valeur ne change pas. PHP 7.4+ tolère des souligné dans les nombres pour séparer afin de rendre le nombre plus lisible : https://php.watch/versions/7.4/underscore_numeric_separator

Par défaut Rector met des soulignés tous les milliers. J’ai corrigé sa proposition pour que ça colle à la date utilisée.

> Dans https://git.spip.net/spip/spip/commit/33f37523d47b7ac610b8f1ff6aaa65b07e67842d le changement de syntaxe pour la valeur de `$spip_version_code` c'est voulu ? Oui, mais la valeur ne change pas. PHP 7.4+ tolère des souligné dans les nombres pour séparer afin de rendre le nombre plus lisible : https://php.watch/versions/7.4/underscore_numeric_separator Par défaut Rector met des soulignés tous les milliers. J’ai corrigé sa proposition pour que ça colle à la date utilisée.
b_b commented 1 week ago
Owner

Oui, mais la valeur ne change pas. PHP 7.4+ tolère des souligné dans les nombres pour séparer afin de rendre le nombre plus lisible

Woa, il est aussi formidable que toi ce PHP ! ^^

> Oui, mais la valeur ne change pas. PHP 7.4+ tolère des souligné dans les nombres pour séparer afin de rendre le nombre plus lisible Woa, il est aussi formidable que toi ce PHP ! ^^
b_b commented 1 week ago
Owner

Dans ac4c801d96 c'est quoi l'intérêt de remplacer rand() par andom_int(0, mt_getrandmax())) ? C'est pour les perfs ?

Sinon, joli le "fix" sur ecrire/action/tester.php dans ce même commit :)

Dans https://git.spip.net/spip/spip/commit/ac4c801d96b9fb91e12f7ec56ce6b8a444b7ada0 c'est quoi l'intérêt de remplacer `rand()` par `andom_int(0, mt_getrandmax()))` ? C'est pour les perfs ? Sinon, joli le "fix" sur `ecrire/action/tester.php` dans ce même commit :)
b_b commented 1 week ago
Owner

Rho et du spread operator dans 8cc7909a0c grande classe :)

Rho et du spread operator dans https://git.spip.net/spip/spip/commit/8cc7909a0c4beeb1388da1f5df2b0ec566e08398 grande classe :)
Poster
Owner

Dans ac4c801d96 c'est quoi l'intérêt de remplacer rand() par random_int(0, mt_getrandmax())) ? C'est pour les perfs ?

Et bien je ne sais pas exactement. Probablement pour la qualité du nombre aléatoire généré.

C’est fait par une instruction pour PHP 7.0+
https://github.com/rectorphp/rector-src/blob/main/rules/Php70/Rector/FuncCall/RandomFunctionRector.php

Qui a été introduit par ce commit, pas plus locace
c8c82aabfd (diff-9c2b7a43d20a62444cc4284b194f6d17d480c6c61afd33ff7754805f9b3c32da)

Et pour retrouver cet historique c’est pas évident tellement iels ont changés les répertoires de place très souvent !

> Dans https://git.spip.net/spip/spip/commit/ac4c801d96b9fb91e12f7ec56ce6b8a444b7ada0 c'est quoi l'intérêt de remplacer `rand()` par `random_int(0, mt_getrandmax()))` ? C'est pour les perfs ? Et bien je ne sais pas exactement. Probablement pour la qualité du nombre aléatoire généré. C’est fait par une instruction pour PHP 7.0+ https://github.com/rectorphp/rector-src/blob/main/rules/Php70/Rector/FuncCall/RandomFunctionRector.php Qui a été introduit par ce commit, pas plus locace https://github.com/rectorphp/rector-src/commit/c8c82aabfd6c4119693576e5154feb9e357aa307#diff-9c2b7a43d20a62444cc4284b194f6d17d480c6c61afd33ff7754805f9b3c32da Et pour retrouver cet historique c’est pas évident tellement iels ont changés les répertoires de place très souvent !
b_b commented 1 week ago
Owner

Bon, j'ai "presque" tout relu (pour le plaisir), maintenant je vais tenter d'approuvver si le diff veut bien s'afficher :p

Bon, j'ai "presque" tout relu (pour le plaisir), maintenant je vais tenter d'approuvver si le diff veut bien s'afficher :p
b_b approved these changes 1 week ago
Poster
Owner

Super, merci.

Super, merci.
marcimat merged commit 4ccfda6be9 into master 1 week ago
marcimat deleted branch issue_4990_raptor_php_74 1 week ago

Reviewers

JamesRezo approved these changes 1 week ago
b_b approved these changes 1 week ago
The pull request has been merged as 4ccfda6be9.
Sign in to join this conversation.
Loading…
There is no content yet.