spip / SPIP

ecrire/inc/flock.php

Spacing +18 added lines, -18 removed lines

@@ -122,7 +122,7 @@ 
 		$contenu = @gzfile($fichier);
 	}
 
-	return is_array($contenu) ? join('', $contenu) : (string)$contenu;
+	return is_array($contenu) ? join('', $contenu) : (string) $contenu;
 }
 
 
@@ -315,9 +315,9 @@ 
  */
 function ecrire_fichier_securise($fichier, $contenu, $ecrire_quand_meme = false, $truncate = true) {
 	if (substr($fichier, -4) !== '.php') {
-		spip_log('Erreur de programmation: ' . $fichier . ' doit finir par .php');
+		spip_log('Erreur de programmation: '.$fichier.' doit finir par .php');
 	}
-	$contenu = '<' . "?php die ('Acces interdit'); ?" . ">\n" . $contenu;
+	$contenu = '<'."?php die ('Acces interdit'); ?".">\n".$contenu;
 
 	return ecrire_fichier($fichier, $contenu, $ecrire_quand_meme, $truncate);
 }
@@ -330,7 +330,7 @@ 
  * @return bool
  */
 function ecrire_fichier_calcule_si_modifie($fichier, $contenu, $force = false, $use_copy = false) {
-	$fichier_tmp = $fichier . '.last';
+	$fichier_tmp = $fichier.'.last';
 	if (!ecrire_fichier($fichier_tmp, $contenu, true)) {
 		return false;
 	}
@@ -370,7 +370,7 @@ 
  */
 function lire_fichier_securise($fichier, &$contenu, $options = []) {
 	if ($res = lire_fichier($fichier, $contenu, $options)) {
-		$contenu = substr($contenu, strlen('<' . "?php die ('Acces interdit'); ?" . ">\n"));
+		$contenu = substr($contenu, strlen('<'."?php die ('Acces interdit'); ?".">\n"));
 	}
 
 	return $res;
@@ -540,7 +540,7 @@ 
 				$wait = 0;
 			}
 		}
-		spip_log('Probleme de configuration opcache.revalidate_freq ' . $duree . 's : on attend ' . $wait . 's', _LOG_INFO_IMPORTANTE);
+		spip_log('Probleme de configuration opcache.revalidate_freq '.$duree.'s : on attend '.$wait.'s', _LOG_INFO_IMPORTANTE);
 		if ($wait) {
 			sleep($duree + 1);
 		}
@@ -568,9 +568,9 @@ 
 		if ($item == '.' || $item == '..') {
 			continue;
 		}
-		if (!supprimer_repertoire($dir . '/' . $item)) {
-			@chmod($dir . '/' . $item, 0777);
-			if (!supprimer_repertoire($dir . '/' . $item)) {
+		if (!supprimer_repertoire($dir.'/'.$item)) {
+			@chmod($dir.'/'.$item, 0777);
+			if (!supprimer_repertoire($dir.'/'.$item)) {
 				return false;
 			}
 		};
@@ -615,7 +615,7 @@ 
 	if (!strlen($subdir)) {
 		$n = strrpos($base, '/');
 		if ($n === false) {
-			return $nobase ? '' : ($base . '/');
+			return $nobase ? '' : ($base.'/');
 		}
 		$subdir = substr($base, $n + 1);
 		$base = substr($base, 0, $n + 1);
@@ -625,14 +625,14 @@ 
 	}
 
 	$baseaff = $nobase ? '' : $base;
-	if (isset($dirs[$base . $subdir])) {
-		return $baseaff . $dirs[$base . $subdir];
+	if (isset($dirs[$base.$subdir])) {
+		return $baseaff.$dirs[$base.$subdir];
 	}
 
-	$path = $base . $subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
+	$path = $base.$subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
 
 	if (file_exists("$path/.ok")) {
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+		return $baseaff.($dirs[$base.$subdir] = "$subdir/");
 	}
 
 	@mkdir($path, _SPIP_CHMOD);
@@ -642,7 +642,7 @@ 
 		@touch("$path/.ok");
 		spip_log("creation $base$subdir/");
 
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+		return $baseaff.($dirs[$base.$subdir] = "$subdir/");
 	}
 
 	// en cas d'echec c'est peut etre tout simplement que le disque est plein :
@@ -652,10 +652,10 @@ 
 		return '';
 	}
 	if (!_DIR_RESTREINT) {
-		$base = preg_replace(',^' . _DIR_RACINE . ',', '', $base);
+		$base = preg_replace(',^'._DIR_RACINE.',', '', $base);
 	}
 	$base .= $subdir;
-	raler_fichier($base . '/.ok');
+	raler_fichier($base.'/.ok');
 }
 
 
