spip / SPIP

ecrire/balise/menu_lang_ecrire.php

Spacing +3 added lines, -3 removed lines

@@ -99,9 +99,9 @@ 
 	include_spip('inc/lang');
 
 	if ($GLOBALS['spip_lang'] <> $default) {
-		$opt = lang_select($default);  # et remplace
+		$opt = lang_select($default); # et remplace
 		if ($GLOBALS['spip_lang'] <> $default) {
-			$default = '';  # annule tout choix par defaut
+			$default = ''; # annule tout choix par defaut
 			if ($opt) {
 				lang_select();
 			}
@@ -110,7 +110,7 @@ 
 
 	# lien a partir de /
 	$cible = parametre_url(self(), 'lang', '', '&');
-	$post = generer_url_action('converser', 'redirect=' . rawurlencode($cible), true);
+	$post = generer_url_action('converser', 'redirect='.rawurlencode($cible), true);
 
 	return [
 		'formulaires/menu_lang',

ecrire/action/api_transmettre.php

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
 
 	$res = ltrim($res['texte']);
 	if (empty($res)) {
-		spip_log("$arg $qs resultat vide", 'transmettre' . _LOG_INFO_IMPORTANTE);
+		spip_log("$arg $qs resultat vide", 'transmettre'._LOG_INFO_IMPORTANTE);
 	}
 
 	echo $res;

ecrire/inc/rubriques.php

Spacing +35 added lines, -35 removed lines

@@ -135,9 +135,9 @@ 
 		sql_updateq(
 			'spip_rubriques',
 			['statut' => 'publie', 'date' => date('Y-m-d H:i:s')],
-			'id_rubrique=' . intval($id_rubrique)
+			'id_rubrique='.intval($id_rubrique)
 		);
-		$id_parent = sql_getfetsel('id_parent', 'spip_rubriques AS R', 'R.id_rubrique=' . intval($id_rubrique));
+		$id_parent = sql_getfetsel('id_parent', 'spip_rubriques AS R', 'R.id_rubrique='.intval($id_rubrique));
 		if (!$id_parent) {
 			break;
 		}
@@ -172,7 +172,7 @@ 
 			return $id_pred != $id_rubrique;
 		}
 		// passer au parent si on a depublie
-		$r = sql_fetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_pred));
+		$r = sql_fetsel('id_parent', 'spip_rubriques', 'id_rubrique='.intval($id_pred));
 		$id_pred = $r['id_parent'];
 	}
 
@@ -197,14 +197,14 @@ 
 		$date = date('Y-m-d H:i:s');
 	}
 	$postdates = ($GLOBALS['meta']['post_dates'] == 'non') ?
-		' AND date <= ' . sql_quote($date) : '';
+		' AND date <= '.sql_quote($date) : '';
 
 	if (!$id_rubrique = intval($id_rubrique)) {
 		return false;
 	}
 
 	// verifier qu'elle existe et est bien publiee
-	$r = sql_fetsel('id_rubrique,statut', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
+	$r = sql_fetsel('id_rubrique,statut', 'spip_rubriques', 'id_rubrique='.intval($id_rubrique));
 	if (!$r or $r['statut'] !== 'publie') {
 		return false;
 	}
@@ -214,12 +214,12 @@ 
 	$compte = [
 		'articles' => sql_countsel(
 			'spip_articles',
-			'id_rubrique=' . intval($id_rubrique) . " AND statut='publie'$postdates"
+			'id_rubrique='.intval($id_rubrique)." AND statut='publie'$postdates"
 		),
-		'rubriques' => sql_countsel('spip_rubriques', 'id_parent=' . intval($id_rubrique) . " AND statut='publie'"),
+		'rubriques' => sql_countsel('spip_rubriques', 'id_parent='.intval($id_rubrique)." AND statut='publie'"),
 		'documents' => sql_countsel(
 			'spip_documents AS D JOIN spip_documents_liens AS L ON D.id_document=L.id_document',
-			'L.id_objet=' . intval($id_rubrique) . " AND L.objet='rubrique' and D.mode NOT IN('logoon', 'logooff') "
+			'L.id_objet='.intval($id_rubrique)." AND L.objet='rubrique' and D.mode NOT IN('logoon', 'logooff') "
 		)
 	];
 
@@ -244,7 +244,7 @@ 
 		}
 	}
 
-	sql_updateq('spip_rubriques', ['statut' => 'prepa'], 'id_rubrique=' . intval($id_rubrique));
+	sql_updateq('spip_rubriques', ['statut' => 'prepa'], 'id_rubrique='.intval($id_rubrique));
 
 #		spip_log("depublier_rubrique $id_pred");
 	return true;
@@ -307,7 +307,7 @@ 
 
 	// Afficher les articles post-dates ?
 	$postdates = ($GLOBALS['meta']['post_dates'] == 'non') ?
-		'AND A.date <= ' . sql_quote(date('Y-m-d H:i:s')) : '';
+		'AND A.date <= '.sql_quote(date('Y-m-d H:i:s')) : '';
 
 	$r = sql_select(
 		'R.id_rubrique AS id, max(A.date) AS date_h',
@@ -319,7 +319,7 @@ 
 		sql_updateq(
 			'spip_rubriques',
 			['statut_tmp' => 'publie', 'date_tmp' => $row['date_h']],
-			'id_rubrique=' . intval($row['id'])
+			'id_rubrique='.intval($row['id'])
 		);
 	}
 
@@ -345,7 +345,7 @@ 
 			sql_updateq(
 				'spip_rubriques',
 				['statut_tmp' => 'publie', 'date_tmp' => $row['date_h']],
-				'id_rubrique=' . intval($row['id'])
+				'id_rubrique='.intval($row['id'])
 			);
 			$continuer = true;
 		}
