spip / SPIP

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>

ecrire/tests/Squelettes/Balise/BaliseDynamiqueLangTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 class BaliseDynamiqueLangTest extends SquelettesTestCase
 {
 	public static function setUpBeforeClass(): void {
-		$GLOBALS['dossier_squelettes'] = self::relativePath(__DIR__ . '/data/squelettes');
+		$GLOBALS['dossier_squelettes'] = self::relativePath(__DIR__.'/data/squelettes');
 	}
 
 	/**

ecrire/tests/Squelettes/Balise/ExposeTest.php

Spacing +6 added lines, -6 removed lines

@@ -15,14 +15,14 @@ 
 
 	public function testExposerRubrique(): void {
 		$id_rubrique = sql_getfetsel('id_rubrique', 'spip_rubriques', [
-			'id_parent=' . sql_quote(0),
-			'statut=' . sql_quote('publie'),
+			'id_parent='.sql_quote(0),
+			'statut='.sql_quote('publie'),
 		]);
 
 		$id_seconde_rubrique = sql_getfetsel('id_rubrique', 'spip_rubriques', [
-			'id_parent=' . sql_quote(0),
-			'statut=' . sql_quote('publie'),
-			'id_rubrique != ' . (int) $id_rubrique,
+			'id_parent='.sql_quote(0),
+			'statut='.sql_quote('publie'),
+			'id_rubrique != '.(int) $id_rubrique,
 		]);
 
 		if (!$id_rubrique || !$id_seconde_rubrique) {
@@ -50,6 +50,6 @@ 
 
 	#[Depends('testExposerRubrique')]
 	public function testExposerRubriqueInclus(): void {
-		$this->assertOkSquelette(__DIR__ . '/data/balise_expose.html');
+		$this->assertOkSquelette(__DIR__.'/data/balise_expose.html');
 	}
 }