remove: Suppression des globales `spip_ecran` et `spip_display` (d00f70f0) · Validations · Glop / spip

CHANGELOG.md

+1 −0

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -28,4 +28,5 @@

		### Removed

		- #5278 Globales obsolètes `spip_ecran` et `spip_display`
		- spip-team/securite#3724 #5150 Suppression de la fonction `initialiser_sel()` (qui ne servait que pour la gestion de htpasswd déportée en plugin).

+8 −9

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -157,23 +157,22 @@ function init_body($rubrique = 'accueil', $sous_rubrique = 'accueil', $id_rubriq
		* @return string Classes CSS (séparées par des espaces)
		*/
		function init_body_class() {
		$prefs = $GLOBALS['visiteur_session']['prefs'] ?? [];

		$GLOBALS['spip_display'] = intval($prefs['display'] ?? 2);
		$spip_display_navigation = isset($prefs['display_navigation']) ? spip_sanitize_classname($prefs['display_navigation']) : 'navigation_avec_icones';
		$GLOBALS['spip_ecran'] = isset($_COOKIE['spip_ecran']) ? spip_sanitize_classname($_COOKIE['spip_ecran']) : 'etroit';

		$display_class = [
		0 => 'icones_img_texte'
		$display_modes = [
		0 => 'icones_img_texte' // défaut.
		/init/,
		1 => 'icones_texte',
		2 => 'icones_img_texte',
		3 => 'icones_img'
		];

		$prefs = $GLOBALS['visiteur_session']['prefs'] ?? [];

		$display_mode = $display_modes[intval($prefs['display'] ?? 0)] ?? $display_modes[0];
		$spip_display_navigation = isset($prefs['display_navigation']) ? spip_sanitize_classname($prefs['display_navigation']) : 'navigation_avec_icones';

		$couleur = intval($prefs['couleur'] ?? 2);

		$classes = $GLOBALS['spip_ecran'] . " spip-theme-colors-$couleur $spip_display_navigation " . $display_class[$GLOBALS['spip_display']];
		$classes = "spip-theme-colors-$couleur $spip_display_navigation $display_mode";
		return spip_sanitize_classname($classes);
		}

+7 −3

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -19,7 +19,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
		function inc_informer_dist($id, $col, $exclus, $rac, $type, $do = 'aff') {
		include_spip('inc/texte');
		$titre = $descriptif = '';
		if ($type == 'rubrique') {
		if ($type === 'rubrique') {
		$row = sql_fetsel('titre, descriptif', 'spip_rubriques', 'id_rubrique = ' . intval($id));
		if ($row) {
		$titre = typo($row['titre']);
		@@ -30,8 +30,12 @@ function inc_informer_dist($id, $col, $exclus, $rac, $type, $do = 'aff') {
		}

		$res = '';
		if ($type == 'rubrique' and $GLOBALS['spip_display'] != 1 and isset($GLOBALS['meta']['image_process'])) {
		if ($GLOBALS['meta']['image_process'] != 'non') {
		if (
		$type === 'rubrique'
		and intval($GLOBALS['visiteur_session']['prefs']['display'] ?? 0) !== 1
		and isset($GLOBALS['meta']['image_process'])
		) {
		if ($GLOBALS['meta']['image_process'] !== 'non') {
		$chercher_logo = charger_fonction('chercher_logo', 'inc');
		if ($res = $chercher_logo($id, 'id_rubrique', 'on')) {
		[$fid, $dir, $nom, $format] = $res;

+0 −4

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -139,10 +139,6 @@ elseif (!preg_match(',^[a-z4_][0-9a-z_-]*$,i', $exec)) {
		set_request('exec', $exec);
		}

		// compatibilite ascendante : obsolete, ne plus utiliser
		$GLOBALS['spip_display'] = intval($GLOBALS['visiteur_session']['prefs']['display'] ?? 0);
		$GLOBALS['spip_ecran'] = isset($_COOKIE['spip_ecran']) ? preg_replace('/[^a-z0-9]/i', '', $_COOKIE['spip_ecran']) : 'etroit';

		// si la langue est specifiee par cookie et ne correspond pas
		// (elle a ete changee dans une autre session, et on retombe sur un vieux cookie)
		// on appelle directement la fonction, car un appel d'action peut conduire a une boucle infinie