@@ -398,7 +398,7 @@ 
 			and $rows = sql_allfetsel(
 				'A.id_rubrique AS id, R.id_secteur AS id_secteur, R.profondeur+1 as profondeur',
 				'spip_rubriques AS A JOIN spip_rubriques AS R ON A.id_parent = R.id_rubrique',
-				'R.profondeur=' . intval($prof) . ' AND (A.id_secteur <> R.id_secteur OR A.profondeur > R.profondeur+1)',
+				'R.profondeur='.intval($prof).' AND (A.id_secteur <> R.id_secteur OR A.profondeur > R.profondeur+1)',
 				'',
 				'R.id_secteur',
 				'0,100'
@@ -437,11 +437,11 @@ 
 			and $rows = sql_allfetsel(
 				'id_rubrique as id',
 				'spip_rubriques',
-				'profondeur=' . intval($prof + 1) . ' AND id_parent NOT IN (' . sql_get_select(
+				'profondeur='.intval($prof + 1).' AND id_parent NOT IN ('.sql_get_select(
 					'zzz.id_rubrique',
 					'spip_rubriques AS zzz',
-					'zzz.profondeur=' . intval($prof)
-				) . ')',
+					'zzz.profondeur='.intval($prof)
+				).')',
 				'',
 				'',
 				'0,100'
@@ -455,7 +455,7 @@ 
 		// si pas de rubrique a profondeur $prof+1 pas la peine de continuer
 		// si il reste des rubriques non vues, c'est une branche morte ou reference circulaire (base foireuse)
 		// on arrete les frais
-		if (sql_countsel('spip_rubriques', 'profondeur=' . intval($prof + 1))) {
+		if (sql_countsel('spip_rubriques', 'profondeur='.intval($prof + 1))) {
 			$prof++;
 			$continuer = true;
 		}
@@ -463,12 +463,12 @@ 
 
 	// loger si la table des rubriques semble foireuse
 	// et mettre un id_secteur=0 sur ces rubriques pour eviter toute selection par les boucles
-	if (sql_countsel('spip_rubriques', 'profondeur>' . intval($prof + 1))) {
+	if (sql_countsel('spip_rubriques', 'profondeur>'.intval($prof + 1))) {
 		spip_log(
-			'Les rubriques de profondeur>' . ($prof + 1) . ' semblent suspectes (branches morte ou reference circulaire dans les parents)',
+			'Les rubriques de profondeur>'.($prof + 1).' semblent suspectes (branches morte ou reference circulaire dans les parents)',
 			_LOG_CRITIQUE
 		);
-		sql_update('spip_rubriques', ['id_secteur' => 0], 'profondeur>' . intval($prof + 1));
+		sql_update('spip_rubriques', ['id_secteur' => 0], 'profondeur>'.intval($prof + 1));
 	}
 
 	// reparer les articles
@@ -479,7 +479,7 @@ 
 	);
 
 	while ($row = sql_fetch($r)) {
-		sql_update('spip_articles', ['id_secteur' => $row['secteur']], 'id_article=' . intval($row['id']));
+		sql_update('spip_articles', ['id_secteur' => $row['secteur']], 'id_article='.intval($row['id']));
 	}
 
 	// avertir les plugins qui peuvent faire leur mises a jour egalement
@@ -510,7 +510,7 @@ 
 		$t = sql_updateq(
 			'spip_rubriques',
 			['lang' => $row['lang'], 'langue_choisie' => 'non'],
-			'id_rubrique=' . intval($id_rubrique)
+			'id_rubrique='.intval($id_rubrique)
 		);
 	}
 
@@ -555,7 +555,7 @@ 
 		sql_updateq(
 			'spip_articles',
 			['lang' => $row['lang'], 'langue_choisie' => 'non'],
-			'id_article=' . intval($id_article)
+			'id_article='.intval($id_article)
 		);
 	}
 
@@ -630,11 +630,11 @@ 
 				];
 				// generer un nom de fonction "anonyme" unique
 				do {
-					$functionname = 'f_calculer_langues_utilisees_' . $boucle->id_table . '_' . time() . '_' . random_int(0, mt_getrandmax());
+					$functionname = 'f_calculer_langues_utilisees_'.$boucle->id_table.'_'.time().'_'.random_int(0, mt_getrandmax());
 				} while (function_exists($functionname));
 				$code = calculer_boucle('calculer_langues_utilisees', $boucles);
-				$code = '$SP=0; $command=array();$command["connect"] = $connect = "' . $serveur . '"; $Pile=array(0=>array());' . "\n" . $code;
-				$code = 'function ' . $functionname . '(){' . $code . '};$res = ' . $functionname . '();';
+				$code = '$SP=0; $command=array();$command["connect"] = $connect = "'.$serveur.'"; $Pile=array(0=>array());'."\n".$code;
+				$code = 'function '.$functionname.'(){'.$code.'};$res = '.$functionname.'();';
 				$res = '';
 				eval($code);
 				$res = explode(',', $res);
@@ -736,11 +736,11 @@ 
 		$maxiter-- and $filles = sql_allfetsel(
 			'id_rubrique',
 			'spip_rubriques',
-			sql_in('id_parent', $r) . ' AND ' . sql_in('id_rubrique', $r, 'NOT')
+			sql_in('id_parent', $r).' AND '.sql_in('id_rubrique', $r, 'NOT')
 		)
 	) {
 		$r = join(',', array_column($filles, 'id_rubrique'));
-		$branche .= ',' . $r;
+		$branche .= ','.$r;
 	}
 
 	# securite pour ne pas plomber la conso memoire sur les sites prolifiques
@@ -780,7 +780,7 @@ 
 
 	if (isset($b[$id])) {
 		// Notre branche commence par la rubrique de depart si $tout=true
-		return $tout ? (strlen($b[$id]) ? $b[$id] . ",$id" : $id) : $b[$id];
+		return $tout ? (strlen($b[$id]) ? $b[$id].",$id" : $id) : $b[$id];
 	}
 
 	$hier = '';
