spip / SPIP

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

ecrire/tests/Squelettes/Critere/DoublonsTest.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 	}
 
 	public function testCritereDoublonsMultiplesBoucles(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/doublons.html');
+		$this->assertOkSquelette(__DIR__.'/data/doublons.html');
 	}
 
 	public function testDoublonsAuteurs(): void {

ecrire/tests/Squelettes/Critere/DoublonsNotesTest.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
 		$id_document = sql_getfetsel(
 			'id_document',
 			'spip_documents',
-			sql_in('mode', ['logoon','logooff','vignette'], 'not'),
+			sql_in('mode', ['logoon', 'logooff', 'vignette'], 'not'),
 			orderby: 'rand()',
 			limit: '0,1'
 		);
@@ -61,7 +61,7 @@ 
 			'id_article' => -1,
 			'titre' => 'test pour doublons_notes.html',
 			'statut' => 'prepa',
-			'texte' => 'hello [[ xx <doc' . $id_document . '> ]].'
+			'texte' => 'hello [[ xx <doc'.$id_document.'> ]].'
 		];
 		$id_article = sql_getfetsel('id_article', 'spip_articles', 'id_article = -1');
 		if ($id_article === null) {

ecrire/tests/Squelettes/Critere/OperatorRegexpLikeTest.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 		$art = $this->getArticleIdTitle();
 		$contexte = [
 			'id_article' => $art['id_article'],
-			'like' => $art['starts_with'] . '%',
+			'like' => $art['starts_with'].'%',
 		];
 		$this->assertOkCode(<<<SPIP
 			<BOUCLE_b(ARTICLES){titre like #ENV{like}}{id_article}>ok</BOUCLE_b>
@@ -70,7 +70,7 @@ 
 		$art = $this->getArticleIdTitle();
 		$contexte = [
 			'id_article' => $art['id_article'],
-			'regexp' => '^' . $art['starts_with'],
+			'regexp' => '^'.$art['starts_with'],
 		];
 		$this->assertOkCode(<<<SPIP
 			<BOUCLE_b(ARTICLES){titre == #ENV{regexp}}{id_article}>ok</BOUCLE_b>