@@ -728,7 +728,7 @@ 
 								$maxfiles - $nbfiles,
 								$recurs
 							);
-							$fichiers = array_merge((array)$beginning, (array)$end);
+							$fichiers = array_merge((array) $beginning, (array) $end);
 							$nbfiles = count($fichiers);
 						}
 					}

ecrire/inc/couleurs.php

Spacing +5 added lines, -5 removed lines

@@ -70,17 +70,17 @@
 		// compat < SPIP 3.3
 		include_spip('inc/filtres_images_mini');
 		$c['couleur_foncee'] = $c['couleur_theme'];
-		$c['couleur_claire'] = '#' . couleur_eclaircir($c['couleur_theme'], .5);
+		$c['couleur_claire'] = '#'.couleur_eclaircir($c['couleur_theme'], .5);
 
 		return
-			'couleur_theme=' . substr($c['couleur_theme'], 1)
+			'couleur_theme='.substr($c['couleur_theme'], 1)
 			// compat < SPIP 3.3
-			. '&couleur_claire=' . substr($c['couleur_claire'], 1)
-			. '&couleur_foncee=' . substr($c['couleur_foncee'], 1);
+			. '&couleur_claire='.substr($c['couleur_claire'], 1)
+			. '&couleur_foncee='.substr($c['couleur_foncee'], 1);
 	} else {
 		if (is_array($choix)) {
 			// compat < SPIP 3.3
-			$compat_spip_33 = function ($c) {
+			$compat_spip_33 = function($c) {
 				if (!isset($c['couleur_theme'])) {
 					$c['couleur_theme'] = $c['couleur_foncee'];
 					unset($c['couleur_foncee']);

ecrire/inc/preselectionner_parent_nouvel_objet.php

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
 		$id_rubrique = $GLOBALS['connect_id_rubrique'][0];
 	} elseif (
 		is_int(_AUTO_SELECTION_RUBRIQUE)
-		and sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . intval(_AUTO_SELECTION_RUBRIQUE))
+		and sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique='.intval(_AUTO_SELECTION_RUBRIQUE))
 	) {
 		$id_rubrique = _AUTO_SELECTION_RUBRIQUE;
 	} else {
@@ -51,12 +51,12 @@ 
 		$id_rubrique = $row_rub['id_rubrique'];
 	}
 	// si le choix ne convient pas, on cherche dans un secteur
-	if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
+	if (!autoriser('creer'.$objet.'dans', 'rubrique', $id_rubrique)) {
 		$id_rubrique = '';
 		// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
 		$res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
 		while (!$id_rubrique and $row_rub = sql_fetch($res)) {
-			if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
+			if (autoriser('creer'.$objet.'dans', 'rubrique', $row_rub['id_rubrique'])) {
 				$id_rubrique = $row_rub['id_rubrique'];
 			}
 		}

ecrire/inc/xml.php

Spacing +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
 					$p !== false
 					and ($morceau = substr($txt, $d, $p - $d))
 					and (($nopen += preg_match_all(
-						'{<' . preg_quote($closing_tag) . '(\s*>|\s[^>]*[^/>]>)}is',
+						'{<'.preg_quote($closing_tag).'(\s*>|\s[^>]*[^/>]>)}is',
 						$morceau,
 						$matches,
 						PREG_SET_ORDER

ecrire/inc/traduire.php

Spacing +24 added lines, -24 removed lines

@@ -41,7 +41,7 @@ 
 	static $dirs = [];
 	$liste = [];
 	foreach (creer_chemin() as $dir) {
-		if (!isset($dirs[$a = $dir . $dirname])) {
+		if (!isset($dirs[$a = $dir.$dirname])) {
 			$dirs[$a] = (is_dir($a) || !$a);
 		}
 		if ($dirs[$a]) {
@@ -67,21 +67,21 @@ 
  **/
 function chercher_module_lang($module, $lang = '') {
 	if ($lang) {
-		$lang = '_' . $lang;
+		$lang = '_'.$lang;
 	}
 
 	// 1) dans un repertoire nomme lang/ se trouvant sur le chemin
 	if (
 		$f = ($module == 'local'
-		? find_in_path($module . $lang . '.php', 'lang/')
-		: find_langs_in_path($module . $lang . '.php', 'lang/'))
+		? find_in_path($module.$lang.'.php', 'lang/')
+		: find_langs_in_path($module.$lang.'.php', 'lang/'))
 	) {
 		return is_array($f) ? $f : [$f];
 	}
 
 	// 2) directement dans le chemin (old style, uniquement pour local)
 	return (($module == 'local') or strpos($module, '/'))
-		? (($f = find_in_path($module . $lang . '.php')) ? [$f] : false)
+		? (($f = find_in_path($module.$lang.'.php')) ? [$f] : false)
 		: false;
 }
 
@@ -105,7 +105,7 @@ 
  **/
 function charger_langue($lang, $module = 'spip') {
 	static $langs = [];
-	$var = 'i18n_' . $module . '_' . $lang;
+	$var = 'i18n_'.$module.'_'.$lang;
 	if (!isset($langs[$lang])) {
 		$langs[$lang] = [];
 		if ($lang) {
@@ -120,13 +120,13 @@ 
 	}
 	foreach ($langs[$lang] as $l) {
 		if ($fichiers_lang = chercher_module_lang($module, $l)) {
-			$GLOBALS['idx_lang'] = 'i18n_' . $module . '_' . $l;
+			$GLOBALS['idx_lang'] = 'i18n_'.$module.'_'.$l;
 			include(array_shift($fichiers_lang));
 			surcharger_langue($fichiers_lang);
 			if ($l !== $lang) {
-				$GLOBALS[$var] = &$GLOBALS['i18n_' . $module . '_' . $l];
+				$GLOBALS[$var] = &$GLOBALS['i18n_'.$module.'_'.$l];
 			}
-			$GLOBALS['lang_' . $var] = $l;
+			$GLOBALS['lang_'.$var] = $l;
 			#spip_log("module de langue : ${module}_$l.php", 'traduire');
 			break;
 		}
@@ -164,7 +164,7 @@ 
 	foreach ($fichiers as $fichier) {
 		if (!isset($surcharges[$fichier])) {
 			$idx_lang_normal = $GLOBALS['idx_lang'];
-			$GLOBALS['idx_lang'] = $GLOBALS['idx_lang'] . '@temporaire';
+			$GLOBALS['idx_lang'] = $GLOBALS['idx_lang'].'@temporaire';
 			include($fichier);
 			$surcharges[$fichier] = $GLOBALS[$GLOBALS['idx_lang']];
 			unset($GLOBALS[$GLOBALS['idx_lang']]);
@@ -172,7 +172,7 @@ 
 		}
 		if (is_array($surcharges[$fichier])) {
 			$GLOBALS[$GLOBALS['idx_lang']] = array_merge(
-				(isset($GLOBALS[$GLOBALS['idx_lang']]) ? (array)$GLOBALS[$GLOBALS['idx_lang']] : []),
+				(isset($GLOBALS[$GLOBALS['idx_lang']]) ? (array) $GLOBALS[$GLOBALS['idx_lang']] : []),
 				$surcharges[$fichier]
 			);
 		}
@@ -250,27 +250,27 @@ 
 	} else {
 		$modules = ['spip', 'ecrire'];
 		$code = $ori;
-		$ori_complet = implode('|', $modules) . ':' . $ori;
+		$ori_complet = implode('|', $modules).':'.$ori;
 	}
 
 	$desc = new SPIP_Traductions_Description();
 
 	// parcourir tous les modules jusqu'a ce qu'on trouve
 	foreach ($modules as $module) {
-		$var = 'i18n_' . $module . '_' . $lang;
+		$var = 'i18n_'.$module.'_'.$lang;
 
 		if (empty($GLOBALS[$var])) {
 			charger_langue($lang, $module);
 			// surcharges persos -- on cherche
 			// (lang/)local_xx.php et/ou (lang/)local.php ...
-			if (!isset($local['local_' . $lang])) {
+			if (!isset($local['local_'.$lang])) {
 				// redéfinir la langue en cours pour les surcharges (chercher_langue a pu le changer)
 				$GLOBALS['idx_lang'] = $var;
 				// ... (lang/)local_xx.php
-				$local['local_' . $lang] = chercher_module_lang('local', $lang);
+				$local['local_'.$lang] = chercher_module_lang('local', $lang);
 			}
-			if ($local['local_' . $lang]) {
-				surcharger_langue($local['local_' . $lang]);
+			if ($local['local_'.$lang]) {
+				surcharger_langue($local['local_'.$lang]);
 			}
 			// ... puis (lang/)local.php
 			if (!isset($local['local'])) {
@@ -284,7 +284,7 @@ 
 		if (isset($GLOBALS[$var][$code])) {
 			$desc->code = $code;
 			$desc->module = $module;
-			$desc->langue = $GLOBALS['lang_' . $var];
+			$desc->langue = $GLOBALS['lang_'.$var];
 			$desc->texte = $GLOBALS[$var][$code];
 			break;
 		}
@@ -339,13 +339,13 @@ 
 	if (!$desc->mode and $desc->texte) {
 		// ne pas modifier 2 fois l'affichage
 		$desc->mode = 'traduction';
-		$classe = 'debug-traduction' . ($desc->module == 'ecrire' ? '-prive' : '');
+		$classe = 'debug-traduction'.($desc->module == 'ecrire' ? '-prive' : '');
 		$desc->texte = '<span '
-			. 'lang=' . $desc->langue
-			. ' class=' . $classe
-			. ' data-module=' . $desc->module
-			. ' data-code=' . $desc->code
-			. ' title=' . $modules . '(' . $desc->langue . ')>'
+			. 'lang='.$desc->langue
+			. ' class='.$classe
+			. ' data-module='.$desc->module
+			. ' data-code='.$desc->code
+			. ' title='.$modules.'('.$desc->langue.')>'
 			. $desc->texte
 			. '</span>';
 		$desc->texte = str_replace(

ecrire/inc/lang_liste.php

Spacing +5 added lines, -5 removed lines

@@ -50,7 +50,7 @@ 
 	'cpf_dom' => 'Kreyòl',
 	'cpf_hat' => 'Kreyòl (Peyi Dayiti)',
 	'cs' => 'čeština',
-	'cy' => 'Cymraeg',  # welsh, gallois
+	'cy' => 'Cymraeg', # welsh, gallois
 	'da' => 'dansk',
 	'de' => 'Deutsch',
 	'dz' => 'Bhutani',
@@ -92,7 +92,7 @@ 
 	'hi' => 'हिंदी',
 	'hr' => 'hrvatski',
 	'hu' => 'magyar',
-	'hy' => 'Հայերեն',// Arménien
+	'hy' => 'Հայերեն', // Arménien
 	'ia' => 'Interlingua',
 	'id' => 'Indonesia',
 	'ie' => 'Interlingue',
@@ -106,7 +106,7 @@ 
 	'ka' => 'ქართული',
 	'kk' => 'қазақ тілі', // Kazakh
 	'kl' => 'kalaallisut',
-	'km' => 'ភាសាខ្មែរ',// Khmer
+	'km' => 'ភាសាខ្មែរ', // Khmer
 	'kn' => 'Kannada',
 	'ko' => '한국어',
 	'kok' => 'कोंकणी',
@@ -176,8 +176,8 @@ 
 	'sh_latn' => 'srpskohrvastski',
 	'sh_cyrl' => 'Српскохрватски',
 	'si' => 'Sinhalese',
-	'sk' => 'slovenčina',  // (Slovakia)
-	'sl' => 'slovenščina',  // (Slovenia)
+	'sk' => 'slovenčina', // (Slovakia)
+	'sl' => 'slovenščina', // (Slovenia)
 	'sm' => 'Samoan',
 	'sn' => 'Shona',
 	'so' => 'Somali',

ecrire/inc/config.php

Spacing +8 added lines, -8 removed lines

@@ -335,7 +335,7 @@ 
 			if (strncmp($url, 'configurer_', 11) == 0) {
 				$deja[$url] = $b;
 			} elseif ($b['url'] == 'configurer' and preg_match(',cfg=([a-z0-9_]+),i', $b['args'], $match)) {
-				$deja['configurer_' . $match[1]] = $b;
+				$deja['configurer_'.$match[1]] = $b;
 			}
 		}
 	}
@@ -348,10 +348,10 @@ 
 
 	// trouver toutes les pages configurer_xxx de l'espace prive
 	// et construire un tableau des entrees qui ne sont pas dans $deja
-	$pages = find_all_in_path('prive/squelettes/contenu/', 'configurer_.*[.]' . _EXTENSION_SQUELETTES . '$');
+	$pages = find_all_in_path('prive/squelettes/contenu/', 'configurer_.*[.]'._EXTENSION_SQUELETTES.'$');
 
 	foreach ($pages as $page) {
-		$configurer = basename($page, '.' . _EXTENSION_SQUELETTES);
+		$configurer = basename($page, '.'._EXTENSION_SQUELETTES);
 		if (!isset($exclure[$configurer])) {
 			$liste[$configurer] = [
 				'parent' => 'bando_configuration',
@@ -371,9 +371,9 @@ 
 
 	// trouver tous les formulaires/configurer_
 	// et construire un tableau des entrees
-	$pages = find_all_in_path('formulaires/', 'configurer_.*[.]' . _EXTENSION_SQUELETTES . '$');
+	$pages = find_all_in_path('formulaires/', 'configurer_.*[.]'._EXTENSION_SQUELETTES.'$');
 	foreach ($pages as $page) {
-		$configurer = basename($page, '.' . _EXTENSION_SQUELETTES);
+		$configurer = basename($page, '.'._EXTENSION_SQUELETTES);
 		if (
 			!isset($forms[$configurer])
 			and !isset($liste[$configurer])
@@ -382,7 +382,7 @@ 
 			$liste[$configurer] = [
 				'parent' => 'bando_configuration',
 				'url' => 'configurer',
-				'args' => 'cfg=' . substr($configurer, 11),
+				'args' => 'cfg='.substr($configurer, 11),
 				'titre' => _T("configurer:{$configurer}_titre"),
 				'icone' => find_in_theme($i = "images/{$configurer}-16.png") ? $i : $icone_defaut,
 			];
@@ -414,8 +414,8 @@ 
 
 	// evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
 	// et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
-	$config = basename(substr($file, 0, -strlen('.' . _EXTENSION_SQUELETTES)));
-	spip_log('Calcul de ' . "prive/squelettes/contenu/$config");
+	$config = basename(substr($file, 0, -strlen('.'._EXTENSION_SQUELETTES)));
+	spip_log('Calcul de '."prive/squelettes/contenu/$config");
 	$fond = recuperer_fond("prive/squelettes/contenu/$config", ['exec' => $config]);
 
 	// passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...

ecrire/inc/securiser_action.php

Spacing +11 added lines, -11 removed lines

@@ -91,7 +91,7 @@ 
 	}
 
 	$arg = parametre_url($url_action, 'arg');
-	$confirm = md5("$action:$arg:" . realpath(__FILE__));
+	$confirm = md5("$action:$arg:".realpath(__FILE__));
 	if (_request('confirm_action') === $confirm) {
 		return true;
 	}
@@ -145,7 +145,7 @@ 
 		} else {
 			return generer_url_action(
 				$action,
-				'arg=' . rawurlencode($arg) . "&hash=$hash" . (!$r ? '' : "&redirect=$r"),
+				'arg='.rawurlencode($arg)."&hash=$hash".(!$r ? '' : "&redirect=$r"),
 				$mode,
 				$public
 			);
@@ -156,9 +156,9 @@ 
 	$hash = calculer_action_auteur("$action-$arg");
 	$att .= " style='margin: 0px; border: 0px'";
 	if ($redirect) {
-		$redirect = "\n\t\t<input name='redirect' type='hidden' value='" . str_replace("'", '&#39;', $redirect) . "' />";
+		$redirect = "\n\t\t<input name='redirect' type='hidden' value='".str_replace("'", '&#39;', $redirect)."' />";
 	}
-	$mode .= $redirect . "
+	$mode .= $redirect."
 <input name='hash' type='hidden' value='$hash' />
 <input name='arg' type='hidden' value='$arg' />";
 
@@ -229,7 +229,7 @@ 
  */
 function _action_auteur($action, $id_auteur, $pass, $alea) {
 	static $sha = [];
-	if (!isset($sha[$id_auteur . $pass . $alea])) {
+	if (!isset($sha[$id_auteur.$pass.$alea])) {
 		if (!isset($GLOBALS['meta'][$alea])) {
 			if (!$exec = _request('exec') or !autoriser_sans_cookie($exec)) {
 				include_spip('inc/acces');
@@ -243,12 +243,12 @@ 
 			}
 		}
 		include_spip('auth/sha256.inc');
-		$sha[$id_auteur . $pass . $alea] = spip_sha256($id_auteur . $pass . @$GLOBALS['meta'][$alea]);
+		$sha[$id_auteur.$pass.$alea] = spip_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
 	}
 	if (function_exists('sha1')) {
-		return sha1($action . $sha[$id_auteur . $pass . $alea]);
+		return sha1($action.$sha[$id_auteur.$pass.$alea]);
 	} else {
-		return md5($action . $sha[$id_auteur . $pass . $alea]);
+		return md5($action.$sha[$id_auteur.$pass.$alea]);
 	}
 }
 
@@ -332,9 +332,9 @@ 
  */
 function calculer_cle_action($action) {
 	if (function_exists('sha1')) {
-		return sha1($action . secret_du_site());
+		return sha1($action.secret_du_site());
 	} else {
-		return md5($action . secret_du_site());
+		return md5($action.secret_du_site());
 	}
 }
 
@@ -374,7 +374,7 @@ 
 	// On nettoie l’URL de tous les var_.
 	$url = nettoyer_uri_var($url);
 
-	$token = _action_auteur('previsualiser-' . $url, $id_auteur, null, $alea);
+	$token = _action_auteur('previsualiser-'.$url, $id_auteur, null, $alea);
 	return "$id_auteur-$token";
 }
 

ecrire/inc/selectionner.php

Spacing +12 added lines, -12 removed lines

@@ -54,7 +54,7 @@ 
 
 	if ($aff_racine) {
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$idom3 = $idom . '_selection';
+		$idom3 = $idom.'_selection';
 
 		$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
 
@@ -72,7 +72,7 @@ 
 			'  '
 		);
 
-		$js_func = $do . '_selection_titre';
+		$js_func = $do.'_selection_titre';
 		$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
 
 		$aff_racine = "<div class='petit-item petite-racine item'>"
@@ -113,12 +113,12 @@ 
  **/
 function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init = '') {
 
-	$idom1 = $idom . '_champ_recherche';
-	$idom2 = $idom . '_principal';
-	$idom3 = $idom . '_selection';
-	$idom4 = $idom . '_col_1';
-	$idom5 = 'img_' . $idom4;
-	$idom6 = $idom . '_fonc';
+	$idom1 = $idom.'_champ_recherche';
+	$idom2 = $idom.'_principal';
+	$idom3 = $idom.'_selection';
+	$idom4 = $idom.'_col_1';
+	$idom5 = 'img_'.$idom4;
+	$idom6 = $idom.'_fonc';
 
 	return "<div id='$idom'>"
 	. "<a id='$idom6' style='visibility: hidden;'"
@@ -128,10 +128,10 @@ 
 	. http_img_pack(
 		'loader.svg',
 		'',
-		"class='loader' style='visibility: hidden;float:" . $GLOBALS['spip_lang_right'] . "' id='$idom5'"
+		"class='loader' style='visibility: hidden;float:".$GLOBALS['spip_lang_right']."' id='$idom5'"
 	)
 	. ''
-	. "<input style='width: 10em;float:" . $GLOBALS['spip_lang_right'] . ";' type='text' class='text search' id='$idom1' placeholder='" . _T('info_rechercher') . "'"
+	. "<input style='width: 10em;float:".$GLOBALS['spip_lang_right'].";' type='text' class='text search' id='$idom1' placeholder='"._T('info_rechercher')."'"
 	// eliminer Return car il provoque la soumission (balise unique)
 	// et eliminer Tab pour la navigation au clavier
 	// ce serait encore mieux de ne le faire que s'il y a encore plusieurs
@@ -178,8 +178,8 @@ 
 
 	$liste = $id_rubrique;
 	$id_rubrique = intval($id_rubrique);
-	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
-		$liste = $id_rubrique . ",$liste";
+	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = '.$id_rubrique)) {
+		$liste = $id_rubrique.",$liste";
 	}
 
 	return explode(',', "0,$liste");