Passer de php 7.4 à php 8

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:

---

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 !

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.

Citation:

---

Envoyé par cavo789

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.

Citation:

---

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

---

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

Citation:

---

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

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.

Citation:

---

Envoyé par Séb.

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.

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)

Citation:

---

Envoyé par Séb.

mois je t'aurais balancé direct une 500.

---

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

Citation:

---

Envoyé par Séb.

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.

Citation:

---

Envoyé par Séb.

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

Citation:

---

Envoyé par Freedolphin

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.