@@ -793,11 +793,11 @@ 
 		$maxiter-- and $parents = sql_allfetsel(
 			'id_parent',
 			'spip_rubriques',
-			sql_in('id_rubrique', $ids_nouveaux_parents) . ' AND ' . sql_in('id_parent', $hier, 'NOT')
+			sql_in('id_rubrique', $ids_nouveaux_parents).' AND '.sql_in('id_parent', $hier, 'NOT')
 		)
 	) {
 		$ids_nouveaux_parents = join(',', array_column($parents, 'id_parent'));
-		$hier = $ids_nouveaux_parents . (strlen($hier) ? ',' . $hier : '');
+		$hier = $ids_nouveaux_parents.(strlen($hier) ? ','.$hier : '');
 	}
 
 	# securite pour ne pas plomber la conso memoire sur les sites prolifiques
@@ -829,7 +829,7 @@ 
 	include_spip('base/abstract_sql');
 	if ($check) {
 		$postdates = ($GLOBALS['meta']['post_dates'] == 'non') ?
-			'AND A.date <= ' . sql_quote(date('Y-m-d H:i:s')) : '';
+			'AND A.date <= '.sql_quote(date('Y-m-d H:i:s')) : '';
 
 		$r = sql_select(
 			'DISTINCT A.id_rubrique AS id',
@@ -846,7 +846,7 @@ 
 	$t = sql_fetsel(
 		'date',
 		'spip_articles',
-		"statut='publie' AND date > " . sql_quote(date('Y-m-d H:i:s')),
+		"statut='publie' AND date > ".sql_quote(date('Y-m-d H:i:s')),
 		'',
 		'date',
 		'1'
@@ -902,7 +902,7 @@ 
 		$r = sql_getfetsel(
 			'id_rubrique',
 			'spip_rubriques',
-			'titre = ' . sql_quote($titre) . ' AND id_parent=' . intval($id_parent),
+			'titre = '.sql_quote($titre).' AND id_parent='.intval($id_parent),
 			$groupby = [],
 			$orderby = [],
 			$limit = '',
@@ -938,7 +938,7 @@ 
 			sql_updateq(
 				'spip_rubriques',
 				['id_secteur' => $id_secteur, 'lang' => $lang],
-				'id_rubrique=' . intval($id_rubrique),
+				'id_rubrique='.intval($id_rubrique),
 				[],
 				$serveur
 			);

ecrire/src/Chiffrer/Password.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
 			$pass_poivre = hash_hmac('sha256', $password_clair, $key);
 			return password_verify($pass_poivre, $password_hash);
 		}
-		spip_log('Aucune clé pour vérifier le mot de passe', 'chiffrer' . _LOG_INFO_IMPORTANTE);
+		spip_log('Aucune clé pour vérifier le mot de passe', 'chiffrer'._LOG_INFO_IMPORTANTE);
 		return false;
 	}
 
@@ -59,7 +59,7 @@ 
 			$pass_poivre = hash_hmac('sha256', $password_clair, $key);
 			return password_hash($pass_poivre, PASSWORD_DEFAULT);
 		}
-		spip_log('Aucune clé pour chiffrer le mot de passe', 'chiffrer' . _LOG_INFO_IMPORTANTE);
+		spip_log('Aucune clé pour chiffrer le mot de passe', 'chiffrer'._LOG_INFO_IMPORTANTE);
 		return null;
 	}
 

ecrire/src/Chiffrer/Cles.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	public function generate(string $name): string {
 		$key = Chiffrement::keygen();
 		$this->keys[$name] = $key;
-		spip_log("Création de la cle $name", 'chiffrer' . _LOG_INFO_IMPORTANTE);
+		spip_log("Création de la cle $name", 'chiffrer'._LOG_INFO_IMPORTANTE);
 		return $key;
 	}
 

ecrire/inc/queue.php

Spacing +37 added lines, -38 removed lines

@@ -70,7 +70,7 @@ 
 	$md5args = md5($arguments);
 
 	// si pas de date programee, des que possible
