Inspection of "docs(changelog): pour #5806" - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ffbbca...ad9227 )

by cam

created 2023-12-22 10:10 UTC

Status

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -10,7 +10,7 @@
 block discarded – undo
 {
 	public static function setUpBeforeClass(): void {
 		include_spip('inc/autoriser');
-		require_once(__DIR__ . '/data/autoriser.php');
+		require_once(__DIR__.'/data/autoriser.php');
 	}
 
 	public function testAutoriserOkNiet(): void {

Please login to merge, or discard this patch.

ecrire/tests/Format/Xml/SpipXmlParseTest.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -25,7 +25,7 @@  discard block
 block discarded – undo
 
 	public static function providerXmlSpipXmlParse(): array {
 		$essais = [];
-		$xml1 = '<' . <<<CODE_SAMPLE
+		$xml1 = '<'.<<<CODE_SAMPLE
 ?xml version="1.0" encoding="utf-8"?>
 <urlset xmlns="http://www.google.com/schemas/sitemap/0.84">
 <url><loc>http://localhost/_core/spip/spip.php?breve25</loc>
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
 		$tree1 = <<<CODE_SAMPLE
 a:1:{s:57:"urlset xmlns="http://www.google.com/schemas/sitemap/0.84"";a:1:{i:0;a:1:{s:3:"url";a:3:{i:0;a:3:{s:3:"loc";a:1:{i:0;s:44:"http://localhost/_core/spip/spip.php?breve25";}s:7:"lastmod";a:1:{i:0;s:10:"2003-12-31";}s:8:"priority";a:1:{i:0;s:3:"0.8";}}i:1;a:3:{s:3:"loc";a:1:{i:0;s:44:"http://localhost/_core/spip/spip.php?breve32";}s:7:"lastmod";a:1:{i:0;s:10:"2004-02-10";}s:8:"priority";a:1:{i:0;s:3:"0.8";}}i:2;a:3:{s:3:"loc";a:1:{i:0;s:44:"http://localhost/_core/spip/spip.php?breve64";}s:7:"lastmod";a:1:{i:0;s:10:"2005-01-31";}s:8:"priority";a:1:{i:0;s:3:"0.8";}}}}}}
 CODE_SAMPLE;
-		$xml2 = '<' . <<<CODE_SAMPLE
+		$xml2 = '<'.<<<CODE_SAMPLE
 ?xml version="1.0" encoding="UTF-8"?>
 <opml version="1.0">
   <head>

Please login to merge, or discard this patch.

ecrire/tests/Urls/LiensAbsolusTest.php 1 patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -22,8 +22,8 @@  discard block
 block discarded – undo
 	public function testLienPrive() {
 		$relatif = generer_url_ecrire('toto', 'truc=machin&chose=bidule', false, true);
 		$absolu = generer_url_ecrire('toto', 'truc=machin&chose=bidule', false, false);
-		$expected = 'bla bla <a href=\'' . str_replace('&amp;', '&#38;', $absolu) . '\'>lien prive</a>';
-		$case = 'bla bla <a href=\'' . $relatif . '\'>lien prive</a>';
+		$expected = 'bla bla <a href=\''.str_replace('&amp;', '&#38;', $absolu).'\'>lien prive</a>';
+		$case = 'bla bla <a href=\''.$relatif.'\'>lien prive</a>';
 		$actual = liens_absolus($case);
 		$this->assertEquals($expected, $actual);
 	}
@@ -32,8 +32,8 @@  discard block
 block discarded – undo
 	public function testLienPublic() {
 		$relatif = generer_url_public('toto', 'truc=machin&chose=bidule', false, true);
 		$absolu = generer_url_public('toto', 'truc=machin&chose=bidule', false, false);
-		$expected = 'bla bla <a href=\'' . str_replace('&amp;', '&#38;', $absolu) . '\'>lien public</a>';
-		$case = 'bla bla <a href=\'' . $relatif . '\'>lien public</a>';
+		$expected = 'bla bla <a href=\''.str_replace('&amp;', '&#38;', $absolu).'\'>lien public</a>';
+		$case = 'bla bla <a href=\''.$relatif.'\'>lien public</a>';
 		$actual = liens_absolus($case);
 		$this->assertEquals($expected, $actual);
 	}

Please login to merge, or discard this patch.

ecrire/tests/Filesystem/FlockTest.php 1 patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -14,16 +14,16 @@
 block discarded – undo
 	}
 
 	public function testSousRepertoire(): void {
-		$sous_repertoire = 'test' . md5(strval(random_int(0, mt_getrandmax())));
+		$sous_repertoire = 'test'.md5(strval(random_int(0, mt_getrandmax())));
 		$this->assertSame(
 			sous_repertoire(_DIR_VAR, $sous_repertoire),
-			_DIR_VAR . $sous_repertoire . '/'
+			_DIR_VAR.$sous_repertoire.'/'
 		);
-		$this->assertTrue(file_exists(_DIR_VAR . $sous_repertoire));
-		$this->assertTrue(is_dir(_DIR_VAR . $sous_repertoire));
+		$this->assertTrue(file_exists(_DIR_VAR.$sous_repertoire));
+		$this->assertTrue(is_dir(_DIR_VAR.$sous_repertoire));
 
 		// Nettoyage
-		@unlink(_DIR_VAR . $sous_repertoire . '/.ok');
-		@rmdir(_DIR_VAR . $sous_repertoire);
+		@unlink(_DIR_VAR.$sous_repertoire.'/.ok');
+		@rmdir(_DIR_VAR.$sous_repertoire);
 	}
 }

Please login to merge, or discard this patch.

ecrire/tests/Filesystem/CreerCheminTest.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -28,7 +28,7 @@  discard block
 block discarded – undo
 
 		$this->assertIsArray($_chemins);
 		$this->assertEquals(count($chemins), count($_chemins) - 1, 'Erreur ajout chemin par la fonction _chemin() : mauvais compte');
-		if (is_dir(_DIR_RACINE . 'squelettes')) {
+		if (is_dir(_DIR_RACINE.'squelettes')) {
 			$this->assertEquals('toto/', $_chemins[1], 'Erreur ajout chemin par la fonction _chemin() : avec squelettes');
 		} else {
 			$this->assertEquals('toto/', $_chemins[0], 'Erreur ajout chemin par la fonction _chemin() : sans squelettes');
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
 		$chemins = creer_chemin();
 		$this->assertIsArray($chemins);
 
-		$squelettes = (int) is_dir(_DIR_RACINE . 'squelettes');
+		$squelettes = (int) is_dir(_DIR_RACINE.'squelettes');
 		$dossier_squelettes = count(explode(':', $GLOBALS['dossier_squelettes']));
 
 		_chemin('toto');

Please login to merge, or discard this patch.

ecrire/prive.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -11,7 +11,7 @@  discard block
 block discarded – undo
 
 // Script pour appeler un squelette apres s'etre authentifie
 
-include_once __DIR__ . '/inc_version.php';
+include_once __DIR__.'/inc_version.php';
 
 include_spip('inc/cookie');
 
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
 	// On reexecute pour deboucher sur le include public.
 	// autrement on insiste
 	if (is_array($var_auth)) {
-		$var_auth = '../?' . $_SERVER['QUERY_STRING'];
+		$var_auth = '../?'.$_SERVER['QUERY_STRING'];
 		include_spip('inc/session');
 		// on prolonge le cookie
 		set_cookie_session(null, time() + 3600 * 24 * 14);
@@ -34,4 +34,4 @@  discard block
 block discarded – undo
 }
 
 // En somme, est prive' ce qui est publiquement nomme'...
-include __DIR__ . '/public.php';
+include __DIR__.'/public.php';

Please login to merge, or discard this patch.

ecs.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -20,7 +20,7 @@  discard block
 block discarded – undo
 use Symplify\EasyCodingStandard\Config\ECSConfig;
 use Symplify\EasyCodingStandard\ValueObject\Set\SetList;
 
-return static function (ECSConfig $ecsConfig): void {
+return static function(ECSConfig $ecsConfig): void {
 	// A. full sets
 	$ecsConfig->sets([SetList::PSR_12, SetList::SYMPLIFY, SetList::COMMON, SetList::CLEAN_CODE]);
 	$ecsConfig->rule(NoExtraBlankLinesFixer::class);
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
 	$ecsConfig->indentation('tab');
 
 	$ecsConfig->paths([
-		__DIR__ . '/ecrire/tests',
+		__DIR__.'/ecrire/tests',
 		# __DIR__ . '/index.php',
 		# __DIR__ . '/spip.php',
 		# __DIR__ . '/ecrire',
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
 	]);
 
 	$ecsConfig->skip([
-		__DIR__ . '/ecrire/lang',
+		__DIR__.'/ecrire/lang',
 		ArrayListItemNewlineFixer::class,
 		ArrayOpenerAndCloserNewlineFixer::class,
 		AssignmentInConditionSniff::class,

Please login to merge, or discard this patch.

ecrire/action/inscrire_auteur.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -58,7 +58,7 @@  discard block
 block discarded – undo
 	}
 
 	include_spip('base/abstract_sql');
-	$res = sql_select('statut, id_auteur, login, email, nom', 'spip_auteurs', 'email=' . sql_quote($desc['email']));
+	$res = sql_select('statut, id_auteur, login, email, nom', 'spip_auteurs', 'email='.sql_quote($desc['email']));
 	// erreur ?
 	if (!$res) {
 		return _T('titre_probleme_technique');
@@ -209,11 +209,11 @@  discard block
 block discarded – undo
 
 	$login = $login_base;
 
-	for ($i = 1;; $i++) {
+	for ($i = 1; ; $i++) {
 		if (!sql_countsel('spip_auteurs', "login='$login'")) {
 			return $login;
 		}
-		$login = $login_base . $i;
+		$login = $login_base.$i;
 	}
 }
 
@@ -353,11 +353,11 @@  discard block
 block discarded – undo
 	do {
 		// Un morceau du jeton est lisible en bdd pour éviter de devoir déchiffrer
 		// tous les jetons connus pour vérifier le jeton d’un auteur.
-		$public = substr((string) creer_uniqid(), 0, 7) . '.';
-		$jeton = $public . creer_uniqid();
-		$jeton_chiffre_prefixe = $public . Chiffrement::chiffrer($jeton, SpipCles::secret_du_site());
-		sql_updateq('spip_auteurs', ['cookie_oubli' => $jeton_chiffre_prefixe], 'id_auteur=' . (int) $id_auteur);
-	} while (sql_countsel('spip_auteurs', 'cookie_oubli=' . sql_quote($jeton_chiffre_prefixe, '', 'string')) > 1);
+		$public = substr((string) creer_uniqid(), 0, 7).'.';
+		$jeton = $public.creer_uniqid();
+		$jeton_chiffre_prefixe = $public.Chiffrement::chiffrer($jeton, SpipCles::secret_du_site());
+		sql_updateq('spip_auteurs', ['cookie_oubli' => $jeton_chiffre_prefixe], 'id_auteur='.(int) $id_auteur);
+	} while (sql_countsel('spip_auteurs', 'cookie_oubli='.sql_quote($jeton_chiffre_prefixe, '', 'string')) > 1);
 
 	return $jeton;
 }
@@ -374,7 +374,7 @@  discard block
 block discarded – undo
  */
 function auteur_lire_jeton(int $id_auteur, bool $autoInit = false): ?string {
 	include_spip('base/abstract_sql');
-	$jeton_chiffre_prefixe = sql_getfetsel('cookie_oubli', 'spip_auteurs', 'id_auteur=' . $id_auteur);
+	$jeton_chiffre_prefixe = sql_getfetsel('cookie_oubli', 'spip_auteurs', 'id_auteur='.$id_auteur);
 	if ($jeton_chiffre_prefixe) {
 		$jeton_chiffre = substr((string) $jeton_chiffre_prefixe, 8);
 		$jeton = Chiffrement::dechiffrer($jeton_chiffre, SpipCles::secret_du_site());
@@ -404,13 +404,13 @@  discard block
 block discarded – undo
 	$public = substr($jeton, 0, 8);
 
 	// Les auteurs qui ont un jetons ressemblant
-	$auteurs = sql_allfetsel('*', 'spip_auteurs', 'cookie_oubli LIKE ' . sql_quote($public . '%'));
+	$auteurs = sql_allfetsel('*', 'spip_auteurs', 'cookie_oubli LIKE '.sql_quote($public.'%'));
 	foreach ($auteurs as $auteur) {
 		$jeton_chiffre = substr((string) $auteur['cookie_oubli'], 8);
 		try {
 			$_jeton = Chiffrement::dechiffrer($jeton_chiffre, SpipCles::secret_du_site());
 		} catch (\Exception $e) {
-			spip_log('Échec du déchiffrage du jeton d’auteur: ' . $e->getMessage(), 'chiffrer.' . _LOG_ERREUR);
+			spip_log('Échec du déchiffrage du jeton d’auteur: '.$e->getMessage(), 'chiffrer.'._LOG_ERREUR);
 			return false;
 		}
 		if ($_jeton && hash_equals($jeton, $_jeton)) {
@@ -428,5 +428,5 @@  discard block
 block discarded – undo
  */
 function auteur_effacer_jeton($id_auteur) {
 	include_spip('base/abstract_sql');
-	return sql_updateq('spip_auteurs', ['cookie_oubli' => ''], 'id_auteur=' . (int) $id_auteur);
+	return sql_updateq('spip_auteurs', ['cookie_oubli' => ''], 'id_auteur='.(int) $id_auteur);
 }

Please login to merge, or discard this patch.

ecrire/inc/session.php 1 patch

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch

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

Please login to merge, or discard this patch.

		@@ -58,7 +58,7 @@ discard block
		block discarded – undo
58	58	}
59	59
60	60	include_spip('base/abstract_sql');
61		- $res = sql_select('statut, id_auteur, login, email, nom', 'spip_auteurs', 'email=' . sql_quote($desc['email']));
	61	+ $res = sql_select('statut, id_auteur, login, email, nom', 'spip_auteurs', 'email='.sql_quote($desc['email']));
62	62	// erreur ?
63	63	if (!$res) {
64	64	return _T('titre_probleme_technique');
		@@ -209,11 +209,11 @@ discard block
		block discarded – undo
209	209
210	210	$login = $login_base;
211	211
212		- for ($i = 1;; $i++) {
	212	+ for ($i = 1; ; $i++) {
213	213	if (!sql_countsel('spip_auteurs', "login='$login'")) {
214	214	return $login;
215	215	}
216		- $login = $login_base . $i;
	216	+ $login = $login_base.$i;
217	217	}
218	218	}
219	219
		@@ -353,11 +353,11 @@ discard block
		block discarded – undo
353	353	do {
354	354	// Un morceau du jeton est lisible en bdd pour éviter de devoir déchiffrer
355	355	// tous les jetons connus pour vérifier le jeton d’un auteur.
356		- $public = substr((string) creer_uniqid(), 0, 7) . '.';
357		- $jeton = $public . creer_uniqid();
358		- $jeton_chiffre_prefixe = $public . Chiffrement::chiffrer($jeton, SpipCles::secret_du_site());
359		- sql_updateq('spip_auteurs', ['cookie_oubli' => $jeton_chiffre_prefixe], 'id_auteur=' . (int) $id_auteur);
360		- } while (sql_countsel('spip_auteurs', 'cookie_oubli=' . sql_quote($jeton_chiffre_prefixe, '', 'string')) > 1);
	356	+ $public = substr((string) creer_uniqid(), 0, 7).'.';
	357	+ $jeton = $public.creer_uniqid();
	358	+ $jeton_chiffre_prefixe = $public.Chiffrement::chiffrer($jeton, SpipCles::secret_du_site());
	359	+ sql_updateq('spip_auteurs', ['cookie_oubli' => $jeton_chiffre_prefixe], 'id_auteur='.(int) $id_auteur);
	360	+ } while (sql_countsel('spip_auteurs', 'cookie_oubli='.sql_quote($jeton_chiffre_prefixe, '', 'string')) > 1);
361	361
362	362	return $jeton;
363	363	}
		@@ -374,7 +374,7 @@ discard block
		block discarded – undo
374	374	*/
375	375	function auteur_lire_jeton(int $id_auteur, bool $autoInit = false): ?string {
376	376	include_spip('base/abstract_sql');
377		- $jeton_chiffre_prefixe = sql_getfetsel('cookie_oubli', 'spip_auteurs', 'id_auteur=' . $id_auteur);
	377	+ $jeton_chiffre_prefixe = sql_getfetsel('cookie_oubli', 'spip_auteurs', 'id_auteur='.$id_auteur);
378	378	if ($jeton_chiffre_prefixe) {
379	379	$jeton_chiffre = substr((string) $jeton_chiffre_prefixe, 8);
380	380	$jeton = Chiffrement::dechiffrer($jeton_chiffre, SpipCles::secret_du_site());
		@@ -404,13 +404,13 @@ discard block
		block discarded – undo
404	404	$public = substr($jeton, 0, 8);
405	405
406	406	// Les auteurs qui ont un jetons ressemblant
407		- $auteurs = sql_allfetsel('*', 'spip_auteurs', 'cookie_oubli LIKE ' . sql_quote($public . '%'));
	407	+ $auteurs = sql_allfetsel('*', 'spip_auteurs', 'cookie_oubli LIKE '.sql_quote($public.'%'));
408	408	foreach ($auteurs as $auteur) {
409	409	$jeton_chiffre = substr((string) $auteur['cookie_oubli'], 8);
410	410	try {
411	411	$_jeton = Chiffrement::dechiffrer($jeton_chiffre, SpipCles::secret_du_site());
412	412	} catch (\Exception $e) {
413		- spip_log('Échec du déchiffrage du jeton d’auteur: ' . $e->getMessage(), 'chiffrer.' . _LOG_ERREUR);
	413	+ spip_log('Échec du déchiffrage du jeton d’auteur: '.$e->getMessage(), 'chiffrer.'._LOG_ERREUR);
414	414	return false;
415	415	}
416	416	if ($_jeton && hash_equals($jeton, $_jeton)) {
		@@ -428,5 +428,5 @@ discard block
		block discarded – undo
428	428	*/
429	429	function auteur_effacer_jeton($id_auteur) {
430	430	include_spip('base/abstract_sql');
431		- return sql_updateq('spip_auteurs', ['cookie_oubli' => ''], 'id_auteur=' . (int) $id_auteur);
	431	+ return sql_updateq('spip_auteurs', ['cookie_oubli' => ''], 'id_auteur='.(int) $id_auteur);
432	432	}

		@@ -81,12 +81,12 @@ discard block
		block discarded – undo
81	81	spip_log("supprimer sessions auteur $id_auteur", 'session');
82	82	if ($toutes \|\| $id_auteur !== $GLOBALS['visiteur_session']['id_auteur']) {
83	83	if ($dir = opendir(_DIR_SESSIONS)) {
84		- $t = $_SERVER['REQUEST_TIME'] - (4 * _RENOUVELLE_ALEA); // 48h par defaut
85		- $t_short = $_SERVER['REQUEST_TIME'] - max(_RENOUVELLE_ALEA / 4, 3 * 3600); // 3h par defaut
	84	+ $t = $_SERVER['REQUEST_TIME'] - (4 * _RENOUVELLE_ALEA); // 48h par defaut
	85	+ $t_short = $_SERVER['REQUEST_TIME'] - max(_RENOUVELLE_ALEA / 4, 3 * 3600); // 3h par defaut
86	86	while (($f = readdir($dir)) !== false) {
87	87	$nb_files++;
88	88	if (preg_match(',^[^\d-]*(-?\d+)_\w{32}\.php[3]?$,', $f, $regs)) {
89		- $f = _DIR_SESSIONS . $f;
	89	+ $f = _DIR_SESSIONS.$f;
90	90	if ($actives && $regs[1] == $id_auteur \|\| $t > filemtime($f)) {
91	91	spip_unlink($f);
92	92	}
		@@ -173,7 +173,7 @@ discard block
		block discarded – undo
173	173	(!$cookie = lire_cookie_session())
174	174	\|\| intval($cookie) !== $id_auteur
175	175	) {
176		- $cookie = $id_auteur . '_' . md5(uniqid(random_int(0, mt_getrandmax()), true));
	176	+ $cookie = $id_auteur.'_'.md5(uniqid(random_int(0, mt_getrandmax()), true));
177	177	}
178	178
179	179	// Maintenant on sait qu'on a des choses à écrire
		@@ -206,7 +206,7 @@ discard block
		block discarded – undo
206	206	} else {
207	207	$fichier_session = chemin_fichier_session('alea_ephemere', $cookie);
208	208	if (!ecrire_fichier_session($fichier_session, $auteur)) {
209		- spip_log('Echec ecriture fichier session ' . $fichier_session, 'session' . _LOG_HS);
	209	+ spip_log('Echec ecriture fichier session '.$fichier_session, 'session'._LOG_HS);
210	210	include_spip('inc/minipres');
211	211	echo minipres();
212	212	exit;
		@@ -229,9 +229,9 @@ discard block
		block discarded – undo
229	229	if (autoriser('ecrire', '', '', $auteur) && _DUREE_COOKIE_ADMIN) {
230	230	spip_setcookie(
231	231	'spip_admin',
232		- '@' . ($auteur['email'] ?: $auteur['login']),
	232	+ '@'.($auteur['email'] ?: $auteur['login']),
233	233	time() + max(_DUREE_COOKIE_ADMIN, $duree),
234		- httponly: true
	234	+ httponly : true
235	235	);
236	236	} else {
237	237	// sinon le supprimer ...
		@@ -269,7 +269,7 @@ discard block
		block discarded – undo
269	269	$coef = 20;
270	270	}
271	271	}
272		- return (int)(_RENOUVELLE_ALEA * $coef);
	272	+ return (int) (_RENOUVELLE_ALEA * $coef);
273	273	}
274	274
275	275	/**
		@@ -381,7 +381,7 @@ discard block
		block discarded – undo
381	381
382	382	// Renouveler la session avec l'alea courant
383	383	include($fichier_session);
384		- spip_log('renouvelle session ' . $GLOBALS['visiteur_session']['id_auteur'], 'session');
	384	+ spip_log('renouvelle session '.$GLOBALS['visiteur_session']['id_auteur'], 'session');
385	385	spip_unlink($fichier_session);
386	386	ajouter_session($GLOBALS['visiteur_session']);
387	387	}
		@@ -630,7 +630,7 @@ discard block
		block discarded – undo
630	630	}
631	631
632	632	// liste des sessions
633		- $sessions = preg_files(_DIR_SESSIONS, '/' . $id_auteur . '_.*\.php$');
	633	+ $sessions = preg_files(_DIR_SESSIONS, '/'.$id_auteur.'_.*\.php$');
634	634
635	635	// si on en a plus que la limite, supprimer les plus vieilles
636	636	// si ce ne sont pas des sessions anonymes car elles sont alors chacune differentes
		@@ -708,12 +708,12 @@ discard block
		block discarded – undo
708	708	$auteur = preparer_ecriture_session($auteur);
709	709
710	710	// enregistrer les autres donnees du visiteur
711		- $texte = '<' . "?php\n";
	711	+ $texte = '<'."?php\n";
712	712	foreach ($auteur as $var => $val) {
713		- $texte .= '$GLOBALS[\'visiteur_session\'][' . var_export($var, true) . '] = '
714		- . var_export($val, true) . ";\n";
	713	+ $texte .= '$GLOBALS[\'visiteur_session\']['.var_export($var, true).'] = '
	714	+ . var_export($val, true).";\n";
715	715	}
716		- $texte .= '?' . ">\n";
	716	+ $texte .= '?'.">\n";
717	717
718	718	return ecrire_fichier($fichier, $texte);
719	719	}
		@@ -737,7 +737,7 @@ discard block
		block discarded – undo
737	737
738	738	$repertoire = sous_repertoire(_DIR_SESSIONS, '', false, $tantpis);
739	739	$id_auteur = intval($cookie_session);
740		- return $repertoire . $id_auteur . '_' . md5($cookie_session . ' ' . $GLOBALS['meta'][$alea]) . '.php';
	740	+ return $repertoire.$id_auteur.'_'.md5($cookie_session.' '.$GLOBALS['meta'][$alea]).'.php';
741	741	}
742	742
743	743	/**
		@@ -765,7 +765,7 @@ discard block
		block discarded – undo
765	765	* @return string
766	766	*/
767	767	function rejouer_session() {
768		- return '<img src="' . generer_url_action('cookie', 'change_session=oui', true) . '" width="0" height="0" alt="" />';
	768	+ return '<img src="'.generer_url_action('cookie', 'change_session=oui', true).'" width="0" height="0" alt="" />';
769	769	}
770	770
771	771
		@@ -780,7 +780,7 @@ discard block
		block discarded – undo
780	780	return $res;
781	781	}
782	782
783		- return $res = md5($GLOBALS['ip'] . ($_SERVER['HTTP_USER_AGENT'] ?? ''));
	783	+ return $res = md5($GLOBALS['ip'].($_SERVER['HTTP_USER_AGENT'] ?? ''));
784	784	}
785	785
786	786

		@@ -10,7 +10,7 @@
		block discarded – undo
10	10	{
11	11	public static function setUpBeforeClass(): void {
12	12	include_spip('inc/autoriser');
13		- require_once(__DIR__ . '/data/autoriser.php');
	13	+ require_once(__DIR__.'/data/autoriser.php');
14	14	}
15	15
16	16	public function testAutoriserOkNiet(): void {

		@@ -25,7 +25,7 @@ discard block
		block discarded – undo
25	25
26	26	public static function providerXmlSpipXmlParse(): array {
27	27	$essais = [];
28		- $xml1 = '<' . <<<CODE_SAMPLE
	28	+ $xml1 = '<'.<<<CODE_SAMPLE
29	29	?xml version="1.0" encoding="utf-8"?>
30	30	<urlset xmlns="http://www.google.com/schemas/sitemap/0.84">
31	31	<url><loc>http://localhost/_core/spip/spip.php?breve25</loc>
		@@ -47,7 +47,7 @@ discard block
		block discarded – undo
47	47	$tree1 = <<<CODE_SAMPLE
48	48	a:1:{s:57:"urlset xmlns="http://www.google.com/schemas/sitemap/0.84"";a:1:{i:0;a:1:{s:3:"url";a:3:{i:0;a:3:{s:3:"loc";a:1:{i:0;s:44:"http://localhost/_core/spip/spip.php?breve25";}s:7:"lastmod";a:1:{i:0;s:10:"2003-12-31";}s:8:"priority";a:1:{i:0;s:3:"0.8";}}i:1;a:3:{s:3:"loc";a:1:{i:0;s:44:"http://localhost/_core/spip/spip.php?breve32";}s:7:"lastmod";a:1:{i:0;s:10:"2004-02-10";}s:8:"priority";a:1:{i:0;s:3:"0.8";}}i:2;a:3:{s:3:"loc";a:1:{i:0;s:44:"http://localhost/_core/spip/spip.php?breve64";}s:7:"lastmod";a:1:{i:0;s:10:"2005-01-31";}s:8:"priority";a:1:{i:0;s:3:"0.8";}}}}}}
49	49	CODE_SAMPLE;
50		- $xml2 = '<' . <<<CODE_SAMPLE
	50	+ $xml2 = '<'.<<<CODE_SAMPLE
51	51	?xml version="1.0" encoding="UTF-8"?>
52	52	<opml version="1.0">
53	53	<head>

		@@ -22,8 +22,8 @@ discard block
		block discarded – undo
22	22	public function testLienPrive() {
23	23	$relatif = generer_url_ecrire('toto', 'truc=machin&chose=bidule', false, true);
24	24	$absolu = generer_url_ecrire('toto', 'truc=machin&chose=bidule', false, false);
25		- $expected = 'bla bla <a href=\'' . str_replace('&', '&', $absolu) . '\'>lien prive</a>';
26		- $case = 'bla bla <a href=\'' . $relatif . '\'>lien prive</a>';
	25	+ $expected = 'bla bla <a href=\''.str_replace('&', '&', $absolu).'\'>lien prive</a>';
	26	+ $case = 'bla bla <a href=\''.$relatif.'\'>lien prive</a>';
27	27	$actual = liens_absolus($case);
28	28	$this->assertEquals($expected, $actual);
29	29	}
		@@ -32,8 +32,8 @@ discard block
		block discarded – undo
32	32	public function testLienPublic() {
33	33	$relatif = generer_url_public('toto', 'truc=machin&chose=bidule', false, true);
34	34	$absolu = generer_url_public('toto', 'truc=machin&chose=bidule', false, false);
35		- $expected = 'bla bla <a href=\'' . str_replace('&', '&', $absolu) . '\'>lien public</a>';
36		- $case = 'bla bla <a href=\'' . $relatif . '\'>lien public</a>';
	35	+ $expected = 'bla bla <a href=\''.str_replace('&', '&', $absolu).'\'>lien public</a>';
	36	+ $case = 'bla bla <a href=\''.$relatif.'\'>lien public</a>';
37	37	$actual = liens_absolus($case);
38	38	$this->assertEquals($expected, $actual);
39	39	}

		@@ -14,16 +14,16 @@
		block discarded – undo
14	14	}
15	15
16	16	public function testSousRepertoire(): void {
17		- $sous_repertoire = 'test' . md5(strval(random_int(0, mt_getrandmax())));
	17	+ $sous_repertoire = 'test'.md5(strval(random_int(0, mt_getrandmax())));
18	18	$this->assertSame(
19	19	sous_repertoire(_DIR_VAR, $sous_repertoire),
20		- _DIR_VAR . $sous_repertoire . '/'
	20	+ _DIR_VAR.$sous_repertoire.'/'
21	21	);
22		- $this->assertTrue(file_exists(_DIR_VAR . $sous_repertoire));
23		- $this->assertTrue(is_dir(_DIR_VAR . $sous_repertoire));
	22	+ $this->assertTrue(file_exists(_DIR_VAR.$sous_repertoire));
	23	+ $this->assertTrue(is_dir(_DIR_VAR.$sous_repertoire));
24	24
25	25	// Nettoyage
26		- @unlink(_DIR_VAR . $sous_repertoire . '/.ok');
27		- @rmdir(_DIR_VAR . $sous_repertoire);
	26	+ @unlink(_DIR_VAR.$sous_repertoire.'/.ok');
	27	+ @rmdir(_DIR_VAR.$sous_repertoire);
28	28	}
29	29	}

		@@ -28,7 +28,7 @@ discard block
		block discarded – undo
28	28
29	29	$this->assertIsArray($_chemins);
30	30	$this->assertEquals(count($chemins), count($_chemins) - 1, 'Erreur ajout chemin par la fonction _chemin() : mauvais compte');
31		- if (is_dir(_DIR_RACINE . 'squelettes')) {
	31	+ if (is_dir(_DIR_RACINE.'squelettes')) {
32	32	$this->assertEquals('toto/', $_chemins[1], 'Erreur ajout chemin par la fonction _chemin() : avec squelettes');
33	33	} else {
34	34	$this->assertEquals('toto/', $_chemins[0], 'Erreur ajout chemin par la fonction _chemin() : sans squelettes');
		@@ -40,7 +40,7 @@ discard block
		block discarded – undo
40	40	$chemins = creer_chemin();
41	41	$this->assertIsArray($chemins);
42	42
43		- $squelettes = (int) is_dir(_DIR_RACINE . 'squelettes');
	43	+ $squelettes = (int) is_dir(_DIR_RACINE.'squelettes');
44	44	$dossier_squelettes = count(explode(':', $GLOBALS['dossier_squelettes']));
45	45
46	46	_chemin('toto');

		@@ -11,7 +11,7 @@ discard block
		block discarded – undo
11	11
12	12	// Script pour appeler un squelette apres s'etre authentifie
13	13
14		-include_once __DIR__ . '/inc_version.php';
	14	+include_once __DIR__.'/inc_version.php';
15	15
16	16	include_spip('inc/cookie');
17	17
		@@ -24,7 +24,7 @@ discard block
		block discarded – undo
24	24	// On reexecute pour deboucher sur le include public.
25	25	// autrement on insiste
26	26	if (is_array($var_auth)) {
27		- $var_auth = '../?' . $_SERVER['QUERY_STRING'];
	27	+ $var_auth = '../?'.$_SERVER['QUERY_STRING'];
28	28	include_spip('inc/session');
29	29	// on prolonge le cookie
30	30	set_cookie_session(null, time() + 3600 * 24 * 14);
		@@ -34,4 +34,4 @@ discard block
		block discarded – undo
34	34	}
35	35
36	36	// En somme, est prive' ce qui est publiquement nomme'...
37		-include __DIR__ . '/public.php';
	37	+include __DIR__.'/public.php';

		@@ -20,7 +20,7 @@ discard block
		block discarded – undo
20	20	use Symplify\EasyCodingStandard\Config\ECSConfig;
21	21	use Symplify\EasyCodingStandard\ValueObject\Set\SetList;
22	22
23		-return static function (ECSConfig $ecsConfig): void {
	23	+return static function(ECSConfig $ecsConfig): void {
24	24	// A. full sets
25	25	$ecsConfig->sets([SetList::PSR_12, SetList::SYMPLIFY, SetList::COMMON, SetList::CLEAN_CODE]);
26	26	$ecsConfig->rule(NoExtraBlankLinesFixer::class);
		@@ -36,7 +36,7 @@ discard block
		block discarded – undo
36	36	$ecsConfig->indentation('tab');
37	37
38	38	$ecsConfig->paths([
39		- __DIR__ . '/ecrire/tests',
	39	+ __DIR__.'/ecrire/tests',
40	40	# __DIR__ . '/index.php',
41	41	# __DIR__ . '/spip.php',
42	42	# __DIR__ . '/ecrire',
		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44	]);
45	45
46	46	$ecsConfig->skip([
47		- __DIR__ . '/ecrire/lang',
	47	+ __DIR__.'/ecrire/lang',
48	48	ArrayListItemNewlineFixer::class,
49	49	ArrayOpenerAndCloserNewlineFixer::class,
50	50	AssignmentInConditionSniff::class,

spip / SPIP

Push — master ( ffbbca...ad9227 )

Status

Category

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch