spip / SPIP

ecrire/tests/Config/DepotMetaPackTest.php

Spacing +5 added lines, -5 removed lines

@@ -87,7 +87,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture " . reset($essai));
+			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture ".reset($essai));
 		}
 
 		$GLOBALS['meta'] = $meta;
@@ -125,7 +125,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, ecrire_config(...$essai), "Echec {$k} : ecriture " . reset($essai));
+			$this->assertEquals($expected, ecrire_config(...$essai), "Echec {$k} : ecriture ".reset($essai));
 		}
 	}
 
@@ -151,7 +151,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture " . reset($essai));
+			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture ".reset($essai));
 		}
 	}
 
@@ -181,7 +181,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, effacer_config(...$essai), "Echec {$k} : effacer " . reset($essai));
+			$this->assertEquals($expected, effacer_config(...$essai), "Echec {$k} : effacer ".reset($essai));
 		}
 	}
 
@@ -201,7 +201,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture " . reset($essai));
+			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture ".reset($essai));
 		}
 	}
 }

ecrire/tests/Config/DepotMetaTest.php

Spacing +6 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, expliquer_config(...$essai), "Echec {$k} : lecture " . end($essai));
+			$this->assertEquals($expected, expliquer_config(...$essai), "Echec {$k} : lecture ".end($essai));
 		}
 	}
 
@@ -96,7 +96,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture " . reset($essai));
+			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture ".reset($essai));
 		}
 
 		$GLOBALS['meta'] = $meta;
@@ -131,7 +131,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, ecrire_config(...$essai), "Echec {$k} : ecriture " . reset($essai));
+			$this->assertEquals($expected, ecrire_config(...$essai), "Echec {$k} : ecriture ".reset($essai));
 		}
 	}
 
@@ -149,7 +149,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture " . reset($essai));
+			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture ".reset($essai));
 		}
 	}
 
@@ -168,7 +168,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, effacer_config(...$essai), "Echec {$k} : effacer " . reset($essai));
+			$this->assertEquals($expected, effacer_config(...$essai), "Echec {$k} : effacer ".reset($essai));
 		}
 	}
 
@@ -187,7 +187,7 @@ 
 
 		foreach ($essais as $k => $essai) {
 			$expected = array_shift($essai);
-			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture " . reset($essai));
+			$this->assertEquals($expected, lire_config(...$essai), "Echec {$k} : lecture ".reset($essai));
 		}
 	}
 }

ecrire/tests/Template.php

Spacing +2 added lines, -2 removed lines

@@ -47,11 +47,11 @@
 		if (!empty($infos['source'])) {
 			// on ajoute des infos supplementaires a celles retournees
 			$path = pathinfo($infos['source']);
-			$infos['fond'] = $path['dirname'] . '/' . $path['filename']; // = $fond;
+			$infos['fond'] = $path['dirname'].'/'.$path['filename']; // = $fond;
 			$infos['erreurs'] = $this->get_compilation_errors();
 		} else {
 			// on a été interrompu par un minipres ?
-			throw new \Exception('Calcul de ' . $this->fond . ' interrompue (minipres?)' . "\n\n" . $infos['texte']);
+			throw new \Exception('Calcul de '.$this->fond.' interrompue (minipres?)'."\n\n".$infos['texte']);
 		}
 
 		return $infos;

ecrire/tests/Plugin/DirPluginsSupplTest.php

Spacing +8 added lines, -8 removed lines

