spip / SPIP

ecrire/tests/Typographie/FrTest.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 	public static function setUpBeforeClass(): void {
 		include_spip('inc/texte');
 		changer_langue(static::$lang);
-		include_spip(static::$root . 'typographie/' . static::$lang);
+		include_spip(static::$root.'typographie/'.static::$lang);
 	}
 
 	public static function providerBase() {

ecrire/tests/Typographie/NotesTest.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 		$expected = '<p>a<span class="spip_note_ref">&nbsp;[<a href="#nb1" class="spip_note" rel="appendix" title="b" id="nh1">1</a>]</span></p>';
 		$this->assertEquals($expected, propre('a[[b]]'));
 
-		$expected = "<div id=\"nb1\">\n<p><span class=\"spip_note_ref\">[<a href=\"#nh1\" class=\"spip_note\" title=\"" . _T('info_notes') . " 1\" rev=\"appendix\">1</a>]&nbsp;</span>b</p>\n</div>";
+		$expected = "<div id=\"nb1\">\n<p><span class=\"spip_note_ref\">[<a href=\"#nh1\" class=\"spip_note\" title=\""._T('info_notes')." 1\" rev=\"appendix\">1</a>]&nbsp;</span>b</p>\n</div>";
 		$this->assertEquals($expected, calculer_notes());
 	}
 

ecrire/tests/Filtre/Form/FormHiddenCase.php

Spacing +13 added lines, -13 removed lines

@@ -65,50 +65,50 @@ 
 		return [
 			0 =>
 			[
-				0 => '<input name="id_rubrique" value="' . $id . '" type="hidden"
+				0 => '<input name="id_rubrique" value="'.$id.'" type="hidden"
 /><input name="page" value="rubrique" type="hidden"
 />',
-				1 => './?rubrique' . $id,
+				1 => './?rubrique'.$id,
 			],
 			1 =>
 			[
 				0 => '<input name="calendrier" value="1" type="hidden"
-/><input name="id_rubrique" value="' . $id . '" type="hidden"
+/><input name="id_rubrique" value="' . $id.'" type="hidden"
 /><input name="page" value="rubrique" type="hidden"
 />',
-				1 => './?rubrique' . $id . '&calendrier=1',
+				1 => './?rubrique'.$id.'&calendrier=1',
 			],
 			2 =>
 			[
-				0 => '<input name="id_rubrique" value="' . $id . '" type="hidden"
+				0 => '<input name="id_rubrique" value="'.$id.'" type="hidden"
 /><input name="page" value="rubrique" type="hidden"
 />',
-				1 => './rubrique' . $id . '.html',
+				1 => './rubrique'.$id.'.html',
 			],
 			3 =>
 			[
 				0 => '<input name="calendrier" value="1" type="hidden"
-/><input name="id_rubrique" value="' . $id . '" type="hidden"
+/><input name="id_rubrique" value="' . $id.'" type="hidden"
 /><input name="page" value="rubrique" type="hidden"
 />',
-				1 => './rubrique' . $id . '.html?calendrier=1',
+				1 => './rubrique'.$id.'.html?calendrier=1',
 			],
 			4 =>
 			[
 				0 => '<input name="calendrier" value="1" type="hidden"
-/><input name="id_rubrique" value="' . $id . '" type="hidden"
+/><input name="id_rubrique" value="' . $id.'" type="hidden"
 /><input name="page" value="rubrique" type="hidden"
 />',
-				1 => './?rubrique' . $id . '&amp;calendrier=1',
+				1 => './?rubrique'.$id.'&amp;calendrier=1',
 			],
 			5 =>
 			[
 				0 => '<input name="calendrier" value="1" type="hidden"
 /><input name="toto" value="2" type="hidden"
-/><input name="id_rubrique" value="' . $id . '" type="hidden"
+/><input name="id_rubrique" value="' . $id.'" type="hidden"
 /><input name="page" value="rubrique" type="hidden"
 />',
-				1 => './rubrique' . $id . '.html?calendrier=1&amp;toto=2',
+				1 => './rubrique'.$id.'.html?calendrier=1&amp;toto=2',
 			],
 		];
 	}
