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.