-	$duplicate_where = 'status=' . intval(_JQ_SCHEDULED) . ' AND ';
+	$duplicate_where = 'status='.intval(_JQ_SCHEDULED).' AND ';
 	if (!$time) {
 		$time = time();
 		$duplicate_where = ''; // ne pas dupliquer si deja le meme job en cours d'execution
@@ -96,9 +96,8 @@ 
 			'id_job',
 			'spip_jobs',
 			$duplicate_where =
-				$duplicate_where . 'fonction=' . sql_quote($function)
-				. (($no_duplicate === 'function_only') ? '' :
-			' AND md5args=' . sql_quote($md5args) . ' AND inclure=' . sql_quote($file))
+				$duplicate_where.'fonction='.sql_quote($function)
+				. (($no_duplicate === 'function_only') ? '' : ' AND md5args='.sql_quote($md5args).' AND inclure='.sql_quote($file))
 		)
 	) {
 		return $id_job;
@@ -111,9 +110,9 @@ 
 	if (
 		$no_duplicate
 		and
-		$id_prev = sql_getfetsel('id_job', 'spip_jobs', 'id_job<' . intval($id_job) . " AND $duplicate_where")
+		$id_prev = sql_getfetsel('id_job', 'spip_jobs', 'id_job<'.intval($id_job)." AND $duplicate_where")
 	) {
-		sql_delete('spip_jobs', 'id_job=' . intval($id_job));
+		sql_delete('spip_jobs', 'id_job='.intval($id_job));
 
 		return $id_prev;
 	}
@@ -125,9 +124,9 @@ 
 	// ie cas d'un char non acceptables sur certains type de champs
 	// qui coupe la valeur
 	if (defined('_JQ_INSERT_CHECK_ARGS') and $id_job) {
-		$args = sql_getfetsel('args', 'spip_jobs', 'id_job=' . intval($id_job));
+		$args = sql_getfetsel('args', 'spip_jobs', 'id_job='.intval($id_job));
 		if ($args !== $arguments) {
-			spip_log('arguments job errones / longueur ' . strlen($args) . ' vs ' . strlen($arguments) . ' / valeur : ' . var_export(
+			spip_log('arguments job errones / longueur '.strlen($args).' vs '.strlen($arguments).' / valeur : '.var_export(
 				$arguments,
 				true
 			), 'queue');
@@ -157,7 +156,7 @@ 
 function queue_purger() {
 	include_spip('base/abstract_sql');
 	sql_delete('spip_jobs');
-	sql_delete('spip_jobs_liens', 'id_job NOT IN (' . sql_get_select('id_job', 'spip_jobs') . ')');
+	sql_delete('spip_jobs_liens', 'id_job NOT IN ('.sql_get_select('id_job', 'spip_jobs').')');
 	include_spip('inc/genie');
 	genie_queue_watch_dist();
 }
@@ -173,8 +172,8 @@ 
 	include_spip('base/abstract_sql');
 
 	if (
-		$row = sql_fetsel('fonction,inclure,date', 'spip_jobs', 'id_job=' . intval($id_job))
-		and $res = sql_delete('spip_jobs', 'id_job=' . intval($id_job))
+		$row = sql_fetsel('fonction,inclure,date', 'spip_jobs', 'id_job='.intval($id_job))
+		and $res = sql_delete('spip_jobs', 'id_job='.intval($id_job))
 	) {
 		queue_unlink_job($id_job);
 		// est-ce une tache cron qu'il faut relancer ?
@@ -224,7 +223,7 @@ 
  *  resultat du sql_delete
  */
 function queue_unlink_job($id_job) {
-	return sql_delete('spip_jobs_liens', 'id_job=' . intval($id_job));
+	return sql_delete('spip_jobs_liens', 'id_job='.intval($id_job));
 }
 
 /**
@@ -240,7 +239,7 @@ 
 	// deserialiser les arguments
 	$args = unserialize($row['args']);
 	if (!is_array($args)) {
-		spip_log('arguments job errones ' . var_export($row, true), 'queue');
+		spip_log('arguments job errones '.var_export($row, true), 'queue');
 		$args = [];
 	}
 
@@ -257,14 +256,14 @@ 
 	}
 
 	if (!function_exists($fonction)) {
-		spip_log("fonction $fonction ($inclure) inexistante " . var_export($row, true), 'queue');
+		spip_log("fonction $fonction ($inclure) inexistante ".var_export($row, true), 'queue');
 
 		return false;
 	}
 
-	spip_log('queue [' . $row['id_job'] . "]: $fonction() start", 'queue');
+	spip_log('queue ['.$row['id_job']."]: $fonction() start", 'queue');
 	$res = $fonction(...$args);
-	spip_log('queue [' . $row['id_job'] . "]: $fonction() end", 'queue');
+	spip_log('queue ['.$row['id_job']."]: $fonction() end", 'queue');
 
 	return $res;
 }
@@ -295,14 +294,14 @@ 
 function queue_schedule($force_jobs = null) {
 	$time = time();
 	if (defined('_DEBUG_BLOCK_QUEUE')) {
-		spip_log('_DEBUG_BLOCK_QUEUE : schedule stop', 'jq' . _LOG_DEBUG);
+		spip_log('_DEBUG_BLOCK_QUEUE : schedule stop', 'jq'._LOG_DEBUG);
 
 		return;
 	}
 
 	// rien a faire si le prochain job est encore dans le futur
 	if (queue_sleep_time_to_next_job() > 0 and (!$force_jobs or !count($force_jobs))) {
-		spip_log('queue_sleep_time_to_next_job', 'jq' . _LOG_DEBUG);
+		spip_log('queue_sleep_time_to_next_job', 'jq'._LOG_DEBUG);
 
 		return;
 	}
@@ -323,7 +322,7 @@ 
 	}
 	$end_time = $time + _JQ_MAX_JOBS_TIME_TO_EXECUTE;
 
-	spip_log("JQ schedule $time / $end_time", 'jq' . _LOG_DEBUG);
+	spip_log("JQ schedule $time / $end_time", 'jq'._LOG_DEBUG);
 
 	if (!defined('_JQ_MAX_JOBS_EXECUTE')) {
 		define('_JQ_MAX_JOBS_EXECUTE', 200);
@@ -337,19 +336,19 @@ 
 	// lorsqu'un job cron n'a pas fini, sa priorite est descendue
 	// pour qu'il ne bloque pas les autres jobs en attente
 	if (is_array($force_jobs) and count($force_jobs)) {
-		$cond = 'status=' . intval(_JQ_SCHEDULED) . ' AND ' . sql_in('id_job', $force_jobs);
+		$cond = 'status='.intval(_JQ_SCHEDULED).' AND '.sql_in('id_job', $force_jobs);
 	} else {
 		$now = date('Y-m-d H:i:s', $time);
-		$cond = 'status=' . intval(_JQ_SCHEDULED) . ' AND date<=' . sql_quote($now);
+		$cond = 'status='.intval(_JQ_SCHEDULED).' AND date<='.sql_quote($now);
 	}
 
 	register_shutdown_function('queue_error_handler'); // recuperer les erreurs auant que possible
-	$res = sql_allfetsel('*', 'spip_jobs', $cond, '', 'priorite DESC,date', '0,' . (_JQ_MAX_JOBS_EXECUTE + 1));
+	$res = sql_allfetsel('*', 'spip_jobs', $cond, '', 'priorite DESC,date', '0,'.(_JQ_MAX_JOBS_EXECUTE + 1));
 	do {
 		if ($row = array_shift($res)) {
 			$nbj++;
 			// il faut un verrou, a base de sql_delete
-			if (sql_delete('spip_jobs', 'id_job=' . intval($row['id_job']) . ' AND status=' . intval(_JQ_SCHEDULED))) {
+			if (sql_delete('spip_jobs', 'id_job='.intval($row['id_job']).' AND status='.intval(_JQ_SCHEDULED))) {
 				#spip_log("JQ schedule job ".$nbj." OK",'jq');
 				// on reinsert dans la base aussitot avec un status=_JQ_PENDING
 				$row['status'] = _JQ_PENDING;
@@ -364,13 +363,13 @@ 
 				queue_close_job($row, $time, $result);
 			}
 		}
-		spip_log('JQ schedule job end time ' . $time, 'jq' . _LOG_DEBUG);
+		spip_log('JQ schedule job end time '.$time, 'jq'._LOG_DEBUG);
 	} while ($nbj < _JQ_MAX_JOBS_EXECUTE and $row and $time < $end_time);
-	spip_log('JQ schedule end time ' . time(), 'jq' . _LOG_DEBUG);
+	spip_log('JQ schedule end time '.time(), 'jq'._LOG_DEBUG);
 
 	if ($row = array_shift($res)) {
 		queue_update_next_job_time(0); // on sait qu'il y a encore des jobs a lancer ASAP
-		spip_log('JQ encore !', 'jq' . _LOG_DEBUG);
+		spip_log('JQ encore !', 'jq'._LOG_DEBUG);
 	} else {
 		queue_update_next_job_time();
 	}
@@ -405,9 +404,9 @@ 
 		}
 	}
 	// purger ses liens eventuels avec des objets
-	sql_delete('spip_jobs_liens', 'id_job=' . intval($row['id_job']));
+	sql_delete('spip_jobs_liens', 'id_job='.intval($row['id_job']));
 	// supprimer le job fini
-	sql_delete('spip_jobs', 'id_job=' . intval($row['id_job']));
+	sql_delete('spip_jobs', 'id_job='.intval($row['id_job']));
 }
 
 /**
@@ -480,18 +479,18 @@ 
 	$res = sql_allfetsel(
 		'*',
 		'spip_jobs',
-		'status=' . intval(_JQ_PENDING) . ' AND date<' . sql_quote(date('Y-m-d H:i:s', $time - 180))
+		'status='.intval(_JQ_PENDING).' AND date<'.sql_quote(date('Y-m-d H:i:s', $time - 180))
 	);
 	if (is_array($res)) {
 		foreach ($res as $row) {
 			queue_close_job($row, $time);
-			spip_log('queue_close_job car _JQ_PENDING depuis +180s : ' . print_r($row, 1), 'job_mort' . _LOG_ERREUR);
+			spip_log('queue_close_job car _JQ_PENDING depuis +180s : '.print_r($row, 1), 'job_mort'._LOG_ERREUR);
 		}
 	}
 
 	// chercher la date du prochain job si pas connu
 	if (is_null($next) or is_null(queue_sleep_time_to_next_job())) {
-		$date = sql_getfetsel('date', 'spip_jobs', 'status=' . intval(_JQ_SCHEDULED), '', 'date', '0,1');
+		$date = sql_getfetsel('date', 'spip_jobs', 'status='.intval(_JQ_SCHEDULED), '', 'date', '0,1');
 		$next = strtotime($date);
 	}
 	if (!is_null($next_time)) {
@@ -504,7 +503,7 @@ 
 		if (is_null($nb_jobs_scheduled)) {
 			$nb_jobs_scheduled = sql_countsel(
 				'spip_jobs',
-				'status=' . intval(_JQ_SCHEDULED) . ' AND date<' . sql_quote(date('Y-m-d H:i:s', $time))
+				'status='.intval(_JQ_SCHEDULED).' AND date<'.sql_quote(date('Y-m-d H:i:s', $time))
 			);
 		} elseif ($next <= $time) {
 			$nb_jobs_scheduled++;
@@ -573,7 +572,7 @@ 
 	}
 
 	// ne pas relancer si on vient de lancer dans la meme seconde par un hit concurent
-	if (file_exists($lock = _DIR_TMP . 'cron.lock') and !(@filemtime($lock) < $_SERVER['REQUEST_TIME'])) {
+	if (file_exists($lock = _DIR_TMP.'cron.lock') and !(@filemtime($lock) < $_SERVER['REQUEST_TIME'])) {
 		return $texte;
 	}
 
@@ -645,7 +644,7 @@ 
 				$port = 80;
 		}
 		$fp = @fsockopen(
-			$scheme . $parts['host'],
+			$scheme.$parts['host'],
 			$parts['port'] ?? $port,
 			$errno,
 			$errstr,
@@ -655,13 +654,13 @@ 
 		if ($fp) {
 			$host_sent = $parts['host'];
 			if (isset($parts['port']) and $parts['port'] !== $port) {
-				$host_sent .= ':' . $parts['port'];
+				$host_sent .= ':'.$parts['port'];
 			}
 			$timeout = 200; // ms
 			stream_set_timeout($fp, 0, $timeout * 1000);
-			$query = $parts['path'] . ($parts['query'] ? '?' . $parts['query'] : '');
-			$out = 'GET ' . $query . " HTTP/1.1\r\n";
-			$out .= 'Host: ' . $host_sent . "\r\n";
+			$query = $parts['path'].($parts['query'] ? '?'.$parts['query'] : '');
+			$out = 'GET '.$query." HTTP/1.1\r\n";
+			$out .= 'Host: '.$host_sent."\r\n";
 			$out .= "Connection: Close\r\n\r\n";
 			fwrite($fp, $out);
 			spip_timer('read');

ecrire/action/editer_rubrique.php

Spacing +2 added lines, -2 removed lines

@@ -150,7 +150,7 @@ 
 			$id_rubrique,
 			[
 			'data' => $set,
-			'nonvide' => ['titre' => _T('titre_nouvelle_rubrique') . ' ' . _T('info_numero_abbreviation') . $id_rubrique]
+			'nonvide' => ['titre' => _T('titre_nouvelle_rubrique').' '._T('info_numero_abbreviation').$id_rubrique]
 			],
 			$c
 		)
@@ -254,7 +254,7 @@ 
 				)
 			) {
 				if ($s['statut'] != 'prepa') {
-					spip_log("deplacement de $id_rubrique vers $id_parent refuse a " . $GLOBALS['visiteur_session']['id_auteur'] . ' ' . $GLOBALS['visiteur_session']['statut']);
+					spip_log("deplacement de $id_rubrique vers $id_parent refuse a ".$GLOBALS['visiteur_session']['id_auteur'].' '.$GLOBALS['visiteur_session']['statut']);
 				}
 			} elseif (editer_rubrique_breves($id_rubrique, $id_parent, $c)) {
 				$statut_ancien = $s['statut'];

ecrire/inc/filtres_dates.php

Spacing +33 added lines, -33 removed lines

@@ -39,7 +39,7 @@ 
 function extraire_date($texte): string {
 	// format = 2001-08
 	if (preg_match(',([1-2][0-9]{3})[^0-9]*(1[0-2]|0?[1-9]),', $texte, $regs)) {
-		return $regs[1] . '-' . sprintf('%02d', $regs[2]) . '-01';
+		return $regs[1].'-'.sprintf('%02d', $regs[2]).'-01';
 	}
 	return '';
 }
@@ -70,11 +70,11 @@ 
 		}
 		if (preg_match('#^([12][0-9]{3})([-/]00)?( [-0-9:]+)?$#', $date, $regs)) {
 			$regs = array_pad($regs, 4, null); // eviter notice php
-			$date = $regs[1] . '-00-00' . $regs[3];
+			$date = $regs[1].'-00-00'.$regs[3];
 		} else {
 			if (preg_match('#^([12][0-9]{3}[-/][01]?[0-9])([-/]00)?( [-0-9:]+)?$#', $date, $regs)) {
 				$regs = array_pad($regs, 4, null); // eviter notice php
-				$date = preg_replace('@/@', '-', $regs[1]) . '-00' . $regs[3];
+				$date = preg_replace('@/@', '-', $regs[1]).'-00'.$regs[3];
 			} else {
 				$date = date('Y-m-d H:i:s', strtotime($date));
 			}
@@ -101,7 +101,7 @@ 
 	$letexte ??= '';
 	if (
 		!$verif_format_date
-		or (in_array(strlen($letexte), [10,19]) and
+		or (in_array(strlen($letexte), [10, 19]) and
 			  preg_match('/^[0-9]{4}-[0-9]{2}-[0-9]{2}(\s[0-9]{2}:[0-9]{2}:[0-9]{2})?$/', $letexte))
 	) {
 		if (strncmp('0000-00-00', $letexte, 10) == 0) {
@@ -388,17 +388,17 @@ 
 	if ($decal > 3600 * 24 * 30) {
 		$mois = floor($decal / (3600 * 24 * 30));
 		if ($mois < 2) {
-			$delai = "$mois " . _T('date_un_mois');
+			$delai = "$mois "._T('date_un_mois');
 		} else {
-			$delai = "$mois " . _T('date_mois');
+			$delai = "$mois "._T('date_mois');
 		}
 	} else {
 		if ($decal > 3600 * 24 * 7) {
 			$semaines = floor($decal / (3600 * 24 * 7));
 			if ($semaines < 2) {
-				$delai = "$semaines " . _T('date_une_semaine');
+				$delai = "$semaines "._T('date_une_semaine');
 			} else {
-				$delai = "$semaines " . _T('date_semaines');
+				$delai = "$semaines "._T('date_semaines');
 			}
 		} else {
 			if ($decal > 3600 * 24) {
@@ -406,30 +406,30 @@ 
 				if ($jours < 2) {
 					return $il_y_a == 'date_dans' ? _T('date_demain') : _T('date_hier');
 				} else {
-					$delai = "$jours " . _T('date_jours');
+					$delai = "$jours "._T('date_jours');
 				}
 			} else {
 				if ($decal >= 3600) {
 					$heures = floor($decal / 3600);
 					if ($heures < 2) {
-						$delai = "$heures " . _T('date_une_heure');
+						$delai = "$heures "._T('date_une_heure');
 					} else {
-						$delai = "$heures " . _T('date_heures');
+						$delai = "$heures "._T('date_heures');
 					}
 				} else {
 					if ($decal >= 60) {
 						$minutes = floor($decal / 60);
 						if ($minutes < 2) {
-							$delai = "$minutes " . _T('date_une_minute');
+							$delai = "$minutes "._T('date_une_minute');
 						} else {
-							$delai = "$minutes " . _T('date_minutes');
+							$delai = "$minutes "._T('date_minutes');
 						}
 					} else {
 						$secondes = ceil($decal);
 						if ($secondes < 2) {
-							$delai = "$secondes " . _T('date_une_seconde');
+							$delai = "$secondes "._T('date_une_seconde');
 						} else {
-							$delai = "$secondes " . _T('date_secondes');
+							$delai = "$secondes "._T('date_secondes');
 						}
 					}
 				}
@@ -522,7 +522,7 @@ 
 		$njour = 0;
 	} else {
 		$njour = intval($jour);
-		if ($jourth = _T('date_jnum' . $jour)) {
+		if ($jourth = _T('date_jnum'.$jour)) {
 			$jour = $jourth;
 		}
 	}
@@ -530,10 +530,10 @@ 
 	$mois = intval($mois);
 	if ($mois > 0 and $mois < 13) {
 		/* Traiter le cas "abbr" pour les noms de mois */
-		$param = ((isset($options['param']) and $options['param'] === 'abbr') ? '_' . $options['param'] : '');
-		$nommois = _T('date_mois_' . $mois . $param);
+		$param = ((isset($options['param']) and $options['param'] === 'abbr') ? '_'.$options['param'] : '');
+		$nommois = _T('date_mois_'.$mois.$param);
 		if ($jour) {
-			$jourmois = _T('date_de_mois_' . $mois, ['j' => $jour, 'nommois' => $nommois]);
+			$jourmois = _T('date_de_mois_'.$mois, ['j' => $jour, 'nommois' => $nommois]);
 		} else {
 			$jourmois = $nommois;
 		}
@@ -543,7 +543,7 @@ 
 	}
 
 	if ($annee < 0) {
-		$annee = -$annee . ' ' . _T('date_avant_jc');
+		$annee = -$annee.' '._T('date_avant_jc');
 		$avjc = true;
 	} else {
 		$avjc = false;
@@ -569,11 +569,11 @@ 
 				}
 			}
 			if ($vue == 'saison') {
-				return $saison ? _T('date_saison_' . $saison) : '';
+				return $saison ? _T('date_saison_'.$saison) : '';
 			} else {
 				return $saison ? trim(_T(
 					'date_fmt_saison_annee',
-					['saison' => _T('date_saison_' . $saison), 'annee' => $annee]
+					['saison' => _T('date_saison_'.$saison), 'annee' => $annee]
 				)) : '';
 			}
 
@@ -650,9 +650,9 @@ 
 			}
 			$nom = mktime(1, 1, 1, $mois, $njour, $annee);
 			$nom = 1 + (int) date('w', $nom);
-			$param = ((isset($options['param']) and $options['param']) ? '_' . $options['param'] : '');
+			$param = ((isset($options['param']) and $options['param']) ? '_'.$options['param'] : '');
 
-			return _T('date_jour_' . $nom . $param);
+			return _T('date_jour_'.$nom.$param);
 
 		case 'mois_annee':
 			if ($avjc) {
@@ -1047,8 +1047,8 @@ 
 
 	$dtstart = $dtend = $dtabbr = '';
 	if (strpos($forme, 'hcal') !== false) {
-		$dtstart = "<abbr class='dtstart' title='" . date_iso($date_debut) . "'>";
-		$dtend = "<abbr class='dtend' title='" . date_iso($date_fin) . "'>";
+		$dtstart = "<abbr class='dtstart' title='".date_iso($date_debut)."'>";
+		$dtend = "<abbr class='dtend' title='".date_iso($date_fin)."'>";
 		$dtabbr = '</abbr>';
 	}
 
@@ -1096,7 +1096,7 @@ 
 			}
 		} else {
 			if ($dtabbr && $dtstart) {
-				$s = $dtstart . spip_ucfirst($s) . $dtabbr;
+				$s = $dtstart.spip_ucfirst($s).$dtabbr;
 			} else {
 				$s = spip_ucfirst($s);
 			}
@@ -1119,8 +1119,8 @@ 
 				$date_debut = _T('date_fmt_jour_heure', ['jour' => $date_debut, 'heure' => $hd]);
 				$date_fin = _T('date_fmt_jour_heure', ['jour' => $date_fin, 'heure' => $hf]);
 			}
-			$date_debut = $dtstart . $date_debut . $dtabbr;
-			$date_fin = $dtend . $date_fin . $dtabbr;
+			$date_debut = $dtstart.$date_debut.$dtabbr;
+			$date_fin = $dtend.$date_fin.$dtabbr;
 
 			$s = _T('date_fmt_periode', ['date_debut' => $date_debut, 'date_fin' => $date_fin]);
 		} else {
@@ -1137,8 +1137,8 @@ 
 				$date_fin = _T('date_fmt_jour_heure', ['jour' => $date_fin, 'heure' => $hf]);
 			}
 
-			$date_debut = $dtstart . $date_debut . $dtabbr;
-			$date_fin = $dtend . $date_fin . $dtabbr;
+			$date_debut = $dtstart.$date_debut.$dtabbr;
+			$date_fin = $dtend.$date_fin.$dtabbr;
 			$s = _T('date_fmt_periode', ['date_debut' => $date_debut, 'date_fin' => $date_fin]);
 		}
 	}