@@ -118,7 +118,7 @@ 
 		$id_rubrique = sql_getfetsel(
 			'id_rubrique',
 			'spip_rubriques',
-			['statut = ' . sql_quote('publie')],
+			['statut = '.sql_quote('publie')],
 			limit: '0, 1',
 		);
 		return $id_rubrique ? (int) $id_rubrique : null;

ecrire/tests/Filtre/ExtraireBalisesTest.php

Spacing +4 added lines, -4 removed lines

@@ -48,7 +48,7 @@ 
 	}
 
 	public function testFiltresExtraireBalisesMediaRss(): void {
-		$rss = file_get_contents(dirname(__DIR__) . '/Fixtures/data/dailymotion.rss');
+		$rss = file_get_contents(dirname(__DIR__).'/Fixtures/data/dailymotion.rss');
 		if (empty($rss)) {
 			$this->markTestSkipped();
 		}
@@ -62,7 +62,7 @@ 
 		return [
 			[['<a href="truc">chose</a>'], 'allo <a href="truc">chose</a>'],
 			[['<a href="truc" />'], 'allo <a href="truc" />'],
-			[["<a\nhref='truc' />"], 'allo' . "\n" . " <a\nhref='truc' />"],
+			[["<a\nhref='truc' />"], 'allo'."\n"." <a\nhref='truc' />"],
 			[[['<a href="1">'], ['<a href="2">']], ['allo <a href="1">', 'allo <a href="2">']],
 			[['<a href="truc">chose</a>'], 'bonjour <a href="truc">chose</a> machin'],
 			[
@@ -74,7 +74,7 @@ 
 			[['<a>chose</a>'], '<a>chose</a>'],
 			[['<a href="truc">chose</a>'], 'allo <a href="truc">chose</a>', 'a'],
 			[['<a href="truc" />'], 'allo <a href="truc" />', 'a'],
-			[["<a\nhref='truc' />"], 'allo' . "\n" . " <a\nhref='truc' />", 'a'],
+			[["<a\nhref='truc' />"], 'allo'."\n"." <a\nhref='truc' />", 'a'],
 			[[['<a href="1">'], ['<a href="2">']], ['allo <a href="1">', 'allo <a href="2">'], 'a'],
 			[['<a href="truc">chose</a>'], 'bonjour <a href="truc">chose</a> machin', 'a'],
 			[
@@ -87,7 +87,7 @@ 
 			[['<a>chose</a>'], '<a>chose</a>', 'a'],
 			[[], 'allo <a href="truc">chose</a>', 'b'],
 			[[], 'allo <a href="truc" />', 'b'],
-			[[], 'allo' . "\n" . " <a\nhref='truc' />", 'b'],
+			[[], 'allo'."\n"." <a\nhref='truc' />", 'b'],
 			[[[], []], ['allo <a href="1">', 'allo <a href="2">'], 'b'],
 			[[], 'bonjour <a href="truc">chose</a> machin', 'b'],
 			[[], 'bonjour <a href="truc">chose</a> machin <A href="truc">machin</a>', 'b'],

ecrire/tests/Filtre/AttributHtmlTest.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 			],
 			2 => [
 				'expected' => 'allons à la mer',
-				'texte' => 'allons ' . chr(195) . chr(160) . ' la mer', // le a` risque de matcher \s
+				'texte' => 'allons '.chr(195).chr(160).' la mer', // le a` risque de matcher \s
 			],
 		];
 	}

ecrire/tests/Sql/SqlSchemaTableTest.php

Spacing +16 added lines, -16 removed lines

@@ -63,8 +63,8 @@ 
 	#[Depends('testInsertData')]
 	public function testMajTimestamp() {
 		$table = 'spip_test_tintin';
-		$where1 = 'id_tintin=' . sql_quote(1);
-		$where2 = 'id_tintin=' . sql_quote(2);
+		$where1 = 'id_tintin='.sql_quote(1);
+		$where2 = 'id_tintin='.sql_quote(2);
 
 		// lecture du timestamp actuel
 		$maj1 = sql_getfetsel('maj', $table, $where1);
@@ -123,17 +123,17 @@ 
 		$this->assertEquals(count($data), sql_count($res), 'sql_count() ne renvoie pas la valeur attendue');
 
 		// selection float
-		$res = sql_select('*', 'spip_test_tintin', ['un_double>' . sql_quote(3)]);
+		$res = sql_select('*', 'spip_test_tintin', ['un_double>'.sql_quote(3)]);
 		$n = count(array_filter($data, fn ($entry) => $entry['un_double'] > 3));
 		$this->assertEquals($n, sql_count($res), 'sql_count() ne renvoie pas la valeur attendue sur un float');
 
 		// selection REGEXP
-		$res = sql_select('*', 'spip_test_tintin', ['un_varchar REGEXP ' . sql_quote('^De')]);
+		$res = sql_select('*', 'spip_test_tintin', ['un_varchar REGEXP '.sql_quote('^De')]);
 		$n = count(array_filter($data, fn ($entry) => str_starts_with($entry['un_varchar'], 'De')));
 		$this->assertEquals($n, sql_count($res), 'sql_count() ne renvoie pas la valeur attendue sur une REGEXP');
 
 		// selection LIKE
-		$res = sql_select('*', 'spip_test_tintin', ['un_varchar LIKE ' . sql_quote('De%')]);
+		$res = sql_select('*', 'spip_test_tintin', ['un_varchar LIKE '.sql_quote('De%')]);
 		$this->assertEquals($n, sql_count($res), 'sql_count() ne renvoie pas la valeur attendue sur un LIKE');
 
 		// selection array(champs)
@@ -167,7 +167,7 @@ 
 				'en' => 'Krack',
 			] as $lg => $res
 		) {
-			$multi = sql_getfetsel(sql_multi('grrrr', $lg), 'spip_test_milou', 'id_milou=' . sql_quote(2));
+			$multi = sql_getfetsel(sql_multi('grrrr', $lg), 'spip_test_milou', 'id_milou='.sql_quote(2));
 			$this->assertEquals($res, $multi, 'sql_multi mal rendu');
 		}
 
