spip / SPIP

ecrire/inc/autoriser.php

Spacing +38 added lines, -38 removed lines

@@ -115,7 +115,7 @@ 
 // mais apres la fonction autoriser()
 if ($f = find_in_path('mes_fonctions.php')) {
 	global $dossier_squelettes;
-	include_once(_ROOT_CWD . $f);
+	include_once(_ROOT_CWD.$f);
 }
 
 
@@ -157,7 +157,7 @@ 
 		$qui = $GLOBALS['visiteur_session'] ?: [];
 		$qui = array_merge(['statut' => '', 'id_auteur' => 0, 'webmestre' => 'non'], $qui);
 	} elseif (is_numeric($qui)) {
-		$qui = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . $qui);
+		$qui = sql_fetsel('*', 'spip_auteurs', 'id_auteur='.$qui);
 	}
 
 	// Admins restreints, on construit ici (pas generique mais...)
@@ -167,8 +167,8 @@ 
 	}
 
 	spip_log(
-		"autoriser $faire $type $id (" . ($qui['nom'] ?? '') . ') ?',
-		'autoriser' . _LOG_DEBUG
+		"autoriser $faire $type $id (".($qui['nom'] ?? '').') ?',
+		'autoriser'._LOG_DEBUG
 	);
 
 	// passer par objet_type pour avoir les alias
@@ -184,7 +184,7 @@ 
 		(isset($GLOBALS['autoriser_exception'][$faire][$type][$id]) and autoriser_exception($faire, $type, $id, 'verifier'))
 		or (isset($GLOBALS['autoriser_exception'][$faire][$type]['*']) and autoriser_exception($faire, $type, '*', 'verifier'))
 	) {
-		spip_log("autoriser ($faire, $type, $id, " . ($qui['nom'] ?? '') . ') : OK Exception', 'autoriser' . _LOG_DEBUG);
+		spip_log("autoriser ($faire, $type, $id, ".($qui['nom'] ?? '').') : OK Exception', 'autoriser'._LOG_DEBUG);
 		return true;
 	}
 
@@ -193,18 +193,18 @@ 
 	// autoriser_faire[_dist], autoriser_defaut[_dist]
 	$fonctions = $type
 		? [
-			'autoriser_' . $type . '_' . $faire,
-			'autoriser_' . $type . '_' . $faire . '_dist',
-			'autoriser_' . $type,
-			'autoriser_' . $type . '_dist',
-			'autoriser_' . $faire,
-			'autoriser_' . $faire . '_dist',
+			'autoriser_'.$type.'_'.$faire,
+			'autoriser_'.$type.'_'.$faire.'_dist',
+			'autoriser_'.$type,
+			'autoriser_'.$type.'_dist',
+			'autoriser_'.$faire,
+			'autoriser_'.$faire.'_dist',
 			'autoriser_defaut',
 			'autoriser_defaut_dist'
 		]
 		: [
-			'autoriser_' . $faire,
-			'autoriser_' . $faire . '_dist',
+			'autoriser_'.$faire,
+			'autoriser_'.$faire.'_dist',
 			'autoriser_defaut',
 			'autoriser_defaut_dist'
 		];
@@ -217,8 +217,8 @@ 
 	}
 
 	spip_log(
-		"$f($faire, $type, $id, " . ($qui['nom'] ?? '') . ') : ' . ($a ? 'OK' : 'niet'),
-		'autoriser' . _LOG_DEBUG
+		"$f($faire, $type, $id, ".($qui['nom'] ?? '').') : '.($a ? 'OK' : 'niet'),
+		'autoriser'._LOG_DEBUG
 	);
 
 	return $a;
@@ -353,7 +353,7 @@ 
 function autoriser_previsualiser_dist(string $faire, string $type, int $id, array $qui, array $opt): bool {
 
 	// Le visiteur a-t-il un statut prevu par la config ?
-	if (strpos($GLOBALS['meta']['preview'], ',' . $qui['statut'] . ',') !== false) {
+	if (strpos($GLOBALS['meta']['preview'], ','.$qui['statut'].',') !== false) {
 		return test_previsualiser_objet_champ($type, $id, $qui, $opt);
 	}
 
@@ -410,7 +410,7 @@ 
 				} // pas de champ passe a la demande => NIET
 				$previsu = explode(',', $c['previsu']);
 				// regarder si ce statut est autorise pour l'auteur
-				if (in_array($opt[$champ] . '/auteur', $previsu)) {
+				if (in_array($opt[$champ].'/auteur', $previsu)) {
 					// retrouver l’id_auteur qui a filé un lien de prévisu éventuellement,
 					// sinon l’auteur en session
 					include_spip('inc/securiser_action');
@@ -424,12 +424,12 @@ 
 
 					if (!$id_auteur) {
 						return false;
-					} elseif (autoriser('previsualiser' . $opt[$champ], $type, 0, $id_auteur)) {
+					} elseif (autoriser('previsualiser'.$opt[$champ], $type, 0, $id_auteur)) {
 						// dans ce cas (admin en general), pas de filtrage sur ce statut
 					} elseif (
 						!sql_countsel(
 							'spip_auteurs_liens',
-							'id_auteur=' . intval($id_auteur) . ' AND objet=' . sql_quote($type) . ' AND id_objet=' . intval($id)
+							'id_auteur='.intval($id_auteur).' AND objet='.sql_quote($type).' AND id_objet='.intval($id)
 						)
 					) {
 						return false;
@@ -472,16 +472,16 @@ 
 			// multilinguisme par secteur et objet rattaché à une rubrique
 			$primary = id_table_objet($type);
 			if ($table != 'spip_rubriques') {
-				$id_rubrique = sql_getfetsel('id_rubrique', "$table", "$primary=" . intval($id));
+				$id_rubrique = sql_getfetsel('id_rubrique', "$table", "$primary=".intval($id));
 			} else {
 				$id_rubrique = $id;
 			}
-			$id_secteur = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
+			$id_secteur = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique='.intval($id_rubrique));
 			if (!$id_secteur > 0) {
 				$id_secteur = $id_rubrique;
 			}
-			$langue_secteur = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . intval($id_secteur));
-			$langue_objet = sql_getfetsel('lang', "$table", "$primary=" . intval($id));
+			$langue_secteur = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique='.intval($id_secteur));
+			$langue_objet = sql_getfetsel('lang', "$table", "$primary=".intval($id));
 			if ($langue_secteur != $langue_objet) {
 				// configuration incohérente, on laisse l'utilisateur corriger la situation
 				return true;
@@ -489,7 +489,7 @@ 
 			if ($table != 'spip_rubriques') { // le choix de la langue se fait seulement sur les rubriques
 				return false;
 			} else {
-				$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id));
+				$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique='.intval($id));
 				if ($id_parent != 0) {
 					// sous-rubriques : pas de choix de langue
 					return false;
@@ -537,7 +537,7 @@ 
 
 	if (!isset($opt['statut'])) {
 		if (isset($desc['field']['statut'])) {
-			$statut = sql_getfetsel('statut', $desc['table'], id_table_objet($type) . '=' . intval($id));
+			$statut = sql_getfetsel('statut', $desc['table'], id_table_objet($type).'='.intval($id));
 		} else {
 			$statut = 'publie';
 		} // pas de statut => publie
@@ -697,11 +697,11 @@ 
 		return false;
 	}
 
-	if (sql_countsel('spip_rubriques', 'id_parent=' . intval($id))) {
+	if (sql_countsel('spip_rubriques', 'id_parent='.intval($id))) {
 		return false;
 	}
 
-	if (sql_countsel('spip_articles', 'id_rubrique=' . intval($id) . " AND (statut<>'poubelle')")) {
+	if (sql_countsel('spip_articles', 'id_rubrique='.intval($id)." AND (statut<>'poubelle')")) {
 		return false;
 	}
 
@@ -736,7 +736,7 @@ 
 	if (!$id) {
 		return false;
 	}
-	$r = sql_fetsel('id_rubrique,statut', 'spip_articles', 'id_article=' . sql_quote($id));
+	$r = sql_fetsel('id_rubrique,statut', 'spip_articles', 'id_article='.sql_quote($id));
 
 	return
 		$r
@@ -747,7 +747,7 @@ 
 				(!isset($opt['statut']) or $opt['statut'] !== 'publie')
 				and in_array($qui['statut'], ['0minirezo', '1comite'])
 				and in_array($r['statut'], ['prop', 'prepa', 'poubelle'])
-				and auteurs_objet('article', $id, 'id_auteur=' . $qui['id_auteur'])
+				and auteurs_objet('article', $id, 'id_auteur='.$qui['id_auteur'])
 			)
 		);
 }
@@ -801,7 +801,7 @@ 
 		if (!$id) {
 			return false;
 		}
-		$statut = sql_getfetsel('statut', 'spip_articles', 'id_article=' . intval($id));
+		$statut = sql_getfetsel('statut', 'spip_articles', 'id_article='.intval($id));
 	}
 
 	return
@@ -812,7 +812,7 @@ 
 		or
 		($id
 			and $qui['id_auteur']
-			and auteurs_objet('article', $id, 'id_auteur=' . $qui['id_auteur']));
+			and auteurs_objet('article', $id, 'id_auteur='.$qui['id_auteur']));
 }
 
 