@@ -1242,7 +1242,7 @@ 
 		$d = date('Y-m-d');
 	}
 
-	return substr($d, 0, 4) . substr($d, 5, 2) . substr($d, 8, 2);
+	return substr($d, 0, 4).substr($d, 5, 2).substr($d, 8, 2);
 }
 
 /**
@@ -1262,7 +1262,7 @@ 
 		$d = date('Y-m-d');
 	}
 
-	return substr($d, 0, 4) . substr($d, 5, 2);
+	return substr($d, 0, 4).substr($d, 5, 2);
 }
 
 /**

ecrire/inc/acces.php

Spacing +22 added lines, -22 removed lines

@@ -31,7 +31,7 @@ 
  *     Mot de passe
  **/
 function creer_pass_aleatoire($longueur = 16, $sel = '') {
-	$seed = (int)round(((float)microtime() + 1) * time());
+	$seed = (int) round(((float) microtime() + 1) * time());
 
 	mt_srand($seed);
 	$s = '';
@@ -42,9 +42,9 @@ 
 			if (!$s) {
 				$s = random_int(0, mt_getrandmax());
 			}
-			$s = substr(md5(uniqid($s) . $sel), 0, 16);
+			$s = substr(md5(uniqid($s).$sel), 0, 16);
 		}