@@ -178,7 +178,7 @@ 
 				'en' => 'Aérieny',
 			] as $lg => $res
 		) {
-			$multi = sql_getfetsel(sql_multi('alcool', $lg), 'spip_test_haddock', 'id_haddock=' . sql_quote(2));
+			$multi = sql_getfetsel(sql_multi('alcool', $lg), 'spip_test_haddock', 'id_haddock='.sql_quote(2));
 			$this->assertEquals($res, $multi, 'sql_multi avec accents, mal rendu');
 		}
 
@@ -190,7 +190,7 @@ 
 				'de' => 'Un début de chaine : Vinasse, et [la fin]',
 			] as $lg => $res
 		) {
-			$multi = sql_getfetsel(sql_multi('alcool', $lg), 'spip_test_haddock', 'id_haddock=' . sql_quote(4));
+			$multi = sql_getfetsel(sql_multi('alcool', $lg), 'spip_test_haddock', 'id_haddock='.sql_quote(4));
 			$this->assertEquals($res, $multi, 'sql_multi avec crochets, mal rendu');
 		}
 	}
@@ -291,10 +291,10 @@ 
 				'2.0/2' => (2.0 / 2),
 				'2/2' => (2 / 2),
 				'md5(8)' => md5('8'),
-				'md5(' . sql_quote('a') . ')' => md5('a'),
+				'md5('.sql_quote('a').')' => md5('a'),
 			] as $func => $expected
 		) {
-			$nb = sql_getfetsel("{$func} AS nb", ['spip_test_tintin'], ['id_tintin=' . sql_quote(1)]);
+			$nb = sql_getfetsel("{$func} AS nb", ['spip_test_tintin'], ['id_tintin='.sql_quote(1)]);
 			$this->assertEquals($expected, $nb, "Selection {$func} en echec");
 		}
 	}
@@ -307,11 +307,11 @@ 
 	function testStringFunctions() {
 		foreach (
 			[
-				'CONCAT(' . sql_quote('cou') . ',' . sql_quote('cou') . ')' => 'coucou',
-				'CONCAT(' . sql_quote('cou,') . ',' . sql_quote('cou') . ')' => 'cou,cou',
+				'CONCAT('.sql_quote('cou').','.sql_quote('cou').')' => 'coucou',
+				'CONCAT('.sql_quote('cou,').','.sql_quote('cou').')' => 'cou,cou',
 			] as $func => $expected
 		) {
-			$nb = sql_getfetsel("{$func} AS nb", ['spip_test_tintin'], ['id_tintin=' . sql_quote(1)]);
+			$nb = sql_getfetsel("{$func} AS nb", ['spip_test_tintin'], ['id_tintin='.sql_quote(1)]);
 			$this->assertEquals($expected, $nb, "Selection {$func} en echec");
 		}
 	}
