Merge pull request #15 from thecodingmachine/phpstan0.12

moufmouf · web-flow · commit 2a62cce9060c · 2019-12-04T10:54:29.000+01:00
Adding PHPStan 0.12 compatibility
diff --git a/composer.json b/composer.json
@@ -12,7 +12,7 @@
     "require": {
         "php": "^7.1",
         "phpstan/phpstan": "^0.10 | ^0.11 | ^0.12",
-        "thecodingmachine/safe": "^0.1.11"
+        "thecodingmachine/safe": "^1.0-beta"
     },
     "require-dev": {
         "phpunit/phpunit": "^7.5.2",
diff --git a/src/Rules/UseSafeFunctionsRule.php b/src/Rules/UseSafeFunctionsRule.php
@@ -16,6 +16,8 @@
 
 /**
  * This rule checks that no superglobals are used in code.
+ *
+ * @implements Rule<Node\Expr\FuncCall>
  */
 class UseSafeFunctionsRule implements Rule
 {
diff --git a/src/Utils/FunctionListLoader.php b/src/Utils/FunctionListLoader.php
@@ -9,6 +9,9 @@
 
 class FunctionListLoader
 {
+    /**
+     * @var string[]
+     */
     private static $functions;
 
     /**
diff --git a/tests/Rules/CallMethodRuleTest.php b/tests/Rules/CallMethodRuleTest.php
@@ -17,7 +17,7 @@ protected function getRule(): Rule
         $ruleLevelHelper = new RuleLevelHelper($broker, true, true, true);
         return new CallMethodsRule(
             $broker,
-            new FunctionCallParametersCheck($ruleLevelHelper, true, true),
+            new FunctionCallParametersCheck($ruleLevelHelper, true, true, true, true),
             $ruleLevelHelper,
             true,
             true

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,8 @@`
`16`	`16`
`17`	`17`	`/**`
`18`	`18`	`* This rule checks that no superglobals are used in code.`
	`19`	`+ *`
	`20`	`+ * @implements Rule<Node\Expr\FuncCall>`
`19`	`21`	`*/`
`20`	`22`	`class UseSafeFunctionsRule implements Rule`
`21`	`23`	`{`