allow multi static calls in one class

src/Command/PrivatizeConstantsCommand.php

@@ -6,7 +6,6 @@
 
 use Nette\Utils\FileSystem;
 use Nette\Utils\Strings;
-use Rector\SwissKnife\FileSystem\PathHelper;
 use Rector\SwissKnife\Finder\PhpFilesFinder;
 use Rector\SwissKnife\Helpers\ClassNameResolver;
 use Rector\SwissKnife\PHPStan\ClassConstantResultAnalyser;
@@ -75,7 +74,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $this->privatizeClassConstants($phpFileInfos);
 
         // special case of self::NAME, that should be protected - their children too
-        $staticClassConstsMatches = $this->staticClassConstResolver->resolve($phpFileInfos);
+        $staticClassConstMatches = $this->staticClassConstResolver->resolve($phpFileInfos);
 
         $phpstanResult = $this->runPHPStanAnalyse($sources);
 
@@ -94,7 +93,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $this->replacePrivateConstWith($publicClassConstMatch, 'protected const');
         }
 
-        $this->replaceClassAndChildWithProtected($phpFileInfos, $staticClassConstsMatches);
+        $this->replaceClassAndChildWithProtected($phpFileInfos, $staticClassConstMatches);
 
         if ($publicAndProtectedClassConstants->getPublicCount() !== 0) {
             $this->symfonyStyle->success(
@@ -108,9 +107,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             );
         }
 
-        if ($staticClassConstsMatches !== []) {
+        if ($staticClassConstMatches !== []) {
             $this->symfonyStyle->success(
-                \sprintf('%d constants made protected for static access', count($staticClassConstsMatches))
+                \sprintf('%d constants made protected for static access', count($staticClassConstMatches))
             );
         }
 

src/Resolver/StaticClassConstResolver.php

@@ -11,6 +11,8 @@
 
 /**
  * Resolve all "static::SOME_CONST" calls
+ *
+ * @see \Rector\SwissKnife\Tests\Resolver\StaticClassConstResolver\StaticClassConstResolverTest
  */
 final class StaticClassConstResolver
 {
@@ -28,17 +30,19 @@ public function resolve(array $phpFileInfos): array
     {
         $staticConstMatches = [];
         foreach ($phpFileInfos as $phpFileInfo) {
-            $match = Strings::match($phpFileInfo->getContents(), self::STATIC_CONST_CALL_REGEX);
-            if ($match === null) {
-                continue;
+            $matches = Strings::matchAll($phpFileInfo->getContents(), self::STATIC_CONST_CALL_REGEX);
+            foreach ($matches as $match) {
+                if ($match === null) {
+                    continue;
+                }
+
+                $fullyQualifiedClassName = ClassNameResolver::resolveFromFileContents($phpFileInfo->getContents());
+                if ($fullyQualifiedClassName === null) {
+                    continue;
+                }
+
+                $staticConstMatches[] = new ClassConstMatch($fullyQualifiedClassName, $match['constant_name']);
             }
-
-            $fullyQualifiedClassName = ClassNameResolver::resolveFromFileContents($phpFileInfo->getContents());
-            if ($fullyQualifiedClassName === null) {
-                continue;
-            }
-
-            $staticConstMatches[] = new ClassConstMatch($fullyQualifiedClassName, $match['constant_name']);
         }
 
         return $staticConstMatches;

tests/Resolver/StaticClassConstResolver/Fixture/FileWithStaticConstCalls.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\SwissKnife\Tests\Resolver\StaticClassConstResolver\Fixture;
+
+final class FileWithStaticConstCalls
+{
+    public const ITEM_NAME = '...';
+
+    public const ITEM_PRICE = 100;
+
+    public function run()
+    {
+        $hash = static::ITEM_NAME . '_' . static::ITEM_PRICE;
+    }
+}

tests/Resolver/StaticClassConstResolver/StaticClassConstResolverTest.php

@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\SwissKnife\Tests\Resolver\StaticClassConstResolver;
+
+use PHPUnit\Framework\TestCase;
+use Rector\SwissKnife\Resolver\StaticClassConstResolver;
+use Rector\SwissKnife\Tests\Resolver\StaticClassConstResolver\Fixture\FileWithStaticConstCalls;
+use Symfony\Component\Finder\SplFileInfo;
+
+final class StaticClassConstResolverTest extends TestCase
+{
+    private StaticClassConstResolver $staticClassConstResolver;
+
+    protected function setUp(): void
+    {
+        $this->staticClassConstResolver = new StaticClassConstResolver();
+    }
+
+    public function test(): void
+    {
+        $splFileInfo = new SplFileInfo(__DIR__ . '/Fixture/FileWithStaticConstCalls.php', '', '');
+
+        $classConstMatches = $this->staticClassConstResolver->resolve([$splFileInfo]);
+        $this->assertCount(2, $classConstMatches);
+
+        $firstClassConstMatch = $classConstMatches[0];
+        $this->assertSame(FileWithStaticConstCalls::class, $firstClassConstMatch->getClassName());
+        $this->assertSame('ITEM_NAME', $firstClassConstMatch->getConstantName());
+
+        $secondClassConstMatch = $classConstMatches[1];
+        $this->assertSame(FileWithStaticConstCalls::class, $secondClassConstMatch->getClassName());
+        $this->assertSame('ITEM_PRICE', $secondClassConstMatch->getConstantName());
+    }
+}