Skip to content

Commit 5570af3

Browse files
committed
PHP/StrictInArray: add tests for namespaced names
1 parent 57a83a4 commit 5570af3

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

WordPress/Tests/PHP/StrictInArrayUnitTest.inc

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
in_array( 1, array( '1', 1, true ), true ); // Ok.
44

55
in_array( 1, array( '1', 1, true ) ); // Warning.
6-
in_array( 1, array( '1', 1, true ), false ); // Warning.
6+
\In_Array( 1, array( '1', 1, true ), false ); // Warning.
77
IN_ARRAY( 1, array( '1', 1, true ), false ); // Warning.
88

99
Foo::in_array( 1, array( '1', 1, true ) ); // Ok.
@@ -55,3 +55,12 @@ array_search(
5555
$haystack,
5656
true // Use strict typing.
5757
); // Ok.
58+
59+
/*
60+
* Safeguard correct handling of all types of namespaced function calls.
61+
*/
62+
\in_array( 1, array( '1', 2 ) ); // Bad.
63+
MyNamespace\array_search( 1, array( '1', 2 ) ); // Ok.
64+
\MyNamespace\array_keys( array( '1', 2 ) ); // Ok.
65+
namespace\in_array( 1, array( '1', 2 ) ); // The sniff should start flagging this once it can resolve relative namespaces.
66+
namespace\Sub\array_search( 1, array( '1', 2 ) ); // Ok.

WordPress/Tests/PHP/StrictInArrayUnitTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ public function getWarningList() {
5252
44 => 1,
5353
48 => 1,
5454
49 => 1,
55+
62 => 1,
5556
);
5657
}
5758
}

0 commit comments

Comments
 (0)