StrictFunctionCallsRule: more functions with $strict parameter

Author: kukulich

src/Rules/StrictCalls/StrictFunctionCallsRule.php

@@ -9,6 +9,8 @@ class StrictFunctionCallsRule implements \PHPStan\Rules\Rule
 	private $functionArguments = [
 		'in_array' => 2,
 		'array_search' => 2,
+		'base64_decode' => 1,
+		'array_keys' => 2,
 	];
 
 	/** @var \PHPStan\Broker\Broker */
@@ -44,6 +46,10 @@ public function processNode(\PhpParser\Node $node, \PHPStan\Analyser\Scope $scop
 			return [];
 		}
 
+		if ($functionName === 'array_keys' && !array_key_exists(1, $node->args)) {
+			return [];
+		}
+
 		$argumentPosition = $this->functionArguments[$functionName];
 		$message = sprintf('Call to function %s() requires parameter #%d to be true.', $functionName, $argumentPosition + 1);
 		if (!array_key_exists($argumentPosition, $node->args)) {

tests/Rules/StrictCalls/StrictFunctionCallsRuleTest.php

@@ -39,6 +39,30 @@ public function testRule(): void
 				'Call to function array_search() requires parameter #3 to be true.',
 				13,
 			],
+			[
+				'Call to function base64_decode() requires parameter #2 to be true.',
+				15,
+			],
+			[
+				'Call to function base64_decode() requires parameter #2 to be true.',
+				17,
+			],
+			[
+				'Call to function base64_decode() requires parameter #2 to be true.',
+				18,
+			],
+			[
+				'Call to function array_keys() requires parameter #3 to be true.',
+				20,
+			],
+			[
+				'Call to function array_keys() requires parameter #3 to be true.',
+				22,
+			],
+			[
+				'Call to function array_keys() requires parameter #3 to be true.',
+				23,
+			],
 		]);
 	}
 

tests/Rules/StrictCalls/data/strict-calls.php

@@ -12,5 +12,16 @@
 array_search(1, [1, 2, 3], false);
 Array_Search(1, [1, 2, 3]);
 
+base64_decode('abcd');
+base64_decode('abcd', true);
+base64_decode('abcd', false);
+Base64_Decode('abcd');
+
+array_keys([1, 2, 3], 1);
+array_keys([1, 2, 3], 1, true);
+array_keys([1, 2, 3], 1, false);
+Array_Keys([1, 2, 3], 1);
+array_keys([1, 2, 3]);
+
 $dynamicCall = 'foo';
 $dynamicCall();