Change early returns

OpenConext · Feb 13, 2024 · d75beff · d75beff
1 parent 88636c4
commit d75beff
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/Surfnet/NoDebugFunctionRule.php b/src/Surfnet/NoDebugFunctionRule.php
@@ -42,11 +42,13 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        if ($node->name instanceof Node\Name) {
-            $functionName = $node->name->toString();
-            if (in_array($functionName, ['dd', 'dump', 'var_dump', 'print_r', 'exit', 'die'])) {
-                return ["Do not use {$functionName} function"];
-            }
+        if (!$node->name instanceof Node\Name) {
+            return [];
+        }
+
+        $functionName = $node->name->toString();
+        if (in_array($functionName, ['dd', 'dump', 'var_dump', 'print_r', 'exit', 'die'])) {
+            return ["Do not use {$functionName} function"];
         }
 
         return [];