apply phpcbf

Author: sasezaki

src/Rules/ResourceOperationMethodCallRule.php

@@ -10,43 +10,47 @@
 use PHPStan\Rules\RuleErrorBuilder;
 use Sfp\ResourceOperations\ResourceOperations;
 
+use function in_array;
+use function sprintf;
+use function strtolower;
+
 /**
  * @implements Rule<Node\Expr\MethodCall>
  */
 final class ResourceOperationMethodCallRule implements Rule
 {
-	public function getNodeType(): string
-	{
-		return Node\Expr\MethodCall::class;
-	}
-
-	public function processNode(Node $node, Scope $scope): array
-	{
-		if (! $node->name instanceof Node\Identifier) {
-			// @codeCoverageIgnoreStart
-			return []; // @codeCoverageIgnoreEnd
-		}
-
-		if ($scope->getFunctionName() !== '__construct') {
-			return [];
-		}
-
-		$calledOnType = $scope->getType($node->var);
-
-		$methodNames = [];
-		foreach ($calledOnType->getObjectClassNames() as $objectClassName) {
-			$methodNames [] = $objectClassName . '::' .strtolower($node->name->name);
-		}
-
-		$errors = [];
-		foreach ($methodNames as $methodName) {
-			if (in_array($methodName, ResourceOperations::getMethods(), true)) {
-				$errors[] = RuleErrorBuilder::message(
-					sprintf("Don't resource operation inside constructor. Method %s() is called.", $methodName)
-				)->identifier('sfp-dont-operation.resourceOperationMethodCall')->build();
-			}
-		}
-
-		return $errors;
-	}
+    public function getNodeType(): string
+    {
+        return Node\Expr\MethodCall::class;
+    }
+
+    public function processNode(Node $node, Scope $scope): array
+    {
+        if (! $node->name instanceof Node\Identifier) {
+            // @codeCoverageIgnoreStart
+            return []; // @codeCoverageIgnoreEnd
+        }
+
+        if ($scope->getFunctionName() !== '__construct') {
+            return [];
+        }
+
+        $calledOnType = $scope->getType($node->var);
+
+        $methodNames = [];
+        foreach ($calledOnType->getObjectClassNames() as $objectClassName) {
+            $methodNames [] = $objectClassName . '::' . strtolower($node->name->name);
+        }
+
+        $errors = [];
+        foreach ($methodNames as $methodName) {
+            if (in_array($methodName, ResourceOperations::getMethods(), true)) {
+                $errors[] = RuleErrorBuilder::message(
+                    sprintf("Don't resource operation inside constructor. Method %s() is called.", $methodName)
+                )->identifier('sfp-dont-operation.resourceOperationMethodCall')->build();
+            }
+        }
+
+        return $errors;
+    }
 }

test/Rules/ResourceOperationMethodCallRuleTest.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace SfpTest\PHPStan\DontOperationInsideConstructor\Rules;
 
 use PHPStan\Rules\Rule;
@@ -11,18 +13,18 @@
  */
 class ResourceOperationMethodCallRuleTest extends RuleTestCase
 {
-	public function getRule(): Rule
-	{
-	    return new ResourceOperationMethodCallRule();
-	}
+    public function getRule(): Rule
+    {
+        return new ResourceOperationMethodCallRule();
+    }
 
-	public function testProcess(): void
-	{
-		$this->analyse([__DIR__ . '/data/resourceOperationMethodCall.php'], [
-			[
-				"Don't resource operation inside constructor. Method SplFileInfo::openfile() is called.",
-				8
-			]
-		]);
-	}
+    public function testProcess(): void
+    {
+        $this->analyse([__DIR__ . '/data/resourceOperationMethodCall.php'], [
+            [
+                "Don't resource operation inside constructor. Method SplFileInfo::openfile() is called.",
+                8,
+            ],
+        ]);
+    }
 }

test/Rules/data/resourceOperationMethodCall.php

@@ -1,10 +1,12 @@
 <?php
 
+declare(strict_types=1);
+
 class Test
 {
-	public function __construct()
-	{
-		$fileInfo = new SplFileInfo('test');
-		$fileInfo->openFile('r');
-	}
+    public function __construct()
+    {
+        $fileInfo = new SplFileInfo('test');
+        $fileInfo->openFile('r');
+    }
 }