-		$r = unpack('Cr', pack('H2', $s . $s));
+		$r = unpack('Cr', pack('H2', $s.$s));
 		$x = $r['r'] & 63;
 		if ($x < 10) {
 			$x = chr($x + 48);
@@ -82,7 +82,7 @@ 
 	static $seeded;
 
 	if (!$seeded) {
-		$seed = (int)round(((float)microtime() + 1) * time());
+		$seed = (int) round(((float) microtime() + 1) * time());
 		mt_srand($seed);
 		$seeded = true;
 	}
@@ -168,10 +168,10 @@ 
 			ecrire_meta('low_sec', $low_sec = creer_pass_aleatoire());
 		}
 	} else {
-		$low_sec = sql_getfetsel('low_sec', 'spip_auteurs', 'id_auteur = ' . intval($id_auteur));
+		$low_sec = sql_getfetsel('low_sec', 'spip_auteurs', 'id_auteur = '.intval($id_auteur));
 		if (!$low_sec) {
 			$low_sec = creer_pass_aleatoire();
-			sql_updateq('spip_auteurs', ['low_sec' => $low_sec], 'id_auteur = ' . intval($id_auteur));
+			sql_updateq('spip_auteurs', ['low_sec' => $low_sec], 'id_auteur = '.intval($id_auteur));
 		}
 	}
 
