phpstan fix (#43) (#44)

Author: janbarasek

src/Bridges/SecurityDI/SecurityExtension.php

@@ -50,6 +50,7 @@ public function getConfigSchema(): Nette\Schema\Schema
 
 	public function loadConfiguration()
 	{
+		/** @var object{debugger: bool, users: array, roles: array, resources: array} $config */
 		$config = $this->config;
 		$builder = $this->getContainerBuilder();
 

src/Security/Permission.php

@@ -597,20 +597,17 @@ public function isAllowed($role = self::ALL, $resource = self::ALL, $privilege =
 						break;
 					}
 				}
-			} else {
-				if (($result = $this->getRuleType($resource, null, $privilege)) !== null) { // look for rule on 'allRoles' pseudo-parent
-					break;
-
-				} elseif (($result = $this->getRuleType($resource, null, null)) !== null) {
-					break;
-				}
+			} elseif (($result = $this->getRuleType($resource, null, $privilege)) !== null) { // look for rule on 'allRoles' pseudo-parent
+				break;
+			} elseif (($result = $this->getRuleType($resource, null, null)) !== null) {
+				break;
 			}
 
 			$resource = $this->resources[$resource]['parent']; // try next Resource
 		} while (true);
 
 		$this->queriedRole = $this->queriedResource = null;
-		return $result;
+		return $result ?? false;
 	}
 
 

tests/phpstan.neon

@@ -3,3 +3,5 @@ includes:
 
 parameters:
 	treatPhpDocTypesAsCertain: false
+	ignoreErrors:
+		- '#Variable \$this in isset\(\) always exists and is not nullable\.#'