spip / SPIP

ecrire/iterateur/data.php

Spacing +17 added lines, -17 removed lines

@@ -234,7 +234,7 @@ 
 		// Si a ce stade on n'a pas de table, il y a un bug
 		if (!is_array($this->tableau)) {
 			$this->err = true;
-			spip_log('erreur datasource ' . var_export($command, true));
+			spip_log('erreur datasource '.var_export($command, true));
 		}
 
 		// {datapath query.results}
@@ -276,7 +276,7 @@ 
 			isset($this->command['sourcemode'])
 			and !in_array($this->command['sourcemode'], ['table', 'array', 'tableau'])
 		) {
-			charger_fonction($this->command['sourcemode'] . '_to_array', 'inc', true);
+			charger_fonction($this->command['sourcemode'].'_to_array', 'inc', true);
 		}
 
 		# le premier argument peut etre un array, une URL etc.
@@ -285,7 +285,7 @@ 
 		# avons-nous un cache dispo ?
 		$cle = null;
 		if (is_string($src)) {
-			$cle = 'datasource_' . md5($this->command['sourcemode'] . ':' . var_export($this->command['source'], true));
+			$cle = 'datasource_'.md5($this->command['sourcemode'].':'.var_export($this->command['source'], true));
 		}
 
 		$cache = $this->cache_get($cle);
@@ -344,7 +344,7 @@ 
 
 					if (
 						!$this->err
-						and $data_to_array = charger_fonction($this->command['sourcemode'] . '_to_array', 'inc', true)
+						and $data_to_array = charger_fonction($this->command['sourcemode'].'_to_array', 'inc', true)
 					) {
 						$args = $this->command['source'];
 						$args[0] = $data;
@@ -484,13 +484,13 @@ 
 							$tv = '%s';
 						} # {par valeur/xx/yy} ??
 						else {
-							$tv = 'table_valeur(%s, ' . var_export($r[1], true) . ')';
+							$tv = 'table_valeur(%s, '.var_export($r[1], true).')';
 						}
 						$sortfunc .= '
-					$a = ' . sprintf($tv, '$aa') . ';
-					$b = ' . sprintf($tv, '$bb') . ';
+					$a = ' . sprintf($tv, '$aa').';
+					$b = ' . sprintf($tv, '$bb').';
 					if ($a <> $b)
-						return ($a ' . (!empty($r[2]) ? '>' : '<') . ' $b) ? -1 : 1;';
+						return ($a ' . (!empty($r[2]) ? '>' : '<').' $b) ? -1 : 1;';
 					}
 				}
 			}
@@ -643,7 +643,7 @@ 
 function inc_sql_to_array_dist($data) {
 	# sortir le connecteur de $data
 	preg_match(',^(?:(\w+):)?(.*)$,Sm', $data, $v);
-	$serveur = (string)$v[1];
+	$serveur = (string) $v[1];
 	$req = trim($v[2]);
 	if ($s = sql_query($req, $serveur)) {
 		$r = [];
@@ -668,7 +668,7 @@ 
 		$json = json_decode($data, true, 512, JSON_THROW_ON_ERROR);
 	} catch (JsonException $e) {
 		$json = null;
-		spip_log('Failed to parse Json data : ' . $e->getMessage(), _LOG_INFO);
+		spip_log('Failed to parse Json data : '.$e->getMessage(), _LOG_INFO);
 	}
 	return is_array($json) ? (array) $json : [];
 }
@@ -688,13 +688,13 @@ 
 	$i = 1;
 	foreach ($entete as $k => $v) {
 		if (trim($v) == '') {
-			$v = 'col' . $i;
+			$v = 'col'.$i;
 		} // reperer des eventuelles cases vides
 		if (is_numeric($v) and $v < 0) {
-			$v = '__' . $v;
+			$v = '__'.$v;
 		} // ne pas risquer d'ecraser une cle numerique
 		if (is_numeric($v)) {
-			$v = '_' . $v;
+			$v = '_'.$v;
 		} // ne pas risquer d'ecraser une cle numerique
 		$v = strtolower(preg_replace(',\W+,', '_', translitteration($v)));
 		foreach ($csv as &$item) {
@@ -780,7 +780,7 @@ 
  * @return array|bool
  */
 function inc_pregfiles_to_array_dist($dir, $regexp = -1, $limit = 10000) {
-	return (array)preg_files($dir, $regexp, $limit);
+	return (array) preg_files($dir, $regexp, $limit);
 }
 
 /**
@@ -795,13 +795,13 @@ 
 	$glob_to_array = charger_fonction('glob_to_array', 'inc');
 	$a = $glob_to_array($data);
 	foreach ($a as &$v) {
-		$b = (array)@stat($v);
+		$b = (array) @stat($v);
 		foreach ($b as $k => $ignore) {
 			if (is_numeric($k)) {
 				unset($b[$k]);
 			}
 		}
-		$b['file'] = preg_replace('`/$`', '', $v) ;
+		$b['file'] = preg_replace('`/$`', '', $v);
 		$v = array_merge(
 			pathinfo($v),
 			$b
@@ -821,7 +821,7 @@ 
 	$xml_array = [];
 	for ($object->rewind(); $object->valid(); $object->next()) {
 		if (array_key_exists($key = $object->key(), $xml_array)) {
-			$key .= '-' . uniqid();
+			$key .= '-'.uniqid();
 		}
 		$vars = get_object_vars($object->current());
 		if (isset($vars['@attributes'])) {

ecrire/maj/2021.php

Spacing +2 added lines, -2 removed lines

@@ -23,6 +23,6 @@
 }
 
 $GLOBALS['maj'][2021_02_18_00] = [
-	['sql_alter', "TABLE spip_auteurs CHANGE imessage imessage VARCHAR(3) DEFAULT '' NOT NULL" ],
-	['sql_updateq', 'spip_auteurs', ['imessage' => 'oui'], "imessage != 'non' OR imessage IS NULL" ],
+	['sql_alter', "TABLE spip_auteurs CHANGE imessage imessage VARCHAR(3) DEFAULT '' NOT NULL"],
+	['sql_updateq', 'spip_auteurs', ['imessage' => 'oui'], "imessage != 'non' OR imessage IS NULL"],
 ];

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)
 			);