@@ -10,17 +10,17 @@ 
 
 class DirPluginsSupplTest extends TestCase
 {
-	const DIR_PLUGINS_SUPPL = _DIR_TMP . 'test_dir_plugins_suppl/';
-	const DIR_PLUGINS_OUTSIDE = _DIR_TMP . 'test_dir_plugins_outisde/';
+	const DIR_PLUGINS_SUPPL = _DIR_TMP.'test_dir_plugins_suppl/';
+	const DIR_PLUGINS_OUTSIDE = _DIR_TMP.'test_dir_plugins_outisde/';
 	const PAQUET_TEST = 'toto/paquet.xml';
 
 	public static function setUpBeforeClass(): void {
 		find_in_path('./inc/plugin.php', '', true);
 		supprimer_repertoire(self::DIR_PLUGINS_SUPPL);
-		mkdir(self::DIR_PLUGINS_SUPPL . 'toto', 0777, true);
-		mkdir(self::DIR_PLUGINS_OUTSIDE . 'toto', 0777, true);
-		copy(__DIR__ . '/data/paquet.xml', self::DIR_PLUGINS_SUPPL . self::PAQUET_TEST);
-		copy(__DIR__ . '/data/paquet.xml', self::DIR_PLUGINS_OUTSIDE . self::PAQUET_TEST);
+		mkdir(self::DIR_PLUGINS_SUPPL.'toto', 0777, true);
+		mkdir(self::DIR_PLUGINS_OUTSIDE.'toto', 0777, true);
+		copy(__DIR__.'/data/paquet.xml', self::DIR_PLUGINS_SUPPL.self::PAQUET_TEST);
+		copy(__DIR__.'/data/paquet.xml', self::DIR_PLUGINS_OUTSIDE.self::PAQUET_TEST);
 	}
 
 	public static function tearDownAfterClass(): void {
@@ -47,10 +47,10 @@ 
 	public function testDirectory() {
 		// le rep suppl existe
 		$this->assertTrue(is_dir(self::DIR_PLUGINS_SUPPL), sprintf('Le répertoire "%s" aurait du être créé', self::DIR_PLUGINS_SUPPL));
-		$this->assertTrue(file_exists(self::DIR_PLUGINS_SUPPL . self::PAQUET_TEST), sprintf('Le fichier "%s" aurait du être créé', self::DIR_PLUGINS_SUPPL . self::PAQUET_TEST));
+		$this->assertTrue(file_exists(self::DIR_PLUGINS_SUPPL.self::PAQUET_TEST), sprintf('Le fichier "%s" aurait du être créé', self::DIR_PLUGINS_SUPPL.self::PAQUET_TEST));
 		// le rep outside existe
 		$this->assertTrue(is_dir(self::DIR_PLUGINS_OUTSIDE), sprintf('Le répertoire "%s" aurait du être créé', self::DIR_PLUGINS_OUTSIDE));
-		$this->assertTrue(file_exists(self::DIR_PLUGINS_OUTSIDE . self::PAQUET_TEST), sprintf('Le fichier "%s" aurait du être créé', self::DIR_PLUGINS_OUTSIDE . self::PAQUET_TEST));
+		$this->assertTrue(file_exists(self::DIR_PLUGINS_OUTSIDE.self::PAQUET_TEST), sprintf('Le fichier "%s" aurait du être créé', self::DIR_PLUGINS_OUTSIDE.self::PAQUET_TEST));
 	}
 
 	#[Depends('testDirectory')]

ecrire/tests/Api/AutoriserTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 {
 	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 {

ecrire/tests/Format/Xml/SpipXmlParseTest.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 
 	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 @@ 
 		$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>

ecrire/tests/Urls/LiensAbsolusTest.php

Spacing +4 added lines, -4 removed lines

@@ -22,8 +22,8 @@ 
 	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 @@ 
 	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);
 	}

ecrire/tests/Texte/LiensTest.php

Spacing +4 added lines, -4 removed lines

@@ -23,8 +23,8 @@ 
 			['id_article', 'lang'],
 			'spip_articles',
 			[
-				'statut = ' . sql_quote('publie'),
-				'lang != ' . sql_quote('')
+				'statut = '.sql_quote('publie'),
+				'lang != '.sql_quote('')
 			],
 			limit: '0,1',
 		);
@@ -35,7 +35,7 @@ 
 		// on se met dans une autre langue que celle de l'article
 		lang_select($lang === 'eo' ? 'fa' : 'eo');
 
-		$case = '[->' . $id_article . ']';
+		$case = '[->'.$id_article.']';
 		$propre = propre($case);
 		$this->assertEquals(
 			$lang,
@@ -46,7 +46,7 @@ 
 
 	#[DataProvider('providerLiensClassCss')]
 	public function testLiensClassCss(string $table, ?string $short) {
-		$id = sql_getfetsel(id_table_objet($table), $table, "statut='publie'", limit: '0,1');
+		$id = sql_getfetsel(id_table_objet($table), $table, "statut='publie'", limit : '0,1');
 		$type = objet_type($table);
 		if (!$id) {
 			$this->markTestSkipped(sprintf('Necessite un·e %s publié', $type));

ecrire/tests/Filesystem/FlockTest.php

Spacing +6 added lines, -6 removed lines

@@ -14,16 +14,16 @@
 	}
 
 	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);
 	}
 }