@@ -229,7 +229,7 @@ 
 function generer_url_api_low_sec(string $script, string $format, string $fond, string $path, string $args, bool $no_entities = false, ?bool $public = null) {
 	$id_auteur = $GLOBALS['visiteur_session']['id_auteur'] ?? 0;
 	$cle = afficher_low_sec($id_auteur, "$script/$format $fond $args");
-	$path = "$id_auteur/$cle/$format/$fond" . ($path ? "/$path" : '');
+	$path = "$id_auteur/$cle/$format/$fond".($path ? "/$path" : '');
 
 	return generer_url_api($script, $path, $args, $no_entities = false, $public);
 }
@@ -253,9 +253,9 @@ 
 	foreach ($args as $val => $var) {
 		if ($var) {
 			if ($val <> 'statut') {
-				$a .= ':' . $val . '-' . $var;
+				$a .= ':'.$val.'-'.$var;
 			}
-			$b .= $val . '=' . $var . '&';
+			$b .= $val.'='.$var.'&';
 		}
 	}
 	$a = substr($a, 1);
@@ -285,7 +285,7 @@ 
  *     Clé
  **/
 function afficher_low_sec($id_auteur, $action = '') {
-	return substr(md5($action . low_sec($id_auteur)), 0, 8);
+	return substr(md5($action.low_sec($id_auteur)), 0, 8);
 }
 
 /**
@@ -316,7 +316,7 @@ 
 	if (!$id_auteur = intval($id_auteur)) {
 		return;
 	} // jamais trop prudent ;)
-	sql_updateq('spip_auteurs', ['low_sec' => ''], 'id_auteur = ' . intval($id_auteur));
+	sql_updateq('spip_auteurs', ['low_sec' => ''], 'id_auteur = '.intval($id_auteur));
 }
 
 
@@ -334,8 +334,8 @@ 
  *     - void sinon.
  **/
 function ecrire_acces() {
-	$htaccess = _DIR_RESTREINT . _ACCESS_FILE_NAME;
-	$htpasswd = _DIR_TMP . _AUTH_USER_FILE;
+	$htaccess = _DIR_RESTREINT._ACCESS_FILE_NAME;
+	$htpasswd = _DIR_TMP._AUTH_USER_FILE;
 
 	// Cette variable de configuration peut etre posee par un plugin
 	// par exemple acces_restreint ;
@@ -346,7 +346,7 @@ 
 		and !@file_exists($htaccess)
 	) {
 		spip_unlink($htpasswd);
-		spip_unlink($htpasswd . '-admin');
+		spip_unlink($htpasswd.'-admin');
 		return;
 	}
 
