Nettoyage de globales (spip_ecran, spip_display)
Nettoyer 2 globales
spip_ecran
Je viens de tomber je sais pas trop comment (ajax?) sur
Warning: Undefined global variable $spip_ecran in ~/plugins-dist/mots/formulaires/editer_mot.php on line 144
Quelques notes :
- On utilise plus cette info écran étroit / large de config depuis SPIP 4.0 je pense et son interface plus responsive
- On peuple encore la globale avec le cookie qui n’est pas sûr d’exister ?
- On peuple une classe CSS dans le body du privé qui ne sert pas
- Il semble que la globale en plus dans certains cas n’est pas renseignée ?
Ma proposition est radicale :) On vire !
La question est quelle valeur on met lorsque c’était utilisé pour des formulaires tel que
$config['lignes'] = ($GLOBALS['spip_ecran'] == 'large') ? 8 : 5;
A priori actuellement ça prend 5 par défaut.
spip_display
Pour spip_display
que je sais plus trop comme ça ce que c’est, qui plus est un coup il est à 0 par défaut, un coup à 2 par défaut
$GLOBALS['spip_display'] = intval($GLOBALS['visiteur_session']['prefs']['display'] ?? 0);
...
$GLOBALS['spip_display'] = intval($prefs['display'] ?? 2);
Ce qui semble vouloir dire la même chose d’ailleurs… 0 étant alors la "valeur par défaut" je suppose ?
$display_class = [
0 => 'icones_img_texte'
/*init*/,
1 => 'icones_texte',
2 => 'icones_img_texte',
3 => 'icones_img'
];
Proposition : on ne peuple plus la globale.
On utilise juste la préférence visiteur si besoin, et pour cibler le privé, il y a déjà les classes CSS adaptées sur le body
si nécessaire.