@@ -339,11 +339,11 @@ 
 	#[Depends('testInsertData')]
 	public function testUpdateData() {
 		// ajouter un champ
-		$nb = sql_getfetsel('un_bigint', 'spip_test_tintin', 'id_tintin=' . sql_quote(1));
+		$nb = sql_getfetsel('un_bigint', 'spip_test_tintin', 'id_tintin='.sql_quote(1));
 		sql_update('spip_test_tintin', [
 			'un_bigint' => 'un_bigint+2',
 		]);
-		$nb2 = sql_getfetsel('un_bigint', 'spip_test_tintin', 'id_tintin=' . sql_quote(1));
+		$nb2 = sql_getfetsel('un_bigint', 'spip_test_tintin', 'id_tintin='.sql_quote(1));
 		$this->assertEquals($nb + 2, $nb2, 'sql_update n’a pas fait l’adition !');
 	}
 
@@ -355,7 +355,7 @@ 
 	public function test_delete_data() {
 		$nb = sql_countsel('spip_test_tintin');
 		// supprimer une ligne
-		sql_delete('spip_test_tintin', 'id_tintin=' . sql_quote(1));
+		sql_delete('spip_test_tintin', 'id_tintin='.sql_quote(1));
 		$this->assertEquals($nb - 1, sql_countsel('spip_test_tintin'), "sql_delete n’a pas supprimé la ligne");
 
 		// supprimer tout

ecrire/tests/Xml/XmlHackTest.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 {
 	#[DataProvider('providerXmlIsNotPhp')]
 	public function testXmlIsNotPhp(string $squelette): void {
-		$skel = $this->relativePath(__DIR__ . '/data/' . $squelette);
+		$skel = $this->relativePath(__DIR__.'/data/'.$squelette);
 		$out = recuperer_fond($skel, [], [
 			'raw' => true,
 			'trim' => true

ecrire/tests/Testing/TemplateTest.php

Spacing +2 added lines, -2 removed lines

@@ -11,12 +11,12 @@
 {
 	public function testNativeRecupererFond(): void {
 		$dir = $this->relativePath(__DIR__);
-		$this->assertEquals('Hello World', recuperer_fond($dir . '/data/texte_hello_world'));
+		$this->assertEquals('Hello World', recuperer_fond($dir.'/data/texte_hello_world'));
 	}
 
 	public function testRenderer(): void {
 		$dir = $this->relativePath(__DIR__);
-		$template = new Template($dir . '/data/texte_hello_world');
+		$template = new Template($dir.'/data/texte_hello_world');
 		$this->assertEquals('Hello World', $template->render());
 	}
 }

ecrire/tests/Testing/TemplatingTest.php

Spacing +3 added lines, -3 removed lines

@@ -19,7 +19,7 @@ 
 		$this->assertInstanceOf(FileLoader::class, $templating->getLoader());
 		$this->assertEquals($loader, $templating->getLoader());
 
-		$file = __DIR__ . '/data/texte_hello_world.html';
+		$file = __DIR__.'/data/texte_hello_world.html';
 		$expected = trim(file_get_contents($file));
 
 		// Indirect render
@@ -37,7 +37,7 @@ 
 
 	public function testFileLoaderException(): void {
 		$templating = new Templating(new FileLoader());
-		$file = __DIR__ . '/data/inexistant_file.html';
+		$file = __DIR__.'/data/inexistant_file.html';
 		$this->expectException(TemplateNotFoundException::class);
 		$templating->render($file);
 	}
@@ -52,7 +52,7 @@ 
 	public function testChainLoader(): void {
 		$template = new Templating(new ChainLoader([new FileLoader(), new StringLoader()]));
 
-		$file = __DIR__ . '/data/texte_hello_world.html';
+		$file = __DIR__.'/data/texte_hello_world.html';
 		$expected = trim(file_get_contents($file));
 		$actual = $template->render($file);
 		$this->assertEquals($expected, $actual);