Bonsoir,
Modified Files:
inc_index.php3
[...]
- $s = $suite[$n = rand(0, $s-1)];
+ $s = $suite[$n = @rand(0, $s-1)];
Tous ces @ dans le code de SPIP, c'est plutôt un frein au déboggage,
qu'autre chose, c'est une façon pas très propre d'éviter de tester, il
faudrait les éviter tant que possible ...
Sur cette ligne là et d'autres, en plus, la pseudo économie de code
est déjà nocive pour la lecture, il ne faut pas avoir peur de séparer
en deux lignes.
De plus, utiliser une même variable (ici $s) pour stocker des valeurs
représentant différentes choses dans un même bout de code est aussi
une mauvaise chose pour la relecture du code (et je ne parle même pas
de typage des données).
Exemple ici, on a :