spip / SPIP

ecrire/maj/legacy/v30.php

Spacing +20 added lines, -20 removed lines

@@ -60,9 +60,9 @@ 
 	$exceptions_pluriel = ['forum' => 'forum', 'syndic' => 'syndic'];
 
 	$pivot = preg_replace(',[^\w],', '', $pivot); // securite
-	$pivots = ($exceptions_pluriel[$pivot] ?? $pivot . 's');
-	$liens = 'spip_' . $pivots . '_liens';
-	$id_pivot = 'id_' . $pivot;
+	$pivots = ($exceptions_pluriel[$pivot] ?? $pivot.'s');
+	$liens = 'spip_'.$pivots.'_liens';
+	$id_pivot = 'id_'.$pivot;
 	// Creer spip_auteurs_liens
 	global $tables_auxiliaires;
 	if (!$l) {
@@ -74,8 +74,8 @@ 
 		$l = preg_replace(',[^\w],', '', $l); // securite
 		$primary = "id_$l";
 		$objet = ($l == 'syndic' ? 'site' : $l);
-		$ls = ($exceptions_pluriel[$l] ?? $l . 's');
-		$ancienne_table = 'spip_' . $pivots . '_' . $ls;
+		$ls = ($exceptions_pluriel[$l] ?? $l.'s');
+		$ancienne_table = 'spip_'.$pivots.'_'.$ls;
 		$pool = 400;
 
 		$trouver_table = charger_fonction('trouver_table', 'base');
@@ -111,12 +111,12 @@ 
 		while ($ids = array_map('reset', sql_allfetsel("$primary", $ancienne_table, '', '', '', "0,$sub_pool"))) {
 			$insert = [];
 			foreach ($ids as $id) {
-				$n = sql_countsel($liens, "objet='$objet' AND id_objet=" . intval($id));
-				while ($t = sql_allfetsel($champs, $ancienne_table, "$primary=" . intval($id), '', $id_pivot, "$n,$pool")) {
+				$n = sql_countsel($liens, "objet='$objet' AND id_objet=".intval($id));
+				while ($t = sql_allfetsel($champs, $ancienne_table, "$primary=".intval($id), '', $id_pivot, "$n,$pool")) {
 					$n += is_countable($t) ? count($t) : 0;
 					// empiler en s'assurant a minima de l'unicite
 					while ($r = array_shift($t)) {
-						$insert[$r[$id_pivot] . ':' . $r['id_objet']] = $r;
+						$insert[$r[$id_pivot].':'.$r['id_objet']] = $r;
 					}
 					if (count($insert) >= $sub_pool) {
 						maj_liens_insertq_multi_check($liens, $insert, $tables_auxiliaires[$liens]);
@@ -267,13 +267,13 @@ 
 
 	// cas particulier spip_auteurs : retablir le collate binary sur le login
 	$desc = $trouver_table('spip_auteurs');
-	spip_log('spip_auteurs : ' . var_export($desc['field'], true), 'maj.' . _LOG_INFO_IMPORTANTE);
+	spip_log('spip_auteurs : '.var_export($desc['field'], true), 'maj.'._LOG_INFO_IMPORTANTE);
 	if (stripos($desc['field']['login'], 'BINARY') === false) {
 		spip_log('Retablir champ login BINARY sur table spip_auteurs', 'maj');
 		sql_alter('table spip_auteurs change login login VARCHAR(255) BINARY');
 		$trouver_table('');
 		$new_desc = $trouver_table('spip_auteurs');
-		spip_log('Apres conversion spip_auteurs : ' . var_export($new_desc['field'], true), 'maj.' . _LOG_INFO_IMPORTANTE);
+		spip_log('Apres conversion spip_auteurs : '.var_export($new_desc['field'], true), 'maj.'._LOG_INFO_IMPORTANTE);
 	}
 
 	foreach ($tables as $table) {
@@ -292,27 +292,27 @@ 
 					$data = sql_allfetsel('*', 'spip_urls', '', '', 'date DESC');
 					$urls = [];
 					foreach ($data as $d) {
-						$key = $d['id_parent'] . '::' . strtolower($d['url']);
+						$key = $d['id_parent'].'::'.strtolower($d['url']);
 						if (!isset($urls[$key])) {
 							$urls[$key] = true;
 						} else {
 							spip_log(
-								'Suppression doublon dans spip_urls avant conversion : ' . serialize($d),
-								'maj.' . _LOG_INFO_IMPORTANTE
+								'Suppression doublon dans spip_urls avant conversion : '.serialize($d),
+								'maj.'._LOG_INFO_IMPORTANTE
 							);
-							sql_delete('spip_urls', 'id_parent=' . sql_quote($d['id_parent']) . ' AND url=' . sql_quote($d['url']));
+							sql_delete('spip_urls', 'id_parent='.sql_quote($d['id_parent']).' AND url='.sql_quote($d['url']));
 						}
 					}
 				}
 				foreach ($desc['field'] as $field => $type) {
 					if ($desc['field'][$field] !== $desc_collate[$field]) {
-						spip_log("Conversion COLLATE table $table", 'maj.' . _LOG_INFO_IMPORTANTE);
-						sql_alter("table $table change $field $field " . $desc_collate[$field]);
+						spip_log("Conversion COLLATE table $table", 'maj.'._LOG_INFO_IMPORTANTE);
+						sql_alter("table $table change $field $field ".$desc_collate[$field]);
 						$trouver_table('');
 						$new_desc = $trouver_table($table);
 						spip_log(
-							"Apres conversion $table : " . var_export($new_desc['field'], true),
-							'maj.' . _LOG_INFO_IMPORTANTE
+							"Apres conversion $table : ".var_export($new_desc['field'], true),
+							'maj.'._LOG_INFO_IMPORTANTE
 						);
 						continue 2; // inutile de continuer pour cette table : un seul alter remet tout a jour en sqlite
 					}
@@ -328,7 +328,7 @@ 
 
 $GLOBALS['maj'][19236] = [
 	['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom='version_installee'"], // version base principale
-	['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom LIKE '%_base_version'"],  // version base plugins
+	['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom LIKE '%_base_version'"], // version base plugins
 	['maj_collation_sqlite'],
 ];
 
@@ -345,7 +345,7 @@ 
 	spip_log('supprimer sessions auteur');
 	if ($dir = opendir(_DIR_SESSIONS)) {
 		while (($f = readdir($dir)) !== false) {
-			spip_unlink(_DIR_SESSIONS . $f);
+			spip_unlink(_DIR_SESSIONS.$f);
 			if (time() >= _TIME_OUT) {
 				return;
 			}

ecrire/action/editer_auteur.php

Spacing +5 added lines, -5 removed lines

@@ -93,7 +93,7 @@ 
 	$champs['source'] = $source ?: 'spip';
 
 	$champs['login'] = '';
-	$champs['statut'] = '5poubelle';  // inutilisable tant qu'il n'a pas ete renseigne et institue
+	$champs['statut'] = '5poubelle'; // inutilisable tant qu'il n'a pas ete renseigne et institue
 	$champs['webmestre'] = 'non';
 	if (empty($champs['imessage'])) {
 		$champs['imessage'] = 'oui';
@@ -313,7 +313,7 @@ 
 		$champs['pass'] = $c['pass'];
 	}
 
-	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur='.intval($id_auteur));
 
 	if (
 		isset($c['statut'])
@@ -373,7 +373,7 @@ 
 	// commencer par traiter les cas particuliers des logins et pass
 	// avant les autres ecritures en base
 	if (isset($champs['login']) or isset($champs['pass'])) {
-		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur='.intval($id_auteur));
 		include_spip('inc/auth');
 		if (isset($champs['login']) and strlen($champs['login'])) {
 			if (!auth_modifier_login($auth_methode, $champs['login'], $id_auteur)) {
@@ -381,7 +381,7 @@ 
 			}
 		}
 		if (isset($champs['pass']) and strlen($champs['pass'])) {
-			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur='.intval($id_auteur));
 			if (!auth_modifier_pass($auth_methode, $champs['login'], $champs['pass'], $id_auteur)) {
 				$erreurs[] = 'ecrire:impossible_modifier_pass_auteur';
 			}
@@ -394,7 +394,7 @@ 
 	if (!(is_countable($champs) ? count($champs) : 0)) {
 		return implode(' ', array_map('_T', $erreurs));
 	}
-	sql_updateq('spip_auteurs', $champs, 'id_auteur=' . $id_auteur);
+	sql_updateq('spip_auteurs', $champs, 'id_auteur='.$id_auteur);
 
 	// .. mettre a jour les fichiers .htpasswd et .htpasswd-admin
 	if (

ecrire/action/editer_liens.php

Spacing +16 added lines, -16 removed lines

@@ -53,7 +53,7 @@ 
 	$l = '';
 	if (
 		$primary = id_table_objet($objet)
-		and $trouver_table($l = $table_sql . '_liens', '', true, ['log_missing' => false])
+		and $trouver_table($l = $table_sql.'_liens', '', true, ['log_missing' => false])
 		and !preg_match(',[^\w],', $primary)
 		and !preg_match(',[^\w],', $l)
 	) {
@@ -212,7 +212,7 @@ 
  * @return int
  */
 function objet_optimiser_liens($objets_source, $objets_lies) {
-	spip_log('objet_optimiser_liens : ' . json_encode($objets_source, JSON_THROW_ON_ERROR) . ', ' . json_encode($objets_lies, JSON_THROW_ON_ERROR), 'genie' . _LOG_DEBUG);
+	spip_log('objet_optimiser_liens : '.json_encode($objets_source, JSON_THROW_ON_ERROR).', '.json_encode($objets_lies, JSON_THROW_ON_ERROR), 'genie'._LOG_DEBUG);
 	return objet_traiter_liaisons('lien_optimise', $objets_source, $objets_lies);
 }
 
@@ -446,7 +446,7 @@ 
 					$where_meme_lien = implode(' AND ', $where_meme_lien);
 					// on decale les liens de rang_lien>=la valeur inseree pour faire la place
 					// sauf sur le meme lien avec un role eventuellement different
-					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . intval($insertions['rang_lien']), "NOT($where_meme_lien)"]);
+					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>='.intval($insertions['rang_lien']), "NOT($where_meme_lien)"]);
 					sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
 				}
 
@@ -511,7 +511,7 @@ 
 					$rang = 1;
 					foreach ($liens as $lien) {
 						if (empty($deja_reordonne[$lien[$primary]][$lien['objet']][$lien['id_objet']])) {
-							$where = lien_where($primary, $lien[$primary], $lien['objet'], $lien['id_objet'], ['rang_lien!=' . intval($rang)]);
+							$where = lien_where($primary, $lien[$primary], $lien['objet'], $lien['id_objet'], ['rang_lien!='.intval($rang)]);
 							sql_updateq($table_lien, ['rang_lien' => $rang], $where);
 
 							if (empty($deja_reordonne[$lien[$primary]])) {
@@ -587,7 +587,7 @@ 
 			addslashes($primary),
 			array_map('intval', $id_source),
 			$not
-		) : addslashes($primary) . ($not ? '<>' : '=') . intval($id_source));
+		) : addslashes($primary).($not ? '<>' : '=').intval($id_source));
 	} elseif ($not) {
 		$where[] = '0=1';
 	} // idiot mais quand meme
@@ -599,14 +599,14 @@ 
 	}
 
 	if ($objet !== '*') {
-		$where[] = 'objet=' . sql_quote($objet);
+		$where[] = 'objet='.sql_quote($objet);
 	}
 	if ($id_objet !== '*') {
 		$where[] = (is_array($id_objet) ? sql_in(
 			'id_objet',
 			array_map('intval', $id_objet),
 			$not
-		) : 'id_objet' . ($not ? '<>' : '=') . intval($id_objet));
+		) : 'id_objet'.($not ? '<>' : '=').intval($id_objet));
 	} elseif ($not) {
 		$where[] = '0=1';
 	} // idiot mais quand meme
@@ -627,7 +627,7 @@ 
 function lien_rang_where($table_lien, $primary, $id_source, $objet, $id_objet, $cond = []) {
 
 	// si on veut compter les rangs autrement que le core ne le fait par defaut, fournir le where adhoc
-	if (function_exists($f = 'lien_rang_where_' . $table_lien)) {
+	if (function_exists($f = 'lien_rang_where_'.$table_lien)) {
 		return $f($primary, $id_source, $objet, $id_objet, $cond);
 	}
 
@@ -792,8 +792,8 @@ 
 					// il faut les eliminier en repetant la condition dans le where L.objet='xxx'
 					"$table_lien AS L
 									LEFT JOIN $spip_table_objet AS O
-										ON (O.$id_table_objet=L.id_objet AND L.objet=" . sql_quote($type) . ')',
-					'L.objet=' . sql_quote($type) . " AND O.$id_table_objet IS NULL"
+										ON (O.$id_table_objet=L.id_objet AND L.objet=".sql_quote($type).')',
+					'L.objet='.sql_quote($type)." AND O.$id_table_objet IS NULL"
 				);
 				// sur une cle primaire composee, pas d'autres solutions que de virer un a un
 				while ($row = sql_fetch($res)) {
@@ -802,13 +802,13 @@ 
 					}
 					$e = sql_delete(
 						$table_lien,
-						["$primary=" . $row['id'], 'id_objet=' . $row['id_objet'], 'objet=' . sql_quote($type)]
+						["$primary=".$row['id'], 'id_objet='.$row['id_objet'], 'objet='.sql_quote($type)]
 					);
 					if ($e != false) {
 						$dels += $e;
 						spip_log(
-							'lien_optimise: Entree ' . $row['id'] . '/' . $row['id_objet'] . "/$type supprimee dans la table $table_lien",
-							'genie' . _LOG_INFO_IMPORTANTE
+							'lien_optimise: Entree '.$row['id'].'/'.$row['id_objet']."/$type supprimee dans la table $table_lien",
+							'genie'._LOG_INFO_IMPORTANTE
 						);
 					}
 				}
@@ -912,7 +912,7 @@ 
 					// sauf sur le meme lien avec un role eventuellement different
 					$where_meme_lien = lien_where($primary, $id, $objet, $id_objet);
 					$where_meme_lien = implode(' AND ', $where_meme_lien);
-					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . intval($qualif['rang_lien']), "NOT($where_meme_lien)"]);
+					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>='.intval($qualif['rang_lien']), "NOT($where_meme_lien)"]);
 					sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
 				}
 				// tous les liens de même rôle recoivent le rang indiqué aussi
@@ -1004,7 +1004,7 @@ 
  */
 function lien_propage_date_modif($objet, $ids) {
 	static $done = [];
-	$hash = md5($objet . serialize($ids));
+	$hash = md5($objet.serialize($ids));
 
 	// sql_updateq, peut être un rien lent.
 	// On évite de l'appeler 2 fois sur les mêmes choses
@@ -1020,7 +1020,7 @@ 
 		and isset($desc['field']['date_modif'])
 	) {
 		$primary = id_table_objet($objet);
-		$where = (is_array($ids) ? sql_in($primary, array_map('intval', $ids)) : "$primary=" . intval($ids));
+		$where = (is_array($ids) ? sql_in($primary, array_map('intval', $ids)) : "$primary=".intval($ids));
 		sql_updateq($table, ['date_modif' => date('Y-m-d H:i:s')], $where);
 	}
 

ecrire/action/editer_objet.php

Spacing +20 added lines, -20 removed lines

@@ -76,12 +76,12 @@ 
  */
 function objet_modifier($objet, $id, $set = null) {
 	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_modifier: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+		spip_log("objet_modifier: appel avec type $objet invalide au lieu de $t", 'editer'._LOG_INFO_IMPORTANTE);
 		$objet = $t;
 	}
 	if (
-		include_spip('action/editer_' . $objet)
-		and function_exists($modifier = $objet . '_modifier')
+		include_spip('action/editer_'.$objet)
+		and function_exists($modifier = $objet.'_modifier')
 	) {
 		return $modifier($id, $set);
 	}
@@ -90,7 +90,7 @@ 
 	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_sql);
 	if (!$desc or !isset($desc['field'])) {
-		spip_log("Objet $objet inconnu dans objet_modifier", 'editer' . _LOG_ERREUR);
+		spip_log("Objet $objet inconnu dans objet_modifier", 'editer'._LOG_ERREUR);
 
 		return _L("Erreur objet $objet inconnu");
 	}
@@ -170,12 +170,12 @@ 
 function objet_inserer($objet, $id_parent = null, $set = null) {
 	$d = null;
 	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_inserer: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+		spip_log("objet_inserer: appel avec type $objet invalide au lieu de $t", 'editer'._LOG_INFO_IMPORTANTE);
 		$objet = $t;
 	}
 	if (
-		include_spip('action/editer_' . $objet)
-		and function_exists($inserer = $objet . '_inserer')
+		include_spip('action/editer_'.$objet)
+		and function_exists($inserer = $objet.'_inserer')
 	) {
 		return $inserer($id_parent, $set);
 	}
@@ -196,7 +196,7 @@ 
 			$row = sql_fetsel('id_rubrique, id_secteur, lang', 'spip_rubriques', 'id_parent=0', '', '0+titre,titre', '1');
 			$id_rubrique = $row['id_rubrique'];
 		} else {
-			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
+			$row = sql_fetsel('lang, id_secteur', 'spip_rubriques', 'id_rubrique='.intval($id_rubrique));
 		}
 
 		$champs['id_rubrique'] = $id_rubrique;
@@ -314,12 +314,12 @@ 
  */
 function objet_instituer($objet, $id, $c, $calcul_rub = true) {
 	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_instituer: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+		spip_log("objet_instituer: appel avec type $objet invalide au lieu de $t", 'editer'._LOG_INFO_IMPORTANTE);
 		$objet = $t;
 	}
 	if (
-		include_spip('action/editer_' . $objet)
-		and function_exists($instituer = $objet . '_instituer')
+		include_spip('action/editer_'.$objet)
+		and function_exists($instituer = $objet.'_instituer')
 	) {
 		return $instituer($id, $c, $calcul_rub);
 	}
@@ -348,7 +348,7 @@ 
 	$sel[] = ($champ_date ? "$champ_date as date" : "'' as date");
 	$sel[] = (isset($desc['field']['id_rubrique']) ? 'id_rubrique' : '0 as id_rubrique');
 
-	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet) . '=' . intval($id));
+	$row = sql_fetsel($sel, $table_sql, id_table_objet($objet).'='.intval($id));
 
 	$id_rubrique = $row['id_rubrique'];
 	$statut_ancien = $statut = $row['statut'];
@@ -371,7 +371,7 @@ 
 			if ($s != 'publie' and autoriser('modifier', $objet, $id)) {
 				$statut = $champs['statut'] = $s;
 			} else {
-				spip_log("editer_objet $objet #$id refus " . json_encode($c, JSON_THROW_ON_ERROR), 'editer' . _LOG_INFO_IMPORTANTE);
+				spip_log("editer_objet $objet #$id refus ".json_encode($c, JSON_THROW_ON_ERROR), 'editer'._LOG_INFO_IMPORTANTE);
 			}
 		}
 
@@ -401,7 +401,7 @@ 
 		and isset($c['id_parent'])
 		and $id_parent = $c['id_parent']
 		and $id_parent != $id_rubrique
-		and (sql_fetsel('1', 'spip_rubriques', 'id_rubrique=' . intval($id_parent)))
+		and (sql_fetsel('1', 'spip_rubriques', 'id_rubrique='.intval($id_parent)))
 	) {
 		$champs['id_rubrique'] = $id_parent;
 
@@ -500,7 +500,7 @@ 
 	// Si on deplace l'objet
 	// changer aussi son secteur et sa langue (si heritee)
 	if (isset($champs['id_rubrique'])) {
-		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique=' . sql_quote($champs['id_rubrique']));
+		$row_rub = sql_fetsel('id_secteur, lang', 'spip_rubriques', 'id_rubrique='.sql_quote($champs['id_rubrique']));
 		$langue = $row_rub['lang'];
 
 		if (isset($desc['field']['id_secteur'])) {
@@ -512,7 +512,7 @@ 
 				sql_fetsel(
 					'1',
 					$table_sql,
-					id_table_objet($objet) . '=' . intval($id) . " AND langue_choisie<>'oui' AND lang<>" . sql_quote($langue)
+					id_table_objet($objet).'='.intval($id)." AND langue_choisie<>'oui' AND lang<>".sql_quote($langue)
 				)
 			) {
 				$champs['lang'] = $langue;
@@ -523,7 +523,7 @@ 
 	if (!$champs) {
 		return;
 	}
-	sql_updateq($table_sql, $champs, id_table_objet($objet) . '=' . intval($id));
+	sql_updateq($table_sql, $champs, id_table_objet($objet).'='.intval($id));
 
 	// Changer le statut des rubriques concernees
 	if ($cond) {
@@ -567,7 +567,7 @@ 
  */
 function objet_lire($objet, $valeur_id, $options = []) {
 	if (($t = objet_type($objet)) !== $objet) {
-		spip_log("objet_lire: appel avec type $objet invalide au lieu de $t", 'editer' . _LOG_INFO_IMPORTANTE);
+		spip_log("objet_lire: appel avec type $objet invalide au lieu de $t", 'editer'._LOG_INFO_IMPORTANTE);
 		$objet = $t;
 	}
 
@@ -589,7 +589,7 @@ 
 	) {
 		// Il est possible pour un type d'objet de fournir une fonction de lecture de tous les champs d'un objet.
 		if (
-			include_spip('action/editer_' . $objet)
+			include_spip('action/editer_'.$objet)
 			and function_exists($lire = "${objet}_lire_champs")
 		) {
 			$valeurs = $lire($objet, $valeur_id, $champ_id);
@@ -599,7 +599,7 @@ 
 
 			// La condition est appliquée sur le champ désigné par l'utilisateur.
 			$where = [
-				"${champ_id}=" . sql_quote($valeur_id)
+				"${champ_id}=".sql_quote($valeur_id)
 			];
 
 			// Acquisition de tous les champs de l'objet : si l'accès SQL retourne une erreur on renvoie un tableau vide.

ecrire/action/editer_logo.php

Spacing +14 added lines, -14 removed lines

@@ -89,7 +89,7 @@ 
 		return $erreur;
 	}
 	// chercher dans la base
-	$mode_document = 'logo' . $mode;
+	$mode_document = 'logo'.$mode;
 
 	include_spip('inc/documents');
 	$erreur = '';
@@ -106,7 +106,7 @@ 
 		$tmp_name = false;
 		if (file_exists($source)) {
 			$tmp_name = $source;
-		} elseif (file_exists($f = determine_upload() . $source)) {
+		} elseif (file_exists($f = determine_upload().$source)) {
 			$tmp_name = $f;
 		}
 		if (!$tmp_name) {
@@ -140,7 +140,7 @@ 
 
 	if (!is_numeric($id_document)) {
 		$erreur = ($id_document ?: 'Erreur inconnue');
-		spip_log("Erreur ajout logo : $erreur pour source=" . json_encode($source, JSON_THROW_ON_ERROR), 'logo');
+		spip_log("Erreur ajout logo : $erreur pour source=".json_encode($source, JSON_THROW_ON_ERROR), 'logo');
 		return $erreur;
 	}
 
@@ -166,23 +166,23 @@ 
 	$desc = $trouver_table($table);
 
 	foreach (['on', 'off'] as $mode) {
-		$nom_base = $type . $mode;
+		$nom_base = $type.$mode;
 		$dir = (defined('_DIR_LOGOS') ? _DIR_LOGOS : _DIR_IMG);
 
-		$files = glob($dir . $nom_base . '*');
+		$files = glob($dir.$nom_base.'*');
 		// est-ce que c'est une nouvelle tentative de migration ?
 		// dans ce cas les logos sont deja dans IMG/logo/
 		if (!(is_countable($files) ? count($files) : 0)) {
-			$files = glob($dir_logos . $nom_base . '*');
+			$files = glob($dir_logos.$nom_base.'*');
 			if (is_countable($files) ? count($files) : 0) {
 				// mais il faut verifier si ils ont pas deja ete migres pour tout ou partie
 				$filescheck = [];
 				foreach ($files as $file) {
-					$short = basename(dirname($file)) . DIRECTORY_SEPARATOR . basename($file);
+					$short = basename(dirname($file)).DIRECTORY_SEPARATOR.basename($file);
 					$filescheck[$short] = $file;
 				}
 				// trouver ceux deja migres
-				$deja = sql_allfetsel('fichier', 'spip_documents', sql_in('fichier', array_keys($filescheck)) . " AND mode LIKE 'logo%'");
+				$deja = sql_allfetsel('fichier', 'spip_documents', sql_in('fichier', array_keys($filescheck))." AND mode LIKE 'logo%'");
 				if (is_countable($deja) ? count($deja) : 0) {
 					$deja = array_column($deja, 'fichier');
 					$restant = array_diff(array_keys($filescheck), $deja);
@@ -201,11 +201,11 @@ 
 			}
 		}
 
-		spip_log("logo_migrer_en_base $objet $mode : " . (is_countable($files) ? count($files) : 0) . ' logos restant', 'maj' . _LOG_INFO_IMPORTANTE);
+		spip_log("logo_migrer_en_base $objet $mode : ".(is_countable($files) ? count($files) : 0).' logos restant', 'maj'._LOG_INFO_IMPORTANTE);
 
 		$deja = [];
 		foreach ($files as $file) {
-			$logo = substr($file, strlen($dir . $nom_base));
+			$logo = substr($file, strlen($dir.$nom_base));
 			$logo = explode('.', $logo);
 			if (
 				is_numeric($logo[0])
@@ -216,16 +216,16 @@ 
 					// if no logo in base
 					if (!$logo or (is_countable($logo) ? count($logo) : 0) < 6) {
 						foreach ($formats_logos as $format) {
-							if (@file_exists($d = ($dir . ($nom = $nom_base . intval($id_objet) . '.' . $format)))) {
+							if (@file_exists($d = ($dir.($nom = $nom_base.intval($id_objet).'.'.$format)))) {
 								if (isset($desc['field']['date_modif'])) {
 									$date_modif = sql_getfetsel('date_modif', $table, "$_id_objet=$id_objet");
 								} else {
 									$date_modif = null;
 								}
 								// logo_modifier commence par supprimer le logo existant, donc on le deplace pour pas le perdre
-								@rename($d, $dir_logos . $nom);
+								@rename($d, $dir_logos.$nom);
 								// et on le declare comme nouveau logo
-								logo_modifier($objet, $id_objet, $mode, $dir_logos . $nom);
+								logo_modifier($objet, $id_objet, $mode, $dir_logos.$nom);
 								if ($date_modif) {
 									sql_updateq($table, ['date_modif' => $date_modif], "$_id_objet=$id_objet");
 								}
@@ -239,7 +239,7 @@ 
 			// si le fichier est encore la on le move : rien a faire ici
 			// (sauf si c'est une re-migration : il est deja dans logo/ donc il bouge pas)
 			if ($dir !== $dir_logos and file_exists($file)) {
-				@rename($file, $dir_logos_erreurs . basename($file));
+				@rename($file, $dir_logos_erreurs.basename($file));
 			}
 
 			if ($time_limit and time() > $time_limit) {

ecrire/action/tester_taille.php

Spacing +4 added lines, -4 removed lines

@@ -76,7 +76,7 @@ 
 		$t = ($GLOBALS['taille_min'] * $GLOBALS['taille_min']);
 		if ($GLOBALS['taille_min'] !== $GLOBALS['taille_max']) {
 			$t = $t * 0.9; // marge de securite
-			echo round($t / 1_000_000, 3) . ' Mpx';
+			echo round($t / 1_000_000, 3).' Mpx';
 		} else {
 			// c'est un cas "on a reussi la borne max initiale, donc on a pas de limite connue"
 			$t = 0;
@@ -112,19 +112,19 @@ 
 	$image_source = chemin_image('test.png');
 	$GLOBALS['redirect'] = generer_url_action(
 		'tester_taille',
-		"i=$i&arg=" . $GLOBALS['taille_min'] . '-' . $GLOBALS['taille_test']
+		"i=$i&arg=".$GLOBALS['taille_min'].'-'.$GLOBALS['taille_test']
 	);
 
 	ob_start('action_tester_taille_error_handler');
 	filtrer('image_recadre', $image_source, $taille, $taille);
-	$GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
+	$GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 
 	// si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
 	// si $GLOBALS['taille_min']==0 (car on est au premier coup)
 	if ($GLOBALS['taille_min'] == 0) {
 		$taille = $GLOBALS['taille_max'];
 		filtrer('image_recadre', $image_source, $taille, $taille);
-		$GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
+		$GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-".$GLOBALS['taille_max']);
 	}
 	ob_end_clean();
 

ecrire/action/cookie.php

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
 	if ($change_session == 'oui') {
 		$session = charger_fonction('session', 'inc');
 		$session(true);
-		spip_log('statut 204 pour ' . $_SERVER['REQUEST_URI']);
+		spip_log('statut 204 pour '.$_SERVER['REQUEST_URI']);
 		http_response_code(204); // No Content
 		return;
 	}
@@ -79,7 +79,7 @@ 
 				_T('info_connexion_refusee'),
 				_T('login_login_pass_incorrect'),
 				_T('login_retour_site'),
-				'url=' . rawurlencode($redirect),
+				'url='.rawurlencode($redirect),
 				_T('login_nouvelle_tentative'),
 				(strpos($url, (string) _DIR_RESTREINT_ABS) !== false)
 			);

ecrire/balise/logo_.php

Spacing +8 added lines, -10 removed lines

@@ -108,7 +108,7 @@ 
 		if ($fichier) {
 			$code = "quete_logo_file($doc, $qconnect)";
 		} else {
-			$code = "quete_logo_document($doc, " . ($lien ?: "''") . ", '$align', '$mode_logo', $coord_x, $coord_y, $qconnect)";
+			$code = "quete_logo_document($doc, ".($lien ?: "''").", '$align', '$mode_logo', $coord_x, $coord_y, $qconnect)";
 		}
 		// (x=non-faux ? y : '') pour affecter x en retournant y
 		if ($p->descr['documents']) {
@@ -125,7 +125,7 @@ 
 
 	// demande de reduction sur logo avec ecriture spip 2.1 : #LOGO_xxx{200, 0}
 	if ($coord_x or $coord_y) {
-		$code = "filtrer('image_graver',filtrer('image_reduire'," . $code . ", '$coord_x', '$coord_y'))";
+		$code = "filtrer('image_graver',filtrer('image_reduire',".$code.", '$coord_x', '$coord_y'))";
 	}
 
 	$p->code = $code;
@@ -158,14 +158,12 @@ 
  *     Code compilé retournant le chemin du logo ou le code HTML du logo.
  **/
 function logo_survol($id_objet, $_id_objet, $type, $align, $fichier, $_lien, $p, $suite) {
-	$code = "quete_logo('$id_objet', '" .
-		(($suite == '_SURVOL') ? 'off' :
-			(($suite == '_NORMAL') ? 'on' : 'ON')) .
-		"', $_id_objet," .
+	$code = "quete_logo('$id_objet', '".
+		(($suite == '_SURVOL') ? 'off' : (($suite == '_NORMAL') ? 'on' : 'ON')).
+		"', $_id_objet,".
 		(($suite == '_RUBRIQUE') ?
-			champ_sql('id_rubrique', $p) :
-			(($type == 'rubrique') ? "quete_parent($_id_objet)" : "''")) .
-		', ' . intval($fichier) . ')';
+			champ_sql('id_rubrique', $p) : (($type == 'rubrique') ? "quete_parent($_id_objet)" : "''")).
+		', '.intval($fichier).')';
 
 	if ($fichier) {
 		return $code;
@@ -173,5 +171,5 @@ 
 
 	$align = preg_replace(',\W,', '', $align);
 
-	return "quete_html_logo($code, '$align', " . ($_lien ?: "''") . ')';
+	return "quete_html_logo($code, '$align', ".($_lien ?: "''").')';
 }

ecrire/balise/url_logout.php

Spacing +1 added lines, -1 removed lines

@@ -73,5 +73,5 @@
 		return '';
 	}
 
-	return generer_url_action('logout', 'logout=public&url=' . rawurlencode($cible ?: self('&')));
+	return generer_url_action('logout', 'logout=public&url='.rawurlencode($cible ?: self('&')));
 }