Discussion :

[laurentSc]

Bonjour,

je suis passé à PHP8 dimanche dernier (pas encore possible sur DVP de le signaler ) et depuis, l'instruction echo "<pre>"; n'est plus prise en compte ! En effet, j'ai une méthode de debug :

Code php :

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

1
2
3
4
5
6
7
8
 static public function debug($location, $var)
    {
        /*exemple d'appel : UtilFct::debug("Record.php 162 data deb de analyze_ticket ",self::$data);*/
        echo "<br/>".(is_string($location))?$location:''."<br/>";
        echo "<pre>";/*la variable $location sert à indiquer le nom du fichier et le numéro de la ligne où on a appelé la fonction car quand on a plus besoin, on sait plus où on l'a placé !*/
        var_dump($var);
        echo "</pre><br/>";
    }

qui utilise cette instruction (ligne 5) et quand je regarde le code html généré :

Code html :

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

csvimportcontroller 136 errors <pre>string(2809) "For SESA=SESA10038 etc

donc il a bouffé le point-virgule. Comment faire SVP ?

[CosmoKnacki]

Rien à voir avec la version de PHP. Je dirais un mauvais placement de parenthèses à la ligne du dessus:

Code :

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

echo "<br/>" . (is_string($location)?$location:'') . "<br/>";

Qu'on peut aussi écrire comme ça (pourquoi faire une concaténation lorsqu'on peut s'en passer):

Code :

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

1
2
3
echo '<br/>',
     ( is_string($location) ? $location : '' ),
     '<br/>';

[laurentSc]

C'est que je ne connaissais pas cette possibilité d'arguments multiples passés à echo...

Le lien avec le passage en PHP8 est que j'ai introduit ce bug en même temps...

[CosmoKnacki]

D'ailleurs le fait d'utiliser les multiples arguments séparés par des virgules avec echo règle le problème de précédence qui se posait avec le point, ce qui fait que tu peux très bien te passer des parenthèses autour de ton test ternaire:

Code :

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

1
2
3
echo '<br/>',
     is_string($location) ? $location : '',
     '<br/>';