Discussion :

[Freedolphin]

Bonjour à tous,

Mon hébergeur me conseille de passer mes sites à PHP 8. J'ai fait un premier test et grosso modo tout passe bien. Je détecte cependant parfois quelques erreurs.
Le problème, c'est qu'en cas d'erreur, le site affiche carrément une page vide. Par exemple, php8 semble très sensible à de petites lacunes, comme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if($_GET["action"]==envoi)

Il manque tout simplement les guillemets autour de "envoi". Un tel manque ne suscite aucune erreur en 7.4, mais avec la version 8, la page plante.

Ma question est : comment identifier les erreurs possibles avec php8 ? Existe-t-il un outil pour analyser le code et indiquer ce qui ne va pas ? Ou dois-je engager un expert pour revoir les fichiers ?

Merci d'avance pour vos réponses !

[cavo789]

Un outil merveilleux oui : Rector https://github.com/rectorphp/rector

Il va te permettre de faire l'upgrade de ton code et, utilisé régulièrement, de tes compétences puisqu'il va t'apprendre à mieux coder.

[Freedolphin]

Un outil merveilleux oui : Rector

Merci, ça a l'air très intéressant, mais alors je n'ai absolument pas compris comment l'utiliser.
Quant à la version en ligne, elle ne décèle pas mon problème de guillemets.

[Séb.]

Par exemple, php8 semble très sensible à de petites lacunes

J'aime comme tu minimises la situation
PHP était bien (trop) gentil d'interpréter ta constante inexistante comme un littéral, mois je t'aurais balancé direct une 500.

Un tel manque ne suscite aucune erreur en 7.4

Si, ce type d'erreur génère une notif comme "Notice: Use of undefined constant".
Tes réglages devaient la cacher.

Perso je corrigerais toutes les erreurs à la mano (on apprend mieux).

Et je passerais la rapport d'erreur au plus strict => error_reporting(E_ALL);

Dans un terminal, pour identifier tous les scripts comportant une égalité suivie d'un caractère (comme dans ton exemple) :

$ grep -rs '==[a-z]' /path/to/root/*

[cavo789]

Rector ne détecte pas les erreurs (il existe des dizaines d'outils pour ça comme phpstan ou encore psalm (ou d'autres)).

Rector permet de migrer ton code depuis une version de php vers une autre. Donc php 7.x verz 8.x entre autre.

[laurentSc]

Et je passerais le rapport d'erreur au plus strict => error_reporting(E_ALL);

C'est ce que je ferais aussi.
Et si c'est possible, exécuter les sites en local pour que les utilisateurs ne voient pas les erreurs.

[cavo789]

Bonjour

Voici un article récent sur Rector https://tighten.com/insights/automat...ith-rector-php

Il y est fait mention de la configuration à utiliser pour migrer ton code vers PHP 8.1 (chapitre Levelling Up)

[Freedolphin]

mois je t'aurais balancé direct une 500.

Rhoooo quel monde cruel ! Rassure-toi, php8 n'est pas avare d'erreur 500.

Perso je corrigerais toutes les erreurs à la mano (on apprend mieux).
Et je passerais la rapport d'erreur au plus strict => error_reporting(E_ALL);

Je vais en effet m'y atteler à la main (Rector n'est pas pour moi, mes compétences s'arrêtent là) et scruter les erreurs avec error_reporting.

Dans un terminal, pour identifier tous les scripts comportant une égalité suivie d'un caractère (comme dans ton exemple) :
$ grep -rs '==[a-z]' /path/to/root/*

Bon, là, je sèche, trop complexe. Petite question (bête), est-ce que les variables sont autorisées sans guillemets, du genre if ($i == $page) ?

[cavo789]

Bon, là, je sèche, trop complexe. Petite question (bête), est-ce que les variables sont autorisées sans guillemets, du genre if ($i == $page) ?

Oui, certainement !

Note $i === $page (avec trois égals) permet de valider et la valeur et le type (integer). Si p.ex. $i serait un chiffre et $page une chaîne de caractères alors le test sera en échec. C'est ce qu'on nomme le "strict type" en PHP et les tests devraient idéalement toujours se faire ainsi.