Skip to content

Commit 6043d96

Browse files
committed
Fixed an issue reported by Psalm
Signed-off-by: Maurício Meneghini Fauth <[email protected]>
1 parent 3482232 commit 6043d96

File tree

4 files changed

+2
-21
lines changed

4 files changed

+2
-21
lines changed

phpstan-baseline.neon

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2410,16 +2410,6 @@ parameters:
24102410
count: 2
24112411
path: tests/Utils/QueryTest.php
24122412

2413-
-
2414-
message: "#^Parameter \\#1 \\$statement of static method PhpMyAdmin\\\\SqlParser\\\\Utils\\\\Query\\:\\:replaceClauses\\(\\) expects PhpMyAdmin\\\\SqlParser\\\\Statement, null given\\.$#"
2415-
count: 1
2416-
path: tests/Utils/QueryTest.php
2417-
2418-
-
2419-
message: "#^Parameter \\#2 \\$list of static method PhpMyAdmin\\\\SqlParser\\\\Utils\\\\Query\\:\\:replaceClauses\\(\\) expects PhpMyAdmin\\\\SqlParser\\\\TokensList, null given\\.$#"
2420-
count: 1
2421-
path: tests/Utils/QueryTest.php
2422-
24232413
-
24242414
message: "#^Method PhpMyAdmin\\\\SqlParser\\\\Tests\\\\Utils\\\\RoutineTest\\:\\:getParameterProvider\\(\\) return type has no value type specified in iterable type array\\.$#"
24252415
count: 1

psalm-baseline.xml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2177,10 +2177,6 @@
21772177
<code>array</code>
21782178
<code>array</code>
21792179
</MixedInferredReturnType>
2180-
<NullArgument occurrences="2">
2181-
<code>null</code>
2182-
<code>null</code>
2183-
</NullArgument>
21842180
<UnusedVariable occurrences="3">
21852181
<code>$delimiter</code>
21862182
<code>$delimiter</code>
@@ -2229,9 +2225,4 @@
22292225
<code>array</code>
22302226
</MixedInferredReturnType>
22312227
</file>
2232-
<file src="tools/doctum-config.php">
2233-
<UndefinedClass occurrences="1">
2234-
<code>Finder</code>
2235-
</UndefinedClass>
2236-
</file>
22372228
</files>

psalm.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
<directory name="tests"/>
1616
<directory name="tools"/>
1717
<ignoreFiles>
18+
<file name="tools/doctum-config.php"/>
1819
<directory name="vendor"/>
1920
</ignoreFiles>
2021
</projectFiles>

tests/Utils/QueryTest.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -626,9 +626,8 @@ public function testReplaceNonExistingPart(): void
626626

627627
public function testReplaceClauses(): void
628628
{
629-
$this->assertEquals('', Query::replaceClauses(null, null, []));
630-
631629
$parser = new Parser('SELECT *, (SELECT 1) FROM film LIMIT 0, 10;');
630+
$this->assertSame('', Query::replaceClauses($parser->statements[0], $parser->list, []));
632631
$this->assertEquals(
633632
'SELECT *, (SELECT 1) FROM film WHERE film_id > 0 LIMIT 0, 10',
634633
Query::replaceClauses(

0 commit comments

Comments
 (0)