[TASK] Use first class callable syntax for array_map calls

eliashaeussler · Jul 6, 2024 · c1f37a8 · c1f37a8
1 parent dd662c7
commit c1f37a8
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/Crawler/CrawlerFactory.php b/src/Crawler/CrawlerFactory.php
@@ -112,7 +112,7 @@ public function parseCrawlerOptions(string|array|null $crawlerOptions): array
         }
 
         // Handle non-associative-array crawler options
-        if ($crawlerOptions !== array_filter($crawlerOptions, 'is_string', ARRAY_FILTER_USE_KEY)) {
+        if ($crawlerOptions !== array_filter($crawlerOptions, is_string(...), ARRAY_FILTER_USE_KEY)) {
             throw new Exception\CrawlerOptionIsInvalid($crawlerOptions);
         }
 

diff --git a/src/Formatter/JsonFormatter.php b/src/Formatter/JsonFormatter.php
@@ -158,7 +158,7 @@ private function addToJson(string $path, string|bool|array $value): void
             Helper\ArrayHelper::setValueByPath($this->json, $path, $value);
         }
         if (is_array($value) && [] !== $value) {
-            Helper\ArrayHelper::setValueByPath($this->json, $path, array_map('strval', $value));
+            Helper\ArrayHelper::setValueByPath($this->json, $path, array_map(strval(...), $value));
         }
     }
 

diff --git a/src/Helper/ArrayHelper.php b/src/Helper/ArrayHelper.php
@@ -123,7 +123,7 @@ public static function trimExplode(string $subject, string $delimiter = ','): ar
         return array_values(
             array_filter(
                 array_map(
-                    'trim',
+                    trim(...),
                     explode($delimiter, $subject),
                 ),
                 static fn (string $value) => '' !== $value,

diff --git a/tests/unit/Command/CacheWarmupCommandTest.php b/tests/unit/Command/CacheWarmupCommandTest.php
@@ -584,7 +584,7 @@ public function executeCrawlsUrlsWithConfiguredStrategy(): void
             'https://www.example.org/',
         ];
 
-        self::assertSame($expected, array_map('strval', Tests\Fixtures\Classes\DummyCrawler::$crawledUrls));
+        self::assertSame($expected, array_map(strval(...), Tests\Fixtures\Classes\DummyCrawler::$crawledUrls));
     }
 
     #[Framework\Attributes\Test]