fix(deprecation warning): str_replace(): Passing null to parameter #3 ($subject)

Remaining self deprecation notices (1)

  1x: str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated
    1x in RateLimitAnnotationListenerTest::testDispatchIsCalledIfThePathLimitProcessorReturnsARateLimit from Noxlogic\RateLimitBundle\Tests\EventListener

Author: orlandothoeny

DependencyInjection/Configuration.php

@@ -26,7 +26,7 @@ public function getConfigTreeBuilder(): TreeBuilder
             ->canBeDisabled()
             ->children()
                 ->enumNode('storage_engine')
-                    ->values(array('redis','memcache','doctrine', 'php_redis', 'php_redis_cluster', 'simple_cache', 'cache'))
+                    ->values(['redis','memcache','doctrine', 'php_redis', 'php_redis_cluster', 'simple_cache', 'cache'])
                     ->defaultValue('redis')
                     ->info('The storage engine where all the rates will be stored')
                 ->end()
@@ -109,19 +109,20 @@ public function getConfigTreeBuilder(): TreeBuilder
                     ->end()
                 ->end()
                 ->arrayNode('path_limits')
-                    ->defaultValue(array())
+                    ->defaultValue([])
                     ->info('Rate limits for paths')
                     ->prototype('array')
                         ->children()
                             ->scalarNode('path')
                                 ->isRequired()
+                                ->cannotBeEmpty()
                             ->end()
                             ->arrayNode('methods')
                                 ->prototype('enum')
-                                    ->values(array('*', 'GET', 'POST', 'PUT', 'DELETE', 'PATCH'))
+                                    ->values(['*', 'GET', 'POST', 'PUT', 'DELETE', 'PATCH'])
                                 ->end()
                                 ->requiresAtLeastOneElement()
-                                ->defaultValue(array('*'))
+                                ->defaultValue(['*'])
                             ->end()
                             ->integerNode('limit')
                                 ->isRequired()

Util/PathLimitProcessor.php

@@ -8,20 +8,19 @@
 
 class PathLimitProcessor
 {
-    private array $pathLimits;
-
-    public function __construct(array $pathLimits)
+    /**
+     * @param array<array{path: string, methods: array<string>, limit: int<-1, max>, period: positive-int}> $pathLimits
+     */
+    public function __construct(private array $pathLimits)
     {
-        $this->pathLimits = $pathLimits;
-
         // Clean up any extra slashes from the config
         foreach ($this->pathLimits as &$pathLimit) {
             $pathLimit['path'] = trim($pathLimit['path'], '/');
         }
 
         // Order the configs so that the most specific paths
         // are matched first
-        usort($this->pathLimits, static function($a, $b) {
+        usort($this->pathLimits, static function($a, $b): int {
             return substr_count($b['path'], '/') - substr_count($a['path'], '/');
         });
     }
@@ -44,7 +43,7 @@ public function getRateLimit(Request $request): ?RateLimit
         return null;
     }
 
-    public function getMatchedPath(Request $request)
+    public function getMatchedPath(Request $request): string
     {
         $path = trim($request->getPathInfo(), '/');
         $method = $request->getMethod();

phpstan-baseline.neon

@@ -384,18 +384,6 @@ parameters:
 			count: 1
 			path: Service/Storage/StorageInterface.php
 
-		-
-			message: '#^Method Noxlogic\\RateLimitBundle\\Util\\PathLimitProcessor\:\:__construct\(\) has parameter \$pathLimits with no value type specified in iterable type array\.$#'
-			identifier: missingType.iterableValue
-			count: 1
-			path: Util/PathLimitProcessor.php
-
-		-
-			message: '#^Method Noxlogic\\RateLimitBundle\\Util\\PathLimitProcessor\:\:getMatchedPath\(\) has no return type specified\.$#'
-			identifier: missingType.return
-			count: 1
-			path: Util/PathLimitProcessor.php
-
 		-
 			message: '#^Method Noxlogic\\RateLimitBundle\\Util\\PathLimitProcessor\:\:methodMatched\(\) has parameter \$expectedMethods with no value type specified in iterable type array\.$#'
 			identifier: missingType.iterableValue
@@ -437,9 +425,3 @@ parameters:
 			identifier: missingType.parameter
 			count: 1
 			path: Util/PathLimitProcessor.php
-
-		-
-			message: '#^Property Noxlogic\\RateLimitBundle\\Util\\PathLimitProcessor\:\:\$pathLimits type has no value type specified in iterable type array\.$#'
-			identifier: missingType.iterableValue
-			count: 1
-			path: Util/PathLimitProcessor.php