@@ -374,10 +374,10 @@ 
 	$pwd_all = ''; // login:htpass pour tous
 	$pwd_admin = ''; // login:htpass pour les admins
 
-	$res = sql_select('login, htpass, statut', 'spip_auteurs', "htpass!='' AND login!='' AND " . sql_in('statut', ['1comite', '0minirezo', 'nouveau']));
+	$res = sql_select('login, htpass, statut', 'spip_auteurs', "htpass!='' AND login!='' AND ".sql_in('statut', ['1comite', '0minirezo', 'nouveau']));
 	while ($row = sql_fetch($res)) {
 		if (strlen($row['login']) and strlen($row['htpass'])) {
-			$ligne = $row['login'] . ':' . $row['htpass'] . "\n";
+			$ligne = $row['login'].':'.$row['htpass']."\n";
 			$pwd_all .= $ligne;
 			if ($row['statut'] == '0minirezo') {
 				$pwd_admin .= $ligne;
@@ -420,7 +420,7 @@ 
  * @return boolean
  */
 function verifier_htaccess($rep, $force = false) {
-	$htaccess = rtrim($rep, '/') . '/' . _ACCESS_FILE_NAME;
+	$htaccess = rtrim($rep, '/').'/'._ACCESS_FILE_NAME;
 	if (((@file_exists($htaccess)) or defined('_TEST_DIRS')) and !$force) {
 		return true;
 	}
@@ -449,17 +449,17 @@ 
 		fputs($ht, $deny);
 		fclose($ht);
 		@chmod($htaccess, _SPIP_CHMOD & 0666);
-		$t = rtrim($rep, '/') . '/.ok';
+		$t = rtrim($rep, '/').'/.ok';
 		if ($ht = @fopen($t, 'w')) {
 			@fclose($ht);
 			include_spip('inc/distant');
 			$t = substr($t, strlen(_DIR_RACINE));
-			$t = url_de_base() . $t;
+			$t = url_de_base().$t;
 			$ht = recuperer_url($t, ['methode' => 'HEAD', 'taille_max' => 0, 'follow_location' => false]);
 			$ht = ($ht['status'] ?? null) === 403;
 		}
 	}
-	spip_log("Creation de $htaccess " . ($ht ? ' reussie' : ' manquee'));
+	spip_log("Creation de $htaccess ".($ht ? ' reussie' : ' manquee'));
 
 	return $ht;
 }
@@ -485,11 +485,11 @@ 
 	$dirs = sql_allfetsel('extension', 'spip_types_documents');
 	$dirs[] = ['extension' => 'distant'];
 	foreach ($dirs as $e) {
-		if (is_dir($dir = _DIR_IMG . $e['extension'])) {
+		if (is_dir($dir = _DIR_IMG.$e['extension'])) {
 			if ($f) {
 				verifier_htaccess($dir);
 			} else {
-				spip_unlink($dir . '/' . _ACCESS_FILE_NAME);
+				spip_unlink($dir.'/'._ACCESS_FILE_NAME);
 			}
 		}
 	}