@@ -831,8 +831,8 @@ 
 function autoriser_voir_dist(string $faire, string $type, int $id, array $qui, array $opt): bool {
 	# securite, mais on aurait pas du arriver ici !
 	if (
-		function_exists($f = 'autoriser_' . $type . '_voir')
-		or function_exists($f = 'autoriser_' . $type . '_voir_dist')
+		function_exists($f = 'autoriser_'.$type.'_voir')
+		or function_exists($f = 'autoriser_'.$type.'_voir_dist')
 	) {
 		return $f($faire, $type, $id, $qui, $opt);
 	}
@@ -953,7 +953,7 @@ 
 	$n = sql_fetsel(
 		'A.id_article',
 		'spip_auteurs_liens AS L LEFT JOIN spip_articles AS A ON (L.objet=\'article\' AND L.id_objet=A.id_article)',
-		"A.statut='publie' AND L.id_auteur=" . sql_quote($id)
+		"A.statut='publie' AND L.id_auteur=".sql_quote($id)
 	);
 
 	return $n ? true : false;
@@ -1155,7 +1155,7 @@ 
 		and $r = sql_allfetsel(
 			'id_objet',
 			'spip_auteurs_liens',
-			'id_auteur=' . intval($id_auteur) . " AND objet='rubrique' AND id_objet!=0"
+			'id_auteur='.intval($id_auteur)." AND objet='rubrique' AND id_objet!=0"
 		)
 		and is_countable($r) ? count($r) : 0
 	) {
@@ -1771,8 +1771,8 @@ 
 function auteurs_objet($objet, $id_objet, $cond = '') {
 	$objet = objet_type($objet);
 	$where = [
-		'objet=' . sql_quote($objet),
-		'id_objet=' . intval($id_objet)
+		'objet='.sql_quote($objet),
+		'id_objet='.intval($id_objet)
 	];
 	if (!empty($cond)) {
 		if (is_array($cond)) {
@@ -1807,7 +1807,7 @@ 
 	return sql_allfetsel(
 		'id_auteur',
 		'spip_auteurs_liens',
-		"objet='article' AND id_objet=" . intval($id_article) . ($cond ? " AND $cond" : '')
+		"objet='article' AND id_objet=".intval($id_article).($cond ? " AND $cond" : '')
 	);
 }
 

ecrire/inc/session.php

Spacing +18 added lines, -18 removed lines

@@ -82,13 +82,13 @@ 
 	spip_log("supprimer sessions auteur $id_auteur", 'session');
 	if ($toutes or $id_auteur !== $GLOBALS['visiteur_session']['id_auteur']) {
 		if ($dir = opendir(_DIR_SESSIONS)) {
-			$t = $_SERVER['REQUEST_TIME']  - (4 * _RENOUVELLE_ALEA); // 48h par defaut
-			$t_short = $_SERVER['REQUEST_TIME']  - max(_RENOUVELLE_ALEA / 4, 3 * 3600); // 3h par defaut
+			$t = $_SERVER['REQUEST_TIME'] - (4 * _RENOUVELLE_ALEA); // 48h par defaut
+			$t_short = $_SERVER['REQUEST_TIME'] - max(_RENOUVELLE_ALEA / 4, 3 * 3600); // 3h par defaut
 			$t = time() - (4 * _RENOUVELLE_ALEA);
 			while (($f = readdir($dir)) !== false) {
 				$nb_files++;
 				if (preg_match(',^[^\d-]*(-?\d+)_\w{32}\.php[3]?$,', $f, $regs)) {
-					$f = _DIR_SESSIONS . $f;
+					$f = _DIR_SESSIONS.$f;
 					if (($actives and $regs[1] == $id_auteur) or ($t > filemtime($f))) {
 						spip_unlink($f);
 					}
@@ -171,9 +171,9 @@ 
 
 	if (
 		!isset($_COOKIE['spip_session'])
-		or !preg_match(',^' . $id_auteur . '_,', $_COOKIE['spip_session'])
+		or !preg_match(',^'.$id_auteur.'_,', $_COOKIE['spip_session'])
 	) {
-		$_COOKIE['spip_session'] = $id_auteur . '_' . md5(uniqid(random_int(0, mt_getrandmax()), true));
+		$_COOKIE['spip_session'] = $id_auteur.'_'.md5(uniqid(random_int(0, mt_getrandmax()), true));
 	}
 
 	// Maintenant on sait qu'on a des choses à écrire
@@ -206,7 +206,7 @@ 
 	} else {
 		$fichier_session = fichier_session('alea_ephemere');
 		if (!ecrire_fichier_session($fichier_session, $auteur)) {
-			spip_log('Echec ecriture fichier session ' . $fichier_session, 'session' . _LOG_HS);
+			spip_log('Echec ecriture fichier session '.$fichier_session, 'session'._LOG_HS);
 			include_spip('inc/minipres');
 			echo minipres();
 			exit;
@@ -229,7 +229,7 @@ 
 		include_spip('inc/autoriser');
 	}
 	if (autoriser('ecrire', '', '', $auteur) and _DUREE_COOKIE_ADMIN) {
-		spip_setcookie('spip_admin', '@' . ($auteur['email'] ?: $auteur['login']), [
+		spip_setcookie('spip_admin', '@'.($auteur['email'] ?: $auteur['login']), [
 			'expires' => time() + max(_DUREE_COOKIE_ADMIN, $duree)
 		]);
 	} // sinon le supprimer ...
@@ -270,7 +270,7 @@ 
 			$coef = 20;
 		}
 	}
-	return (int)(_RENOUVELLE_ALEA * $coef);
+	return (int) (_RENOUVELLE_ALEA * $coef);
 }
 
 /**
@@ -318,7 +318,7 @@ 
 
 			// Renouveler la session avec l'alea courant
 			include($fichier_session);
-			spip_log('renouvelle session ' . $GLOBALS['visiteur_session']['id_auteur'], 'session');
+			spip_log('renouvelle session '.$GLOBALS['visiteur_session']['id_auteur'], 'session');
 			spip_unlink($fichier_session);
 			ajouter_session($GLOBALS['visiteur_session']);
 		}
@@ -350,7 +350,7 @@ 
 		}
 	} else {
 		if ($change) {
-			spip_log("rejoue session $fichier_session " . $_COOKIE['spip_session'], 'session');
+			spip_log("rejoue session $fichier_session ".$_COOKIE['spip_session'], 'session');
 			if ($fichier_session) {
 				spip_unlink($fichier_session);
 			}
@@ -567,7 +567,7 @@ 
 	}
 
 	// liste des sessions
-	$sessions = preg_files(_DIR_SESSIONS, '/' . $id_auteur . '_.*\.php$');
+	$sessions = preg_files(_DIR_SESSIONS, '/'.$id_auteur.'_.*\.php$');
 
 	// si on en a plus que la limite, supprimer les plus vieilles
 	// si ce ne sont pas des sessions anonymes car elles sont alors chacune differentes
@@ -648,12 +648,12 @@ 
 	$auteur = preparer_ecriture_session($auteur);
 
 	// enregistrer les autres donnees du visiteur
-	$texte = '<' . "?php\n";
+	$texte = '<'."?php\n";
 	foreach ($auteur as $var => $val) {
-		$texte .= '$GLOBALS[\'visiteur_session\'][' . var_export($var, true) . '] = '
-			. var_export($val, true) . ";\n";
+		$texte .= '$GLOBALS[\'visiteur_session\']['.var_export($var, true).'] = '
+			. var_export($val, true).";\n";
 	}
-	$texte .= '?' . ">\n";
+	$texte .= '?'.">\n";
 
 	return ecrire_fichier($fichier, $texte);
 }
@@ -683,7 +683,7 @@ 
 		$repertoire = sous_repertoire(_DIR_SESSIONS, '', false, $tantpis);
 		$c = $_COOKIE['spip_session'];
 
-		return $repertoire . intval($c) . '_' . md5($c . ' ' . $GLOBALS['meta'][$alea]) . '.php';
+		return $repertoire.intval($c).'_'.md5($c.' '.$GLOBALS['meta'][$alea]).'.php';
 	}
 }
 
@@ -701,7 +701,7 @@ 
  * @return string
  */
 function rejouer_session() {
-	return '<img src="' . generer_url_action('cookie', 'change_session=oui', true) . '" width="0" height="0" alt="" />';
+	return '<img src="'.generer_url_action('cookie', 'change_session=oui', true).'" width="0" height="0" alt="" />';
 }
 
 
@@ -716,7 +716,7 @@ 
 		return $res;
 	}
 
-	return $res = md5($GLOBALS['ip'] . ($_SERVER['HTTP_USER_AGENT'] ?? ''));
+	return $res = md5($GLOBALS['ip'].($_SERVER['HTTP_USER_AGENT'] ?? ''));
 }
 
 

ecrire/base/delete_all.php

Spacing +1 added lines, -1 removed lines

@@ -51,5 +51,5 @@
 	}
 	$d = is_countable($delete) ? count($delete) : 0;
 	$r = count($res);
-	spip_log("Tables detruites: $r sur $d: " . join(', ', $res), _LOG_INFO_IMPORTANTE);
+	spip_log("Tables detruites: $r sur $d: ".join(', ', $res), _LOG_INFO_IMPORTANTE);
 }

ecrire/base/objets.php

Spacing +23 added lines, -23 removed lines

@@ -703,7 +703,7 @@ 
 		$infos['url_voir'] = $infos['type'];
 	}
 	if (!isset($infos['url_edit'])) {
-		$infos['url_edit'] = $infos['url_voir'] . ($infos['editable'] ? '_edit' : '');
+		$infos['url_edit'] = $infos['url_voir'].($infos['editable'] ? '_edit' : '');
 	}
 	if (!isset($infos['icone_objet'])) {
 		$infos['icone_objet'] = $infos['type'];
@@ -715,48 +715,48 @@ 
 		$infos['texte_retour'] = 'icone_retour';
 	}
 	if (!isset($infos['texte_modifier'])) {
-		$infos['texte_modifier'] = $infos['type'] . ':' . 'icone_modifier_' . $infos['type'];
+		$infos['texte_modifier'] = $infos['type'].':'.'icone_modifier_'.$infos['type'];
 	}
 	if (!isset($infos['texte_creer'])) {
-		$infos['texte_creer'] = $infos['type'] . ':' . 'icone_creer_' . $infos['type'];
+		$infos['texte_creer'] = $infos['type'].':'.'icone_creer_'.$infos['type'];
 	}
 	if (!isset($infos['texte_creer_associer'])) {
-		$infos['texte_creer_associer'] = $infos['type'] . ':' . 'texte_creer_associer_' . $infos['type'];
+		$infos['texte_creer_associer'] = $infos['type'].':'.'texte_creer_associer_'.$infos['type'];
 	}
 	if (!isset($infos['texte_ajouter'])) {
 		// Ajouter un X
-		$infos['texte_ajouter'] = $infos['type'] . ':' . 'texte_ajouter_' . $infos['type'];
+		$infos['texte_ajouter'] = $infos['type'].':'.'texte_ajouter_'.$infos['type'];
 	}
 	if (!isset($infos['texte_objets'])) {
-		$infos['texte_objets'] = $infos['type'] . ':' . 'titre_' . $infos['table_objet'];
+		$infos['texte_objets'] = $infos['type'].':'.'titre_'.$infos['table_objet'];
 	}
 	if (!isset($infos['texte_objet'])) {
-		$infos['texte_objet'] = $infos['type'] . ':' . 'titre_' . $infos['type'];
+		$infos['texte_objet'] = $infos['type'].':'.'titre_'.$infos['type'];
 	}
 	if (!isset($infos['texte_logo_objet'])) {
 		// objet:titre_logo_objet "Logo de ce X"
-		$infos['texte_logo_objet'] = $infos['type'] . ':' . 'titre_logo_' . $infos['type'];
+		$infos['texte_logo_objet'] = $infos['type'].':'.'titre_logo_'.$infos['type'];
 	}
 	if (!isset($infos['texte_langue_objet'])) {
 		// objet:texte_langue_objet "Langue de ce X"
-		$infos['texte_langue_objet'] = $infos['type'] . ':' . 'titre_langue_' . $infos['type'];
+		$infos['texte_langue_objet'] = $infos['type'].':'.'titre_langue_'.$infos['type'];
 	}
 	if (!isset($infos['texte_definir_comme_traduction_objet'])) {
 		// "Ce X est une traduction du X numéro :"
-		$infos['texte_definir_comme_traduction_objet'] = $infos['type'] . ':' . 'texte_definir_comme_traduction_' . $infos['type'];
+		$infos['texte_definir_comme_traduction_objet'] = $infos['type'].':'.'texte_definir_comme_traduction_'.$infos['type'];
 	}
 
 	// objet:info_aucun_objet
 	if (!isset($infos['info_aucun_objet'])) {
-		$infos['info_aucun_objet'] = $infos['type'] . ':' . 'info_aucun_' . $infos['type'];
+		$infos['info_aucun_objet'] = $infos['type'].':'.'info_aucun_'.$infos['type'];
 	}
 	// objet:info_1_objet
 	if (!isset($infos['info_1_objet'])) {
-		$infos['info_1_objet'] = $infos['type'] . ':' . 'info_1_' . $infos['type'];
+		$infos['info_1_objet'] = $infos['type'].':'.'info_1_'.$infos['type'];
 	}
 	// objet:info_nb_objets
 	if (!isset($infos['info_nb_objets'])) {
-		$infos['info_nb_objets'] = $infos['type'] . ':' . 'info_nb_' . $infos['table_objet'];
+		$infos['info_nb_objets'] = $infos['type'].':'.'info_nb_'.$infos['table_objet'];
 	}
 
 	if (!isset($infos['champs_editables'])) {
@@ -956,7 +956,7 @@ 
 		}
 		$ts = sql_alltable(null, $serveur); // toutes les tables "spip_" (ou prefixe perso)
 		$connexion = $GLOBALS['connexions'][$serveur ?: 0];
-		$spip = $connexion['prefixe'] . '_';
+		$spip = $connexion['prefixe'].'_';
 		foreach ($ts as $t) {
 			$t = substr($t, strlen($spip));
 			$tables[$serveur]["spip_$t"] = $t;
@@ -1025,10 +1025,10 @@ 
 	if ($serveur !== false) {
 		$t = lister_tables_spip($serveur);
 		$trouver_table = charger_fonction('trouver_table', 'base');
-		$typetrim = rtrim($type, 's') . 's';
+		$typetrim = rtrim($type, 's').'s';
 		if (
 			(isset($t[$typetrim]) or in_array($typetrim, $t))
-			and ($desc = $trouver_table(rtrim($type, 's') . 's', $serveur))
+			and ($desc = $trouver_table(rtrim($type, 's').'s', $serveur))
 		) {
 			return $desc['id_table'];
 		} elseif (
@@ -1038,11 +1038,11 @@ 
 			return $desc['id_table'];
 		}
 
-		spip_log('table_objet(' . $type . ') calculee sans verification');
+		spip_log('table_objet('.$type.') calculee sans verification');
 		#spip_log(debug_backtrace(),'db');
 	}
 
-	return rtrim($type, 's') . 's'; # cas historique ne devant plus servir, sauf si $serveur=false
+	return rtrim($type, 's').'s'; # cas historique ne devant plus servir, sauf si $serveur=false
 }
 
 /**
@@ -1226,7 +1226,7 @@ 
 function objet_test_si_publie($objet, $id_objet, $serveur = '') {
 	// voir si une fonction est definie pour faire le boulot
 	// elle a la priorite dans ce cas
-	if ($f = charger_fonction($objet . '_test_si_publie', 'base', true)) {
+	if ($f = charger_fonction($objet.'_test_si_publie', 'base', true)) {
 		return $f($objet, $id_objet, $serveur);
 	}
 
@@ -1248,7 +1248,7 @@ 
 		$boucle->sql_serveur = $serveur;
 		$boucle->select[] = $id_table_objet;
 		$boucle->from[$table_objet] = table_objet_sql($objet, $serveur);
-		$boucle->where[] = $id_table . '.' . $id_table_objet . '=' . intval($id_objet);
+		$boucle->where[] = $id_table.'.'.$id_table_objet.'='.intval($id_objet);
 
 		$boucle->descr['nom'] = 'objet_test_si_publie'; // eviter notice php
 		$boucle->descr['sourcefile'] = 'internal';
@@ -1375,7 +1375,7 @@ 
 						? "{$parent_methode['source_champ']} = $id_objet"
 						: "${cle_objet} = $id_objet";
 					if (isset($parent_methode['source_champ_type'])) {
-						$where[] = "{$parent_methode['source_champ_type']} = " . sql_quote($objet);
+						$where[] = "{$parent_methode['source_champ_type']} = ".sql_quote($objet);
 					}
 					// -- Condition supplémentaire sur la détection du parent
 					if (isset($parent_methode['table_condition'])) {
@@ -1499,11 +1499,11 @@ 
 			$where = [];
 			// -- L'identifiant du parent
 			if (isset($_methode_parent['champ'])) {
-				$where[] = $_methode_parent['champ'] . ' = ' . $id_objet;
+				$where[] = $_methode_parent['champ'].' = '.$id_objet;
 			}
 			// -- Si le parent est variable
 			if (isset($_methode_parent['champ_type'])) {
-				$where[] = $_methode_parent['champ_type'] . ' = ' . sql_quote($objet);
+				$where[] = $_methode_parent['champ_type'].' = '.sql_quote($objet);
 			}
 
 			// On détermine la table, le champ id des enfants et on complète éventuellement les conditions

ecrire/base/abstract_sql.php

Spacing +11 added lines, -11 removed lines

@@ -53,10 +53,10 @@ 
 
 	if ($compil_info) {
 		$contexte_compil = [
-			$trace[0]['file'],// sourcefile
+			$trace[0]['file'], // sourcefile
 			'', //nom
-			(isset($trace[1]) ? $trace[1]['function'] . "(){\n" : '')
-			. $trace[0]['function'] . '();'
+			(isset($trace[1]) ? $trace[1]['function']."(){\n" : '')
+			. $trace[0]['function'].'();'
 			. (isset($trace[1]) ? "\n}" : ''), //id_boucle
 			$trace[0]['line'], // ligne
 			$GLOBALS['spip_lang'], // lang
@@ -65,7 +65,7 @@ 
 		return $contexte_compil;
 	}
 
-	$message = count($trace) ? $trace[0]['file'] . ' L' . $trace[0]['line'] : '';
+	$message = count($trace) ? $trace[0]['file'].' L'.$trace[0]['line'] : '';
 	$f = [];
 	while (count($trace) and $t = array_shift($trace)) {
 		if (in_array($t['function'], ['include_once', 'include_spip', 'find_in_path'])) {
@@ -74,7 +74,7 @@ 
 		$f[] = $t['function'];
 	}
 	if (count($f)) {
-		$message .= ' [' . implode('(),', $f) . '()]';
+		$message .= ' ['.implode('(),', $f).'()]';
 	}
 
 	return $message;
@@ -267,7 +267,7 @@ 
 		// le debug, c'est pour ce qui a ete produit par le compilateur
 		if (isset($GLOBALS['debug']['aucasou'])) {
 			[$table, $id, ] = $GLOBALS['debug']['aucasou'];
-			$nom = $GLOBALS['debug_objets']['courant'] . $id;
+			$nom = $GLOBALS['debug_objets']['courant'].$id;
 			$GLOBALS['debug_objets']['requete'][$nom] = $query;
 		}
 		$res = $f($select, $from, $where, $groupby, $orderby, $limit, $having, $serveur, true);
@@ -1144,7 +1144,7 @@ 
 	// la globale n'est remplie qu'apres l'appel de sql_serveur.
 	if ($spip == null) {
 		$connexion = $GLOBALS['connexions'][$serveur ? strtolower($serveur) : 0];
-		$spip = $connexion['prefixe'] . '\_%';
+		$spip = $connexion['prefixe'].'\_%';
 	}
 
 	return $f($spip, $serveur, $option !== false);
@@ -2302,9 +2302,9 @@ 
 		$jour = 0;
 	}
 
-	return sprintf('%04u', $annee) . '-' . sprintf('%02u', $mois) . '-'
-	. sprintf('%02u', $jour) . ' ' . sprintf('%02u', $h) . ':'
-	. sprintf('%02u', $m) . ':' . sprintf('%02u', $s);
+	return sprintf('%04u', $annee).'-'.sprintf('%02u', $mois).'-'
+	. sprintf('%02u', $jour).' '.sprintf('%02u', $h).':'
+	. sprintf('%02u', $m).':'.sprintf('%02u', $s);
 }
 
 
@@ -2366,7 +2366,7 @@ 
  */
 function prefixer_table_spip($table, $prefixe) {
 	if ($prefixe) {
-		$table = preg_replace('/^spip_/', $prefixe . '_', $table);
+		$table = preg_replace('/^spip_/', $prefixe.'_', $table);
 	}
 	return $table;
 }

ecrire/base/connect_sql.php

Spacing +12 added lines, -12 removed lines

@@ -18,7 +18,7 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
-require_once _ROOT_RESTREINT . 'base/objets.php';
+require_once _ROOT_RESTREINT.'base/objets.php';
 
 
 /**
@@ -59,7 +59,7 @@ 
 		$f = (!preg_match('/^[\w\.]*$/', $serveur))
 			? '' // nom de serveur mal ecrit
 			: ($serveur ?
-				(_DIR_CONNECT . $serveur . '.php') // serveur externe
+				(_DIR_CONNECT.$serveur.'.php') // serveur externe
 				: (_FILE_CONNECT ?: ($install ? _FILE_CONNECT_TMP // init du serveur principal
 						: ''))); // installation pas faite
 
@@ -101,9 +101,9 @@ 
 	// chargement de la version du jeu de fonctions
 	// si pas dans le fichier par defaut
 	$type = $GLOBALS['db_ok']['type'];
-	$jeu = 'spip_' . $type . '_functions_' . $version;
+	$jeu = 'spip_'.$type.'_functions_'.$version;
 	if (!isset($GLOBALS[$jeu])) {
-		if (!find_in_path($type . '_' . $version . '.php', 'req/', true)) {
+		if (!find_in_path($type.'_'.$version.'.php', 'req/', true)) {
 			spip_log("spip_connect: serveur $index version '$version' non defini pour '$type'", _LOG_HS);
 
 			// ne plus reessayer
@@ -165,9 +165,9 @@ 
 	$connexion = spip_connect($serveur);
 	$e = sql_errno($serveur);
 	$t = ($connexion['type'] ?? 'sql');
-	$m = "Erreur $e de $t: " . sql_error($serveur) . "\nin " . sql_error_backtrace() . "\n" . trim($connexion['last']);
-	$f = $t . $serveur;
-	spip_log($m, $f . '.' . _LOG_ERREUR);
+	$m = "Erreur $e de $t: ".sql_error($serveur)."\nin ".sql_error_backtrace()."\n".trim($connexion['last']);
+	$f = $t.$serveur;
+	spip_log($m, $f.'.'._LOG_ERREUR);
 }
 
 /**
@@ -253,7 +253,7 @@ 
 	// si en cours d'installation ou si db=@test@ on ne pose rien
 	// car c'est un test de connexion
 	if (!defined('_ECRIRE_INSTALL') and $db !== '@test@') {
-		$f = _DIR_TMP . $type . '.' . substr(md5($host . $port . $db), 0, 8) . '.out';
+		$f = _DIR_TMP.$type.'.'.substr(md5($host.$port.$db), 0, 8).'.out';
 	} elseif ($db == '@test@') {
 		$db = '';
 	}
@@ -292,7 +292,7 @@ 
 	// En cas d'indisponibilite du serveur, eviter de le bombarder
 	if ($f) {
 		@touch($f);
-		spip_log("Echec connexion serveur $type : host[$host] port[$port] login[$login] base[$db]", $type . '.' . _LOG_HS);
+		spip_log("Echec connexion serveur $type : host[$host] port[$port] login[$login] base[$db]", $type.'.'._LOG_HS);
 	}
 }
 
@@ -389,11 +389,11 @@ 
 	} elseif (is_array($a)) {
 		return join(',', array_map('_q', $a));
 	} elseif (is_scalar($a)) {
-		return ("'" . addslashes($a) . "'");
+		return ("'".addslashes($a)."'");
 	} elseif ($a === null) {
 		return "''";
 	}
-	throw new \RuntimeException("Can’t use _q with " . gettype($a));
+	throw new \RuntimeException("Can’t use _q with ".gettype($a));
 }
 
 /**
@@ -453,7 +453,7 @@ 
 				break;
 			default:
 				$replace = range(1, count($textes));
-				$replace = '%' . implode('$s,%', $replace) . '$s';
+				$replace = '%'.implode('$s,%', $replace).'$s';
 				$replace = explode(',', $replace);
 				break;
 		}

ecrire/base/dump.php

Spacing +35 added lines, -35 removed lines

@@ -38,7 +38,7 @@ 
  * @return string
  **/
 function base_dump_meta_name($rub) {
-	return $meta = "status_dump_{$rub}_" . abs($GLOBALS['visiteur_session']['id_auteur']);
+	return $meta = "status_dump_{$rub}_".abs($GLOBALS['visiteur_session']['id_auteur']);
 }
 
 /**
@@ -84,7 +84,7 @@ 
 	$connexion = $GLOBALS['connexions'][$serveur ?: 0];
 	$prefixe = $connexion['prefixe'];
 
-	$p = '/^' . $prefixe . '/';
+	$p = '/^'.$prefixe.'/';
 	$res = $tables;
 	foreach (sql_alltable(null, $serveur) as $t) {
 		if (preg_match($p, $t)) {
@@ -349,8 +349,8 @@ 
 	$trouver_table = charger_fonction('trouver_table', 'base');
 
 	spip_log(
-		'Vider ' . count($tables) . " tables sur serveur '$serveur' : " . join(', ', $tables),
-		'base.' . _LOG_INFO_IMPORTANTE
+		'Vider '.count($tables)." tables sur serveur '$serveur' : ".join(', ', $tables),
+		'base.'._LOG_INFO_IMPORTANTE
 	);
 	foreach ($tables as $table) {
 		if (!in_array($table, $exclure_tables)) {
@@ -390,15 +390,15 @@ 
 	// s'asurer qu'on a pas deja fait la manip !
 	if ($GLOBALS['visiteur_session']['id_auteur'] > 0 and sql_countsel('spip_auteurs', 'id_auteur>0')) {
 		spip_log(
-			'Conserver copieur dans id_auteur=' . $GLOBALS['visiteur_session']['id_auteur'] . " pour le serveur '$serveur'",
-			'dump.' . _LOG_INFO_IMPORTANTE
+			'Conserver copieur dans id_auteur='.$GLOBALS['visiteur_session']['id_auteur']." pour le serveur '$serveur'",
+			'dump.'._LOG_INFO_IMPORTANTE
 		);
 		sql_delete('spip_auteurs', 'id_auteur<0', $serveur);
 		if ($move) {
 			sql_updateq(
 				'spip_auteurs',
 				['id_auteur' => -$GLOBALS['visiteur_session']['id_auteur']],
-				'id_auteur=' . intval($GLOBALS['visiteur_session']['id_auteur']),
+				'id_auteur='.intval($GLOBALS['visiteur_session']['id_auteur']),
 				[],
 				$serveur
 			);
@@ -406,7 +406,7 @@ 
 			$row = sql_fetsel(
 				'*',
 				'spip_auteurs',
-				'id_auteur=' . $GLOBALS['visiteur_session']['id_auteur'],
+				'id_auteur='.$GLOBALS['visiteur_session']['id_auteur'],
 				'',
 				'',
 				'',
@@ -434,17 +434,17 @@ 
 	// rien a faire si ce n'est pas le serveur principal !
 	if ($serveur == '') {
 		if (sql_countsel('spip_auteurs', 'id_auteur>0')) {
-			spip_log("Detruire copieur id_auteur<0 pour le serveur '$serveur'", 'dump.' . _LOG_INFO_IMPORTANTE);
+			spip_log("Detruire copieur id_auteur<0 pour le serveur '$serveur'", 'dump.'._LOG_INFO_IMPORTANTE);
 			sql_delete('spip_auteurs', 'id_auteur<0', $serveur);
 		} else {
 			spip_log(
 				"Restaurer copieur id_auteur<0 pour le serveur '$serveur' (aucun autre auteur en base)",
-				'dump.' . _LOG_INFO_IMPORTANTE
+				'dump.'._LOG_INFO_IMPORTANTE
 			);
 			sql_update('spip_auteurs', ['id_auteur' => '-id_auteur'], 'id_auteur<0');
 		}
 	} else {
-		spip_log("Pas de destruction copieur sur serveur '$serveur'", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Pas de destruction copieur sur serveur '$serveur'", 'dump.'._LOG_INFO_IMPORTANTE);
 	}
 }
 
@@ -478,19 +478,19 @@ 
 			}
 		} else {
 			sql_drop_table($table, '', $serveur_dest);
-			spip_log("drop table '$table' sur serveur '$serveur_dest'", 'dump.' . _LOG_INFO_IMPORTANTE);
+			spip_log("drop table '$table' sur serveur '$serveur_dest'", 'dump.'._LOG_INFO_IMPORTANTE);
 		}
 		$desc_dest = false;
 	}
 	// si la table n'existe pas dans la destination, la creer a l'identique !
 	if (!$desc_dest) {
-		spip_log("creation '$table' sur serveur '$serveur_dest'", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("creation '$table' sur serveur '$serveur_dest'", 'dump.'._LOG_INFO_IMPORTANTE);
 		include_spip('base/create');
 		creer_ou_upgrader_table($table, $desc, 'auto', $upgrade, $serveur_dest);
 		$desc_dest = sql_showtable($table, true, $serveur_dest);
 	}
 	if (!$desc_dest) {
-		spip_log("Erreur creation '$table' sur serveur '$serveur_dest'" . var_export($desc, 1), 'dump.' . _LOG_ERREUR);
+		spip_log("Erreur creation '$table' sur serveur '$serveur_dest'".var_export($desc, 1), 'dump.'._LOG_ERREUR);
 	}
 
 	return $desc_dest;
@@ -551,17 +551,17 @@ 
 	$data_pool = $options['data_pool'] ?? 50 * 1024;
 
 	spip_log(
-		'Copier ' . count($tables) . " tables de '$serveur_source' vers '$serveur_dest'",
-		'dump.' . _LOG_INFO_IMPORTANTE
+		'Copier '.count($tables)." tables de '$serveur_source' vers '$serveur_dest'",
+		'dump.'._LOG_INFO_IMPORTANTE
 	);
 
 	if (!$inserer_copie = charger_fonction($fonction_base_inserer, $racine_fonctions, true)) {
-		spip_log("Fonction '{$racine_fonctions}_$fonction_base_inserer' inconnue. Abandon", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Fonction '{$racine_fonctions}_$fonction_base_inserer' inconnue. Abandon", 'dump.'._LOG_INFO_IMPORTANTE);
 
 		return true; // echec mais on a fini, donc true
 	}
 	if (!$preparer_table_dest = charger_fonction('preparer_table_dest', $racine_fonctions, true)) {
-		spip_log("Fonction '{$racine_fonctions}_$preparer_table_dest' inconnue. Abandon", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Fonction '{$racine_fonctions}_$preparer_table_dest' inconnue. Abandon", 'dump.'._LOG_INFO_IMPORTANTE);
 
 		return true; // echec mais on a fini, donc true
 	}
@@ -589,7 +589,7 @@ 
 		) {
 			spip_log(
 				"Fonction '{$racine_fonctions}_vider_tables_destination_copie' inconnue. Abandon",
-				'dump.' . _LOG_INFO_IMPORTANTE
+				'dump.'._LOG_INFO_IMPORTANTE
 			);
 
 			return true; // echec mais on a fini, donc true
@@ -608,7 +608,7 @@ 
 		$tables = array_diff($tables, ['spip_meta']);
 		$tables[] = 'spip_meta';
 	}
-	spip_log('Tables a copier :' . implode(', ', $tables), 'dump.' . _LOG_INFO);
+	spip_log('Tables a copier :'.implode(', ', $tables), 'dump.'._LOG_INFO);
 
 	$trouver_table = charger_fonction('trouver_table', 'base');
 
@@ -672,7 +672,7 @@ 
 						// mais si ca renvoie false c'est une erreur fatale => abandon
 						if ($inserer_copie($table, $rows, $desc_dest, $serveur_dest) === false) {
 							// forcer la sortie, charge a l'appelant de gerer l'echec
-							spip_log("Erreur fatale dans $inserer_copie table $table", 'dump' . _LOG_ERREUR);
+							spip_log("Erreur fatale dans $inserer_copie table $table", 'dump'._LOG_ERREUR);
 							$status['errors'][] = "Erreur fatale  lors de la copie de la table $table";
 							ecrire_fichier($status_file, serialize($status));
 
@@ -687,7 +687,7 @@ 
 					if ($n == $status['tables_copiees'][$table]) {
 						break;
 					}
-					spip_log("recopie $table " . $status['tables_copiees'][$table], 'dump.' . _LOG_INFO_IMPORTANTE);
+					spip_log("recopie $table ".$status['tables_copiees'][$table], 'dump.'._LOG_INFO_IMPORTANTE);
 					if ($callback_progression) {
 						$callback_progression($status['tables_copiees'][$table], 0, $table);
 					}
@@ -698,30 +698,30 @@ 
 				}
 				if ($drop_source) {
 					sql_drop_table($table, '', $serveur_source);
-					spip_log("drop $table sur serveur source '$serveur_source'", 'dump.' . _LOG_INFO_IMPORTANTE);
+					spip_log("drop $table sur serveur source '$serveur_source'", 'dump.'._LOG_INFO_IMPORTANTE);
 				}
 				$status['tables_copiees'][$table] = ($status['tables_copiees'][$table] ? -$status['tables_copiees'][$table] : 'zero');
 				ecrire_fichier($status_file, serialize($status));
-				spip_log('tables_recopiees ' . implode(',', array_keys($status['tables_copiees'])), 'dump.' . _LOG_INFO);
+				spip_log('tables_recopiees '.implode(',', array_keys($status['tables_copiees'])), 'dump.'._LOG_INFO);
 				if ($callback_progression) {
 					$callback_progression($status['tables_copiees'][$table], $status['tables_copiees'][$table], $table);
 				}
 			} else {
 				if ($status['tables_copiees'][$table] < 0) {
-					spip_log("Table $table deja copiee : " . $status['tables_copiees'][$table], 'dump.' . _LOG_INFO);
+					spip_log("Table $table deja copiee : ".$status['tables_copiees'][$table], 'dump.'._LOG_INFO);
 				}
 				if ($callback_progression) {
 					$callback_progression(
 						0,
 						$status['tables_copiees'][$table],
-						"$table" . ((is_numeric($status['tables_copiees'][$table]) and $status['tables_copiees'][$table] >= 0) ? '[Echec]' : '')
+						"$table".((is_numeric($status['tables_copiees'][$table]) and $status['tables_copiees'][$table] >= 0) ? '[Echec]' : '')
 					);
 				}
 			}
 		} else {
 			$status['errors'][] = "Impossible de lire la description de la table $table";
 			ecrire_fichier($status_file, serialize($status));
-			spip_log("Impossible de lire la description de la table $table", 'dump.' . _LOG_ERREUR);
+			spip_log("Impossible de lire la description de la table $table", 'dump.'._LOG_ERREUR);
 		}
 	}
 
@@ -729,17 +729,17 @@ 
 	// abandonner
 	if ((is_countable($status['tables_copiees']) ? count($status['tables_copiees']) : 0) < count($tables)) {
 		spip_log(
-			'Nombre de tables copiees incorrect : ' . (is_countable($status['tables_copiees']) ? count($status['tables_copiees']) : 0) . '/' . count($tables),
-			'dump.' . _LOG_ERREUR
+			'Nombre de tables copiees incorrect : '.(is_countable($status['tables_copiees']) ? count($status['tables_copiees']) : 0).'/'.count($tables),
+			'dump.'._LOG_ERREUR
 		);
-		$status['errors'][] = 'Nombre de tables copiees incorrect : ' . (is_countable($status['tables_copiees']) ? count($status['tables_copiees']) : 0) . '/' . count($tables);
+		$status['errors'][] = 'Nombre de tables copiees incorrect : '.(is_countable($status['tables_copiees']) ? count($status['tables_copiees']) : 0).'/'.count($tables);
 		ecrire_fichier($status_file, serialize($status));
 	}
 
 	if ($detruire_copieur_si_besoin = charger_fonction('detruire_copieur_si_besoin', $racine_fonctions, true)) {
 		$detruire_copieur_si_besoin($serveur_dest);
 	} else {
-		spip_log("Fonction '{$racine_fonctions}_detruire_copieur_si_besoin' inconnue.", 'dump.' . _LOG_INFO_IMPORTANTE);
+		spip_log("Fonction '{$racine_fonctions}_detruire_copieur_si_besoin' inconnue.", 'dump.'._LOG_INFO_IMPORTANTE);
 	}
 
 	// OK, copie complete
@@ -764,8 +764,8 @@ 
 	$nb = sql_countsel($table, '', '', '', $serveur_dest);
 	if ($nb - $nb1 < count($rows)) {
 		spip_log(
-			'base_inserer_copie : ' . ($nb - $nb1) . ' insertions au lieu de ' . count($rows) . '. On retente 1 par 1',
-			'dump' . _LOG_INFO_IMPORTANTE
+			'base_inserer_copie : '.($nb - $nb1).' insertions au lieu de '.count($rows).'. On retente 1 par 1',
+			'dump'._LOG_INFO_IMPORTANTE
 		);
 		foreach ($rows as $row) {
 			// si l'enregistrement est deja en base, ca fera un echec ou un doublon
@@ -776,8 +776,8 @@ 
 		$nb = sql_countsel($table, '', '', '', $serveur_dest);
 		if ($nb - $nb1 < count($rows)) {
 			spip_log(
-				'base_inserer_copie : ' . ($nb - $nb1) . ' insertions au lieu de ' . count($rows) . ' apres insertion 1 par 1',
-				'dump' . _LOG_ERREUR
+				'base_inserer_copie : '.($nb - $nb1).' insertions au lieu de '.count($rows).' apres insertion 1 par 1',
+				'dump'._LOG_ERREUR
 			);
 			$r = false;
 		}

ecrire/install/etape_2.php

Spacing +18 added lines, -18 removed lines

@@ -50,7 +50,7 @@ 
 	$GLOBALS['connexions'][$server_db] = $link;
 
 	$GLOBALS['connexions'][$server_db][$GLOBALS['spip_sql_version']]
-		= $GLOBALS['spip_' . $server_db . '_functions_' . $GLOBALS['spip_sql_version']];
+		= $GLOBALS['spip_'.$server_db.'_functions_'.$GLOBALS['spip_sql_version']];
 
 	echo install_debut_html();
 
@@ -68,18 +68,18 @@ 
 	//echo "\n-->\n";
 
 	if (($db_connect == '0') && $link) {
-		echo "<div class='success'><b>" . _T('info_connexion_ok') . '</b></div>';
+		echo "<div class='success'><b>"._T('info_connexion_ok').'</b></div>';
 		echo info_progression_etape(2, 'etape_', 'install/');
 
-		echo info_etape(_T('menu_aide_installation_choix_base') . aider('install2', true));
+		echo info_etape(_T('menu_aide_installation_choix_base').aider('install2', true));
 
 		echo "\n", '<!-- ', sql_version($server_db), ' -->';
 		[$checked, $res] = install_etape_2_bases($login_db, $server_db);
 
 		$hidden = (defined('_SPIP_CHMOD')
 				? ''
-				: ("\n<input type='hidden' name='chmod' value='" . spip_htmlspecialchars($chmod) . "' />"))
-			. predef_ou_cache($adresse_db . ($port ? ':' . $port : ''), $login_db, $pass_db, $server_db);
+				: ("\n<input type='hidden' name='chmod' value='".spip_htmlspecialchars($chmod)."' />"))
+			. predef_ou_cache($adresse_db.($port ? ':'.$port : ''), $login_db, $pass_db, $server_db);
 
 		echo install_etape_2_form($hidden, $checked, $res, 3);
 	} else {
@@ -87,8 +87,8 @@ 
 
 		echo "<div class='error'>";
 		echo info_etape(_T('info_connexion_base'));
-		echo '<h3>' . _T('avis_connexion_echec_1') . '</h3>';
-		echo '<p>' . _T('avis_connexion_echec_2') . '</p>';
+		echo '<h3>'._T('avis_connexion_echec_1').'</h3>';
+		echo '<p>'._T('avis_connexion_echec_2').'</p>';
 
 		echo "<p style='font-size: small;'>",
 		_T('avis_connexion_echec_3'),
@@ -121,8 +121,8 @@ 
 			. ' '
 		];
 	}
-	$res = '<b>' . _T('avis_lecture_noms_bases_1') . '</b>
-		' . _T('avis_lecture_noms_bases_2') . '<p>';
+	$res = '<b>'._T('avis_lecture_noms_bases_1').'</b>
+		' . _T('avis_lecture_noms_bases_2').'<p>';
 
 	$checked = false;
 	if ($login_db) {
@@ -139,10 +139,10 @@ 
 		if ($ok) {
 			$res .= _T('avis_lecture_noms_bases_3')
 				. '<ul>'
-				. '<li><input name="choix_db" value="' . $test_base . "\" type='radio' id='stand' checked='checked' />"
-				. "<label for='stand'>" . $test_base . "</label></li>\n"
+				. '<li><input name="choix_db" value="'.$test_base."\" type='radio' id='stand' checked='checked' />"
+				. "<label for='stand'>".$test_base."</label></li>\n"
 				. '</ul>'
-				. '<p>' . _T('info_ou') . ' ';
+				. '<p>'._T('info_ou').' ';
 			$checked = true;
 		}
 	}
@@ -156,20 +156,20 @@ 
 		"\n<input type='hidden' name='etape' value='$etape' />"
 		. $hidden
 		. (defined('_INSTALL_NAME_DB')
-			? '<h3>' . _T('install_nom_base_hebergeur') . ' <tt>' . _INSTALL_NAME_DB . '</tt>' . '</h3>'
-			: "\n<fieldset><legend>" . _T('texte_choix_base_1') . "</legend>\n"
+			? '<h3>'._T('install_nom_base_hebergeur').' <tt>'._INSTALL_NAME_DB.'</tt>'.'</h3>'
+			: "\n<fieldset><legend>"._T('texte_choix_base_1')."</legend>\n"
 			. $res
 			. "\n<input name=\"choix_db\" value=\"new_spip\" type='radio' id='nou'"
 			. ($checked ? '' : " checked='checked'")
-			. " />\n<label for='nou'>" . _T('info_creer_base') . "</label></p>\n<p>"
+			. " />\n<label for='nou'>"._T('info_creer_base')."</label></p>\n<p>"
 			. "\n<input type='text' name='table_new' class='text' value=\"spip\" size='20' /></p></fieldset>\n"
 		)
 
 		. ((defined('_INSTALL_TABLE_PREFIX')
 			or $GLOBALS['table_prefix'] != 'spip')
-			? '<h3>' . _T('install_table_prefix_hebergeur') . '  <tt>' . $GLOBALS['table_prefix'] . '</tt>' . '</h3>'
-			: '<fieldset><legend>' . _T('texte_choix_table_prefix') . "</legend>\n"
-			. "<p><label for='table_prefix'>" . _T('info_table_prefix') . '</label></p><p>'
+			? '<h3>'._T('install_table_prefix_hebergeur').'  <tt>'.$GLOBALS['table_prefix'].'</tt>'.'</h3>'
+			: '<fieldset><legend>'._T('texte_choix_table_prefix')."</legend>\n"
+			. "<p><label for='table_prefix'>"._T('info_table_prefix').'</label></p><p>'
 			. "\n<input type='text' id='tprefix' name='tprefix' class='text' value='"
 			. 'spip' # valeur par defaut
 			. "' size='20' /></p></fieldset>"

ecrire/install/etape_3b.php

Spacing +6 added lines, -6 removed lines

@@ -51,9 +51,9 @@ 
 		if ($echec) {
 			echo minipres(
 				'AUTO',
-				info_progression_etape(3, 'etape_', 'install/', true) .
-				"<div class='error'><h3>$echec</h3>\n" .
-				'<p>' . _T('avis_connexion_echec_2') . '</p>' .
+				info_progression_etape(3, 'etape_', 'install/', true).
+				"<div class='error'><h3>$echec</h3>\n".
+				'<p>'._T('avis_connexion_echec_2').'</p>'.
 				'</div>'
 			);
 			exit;
@@ -89,7 +89,7 @@ 
 		$htpass = generer_htpass($pass);
 		$alea_actuel = creer_uniqid();
 		$alea_futur = creer_uniqid();
-		$shapass = spip_sha256($alea_actuel . $pass);
+		$shapass = spip_sha256($alea_actuel.$pass);
 		// prelablement, creer le champ webmestre si il n'existe pas (install neuve
 		// sur une vieille base
 		$t = sql_showtable('spip_auteurs', true);
@@ -97,7 +97,7 @@ 
 			@sql_alter("TABLE spip_auteurs ADD webmestre varchar(3)  DEFAULT 'non' NOT NULL");
 		}
 
-		$id_auteur = sql_getfetsel('id_auteur', 'spip_auteurs', 'login=' . sql_quote($login));
+		$id_auteur = sql_getfetsel('id_auteur', 'spip_auteurs', 'login='.sql_quote($login));
 		if ($id_auteur !== null) {
 			sql_updateq('spip_auteurs', [
 				'nom' => $nom,
@@ -137,7 +137,7 @@ 
 			!$auteur = auth_identifier_login($login, $pass)
 			or !auth_loger($auteur)
 		) {
-			spip_log("login automatique impossible $auth_spip $session" . (is_countable($row) ? count($row) : 0));
+			spip_log("login automatique impossible $auth_spip $session".(is_countable($row) ? count($row) : 0));
 		}
 	}