refactor: enable code quality level 34 for rector

phpstan-baseline.php

@@ -7453,12 +7453,6 @@
 	'count' => 1,
 	'path' => __DIR__ . '/system/Images/Handlers/BaseHandler.php',
 ];
-$ignoreErrors[] = [
-	// identifier: return.missing
-	'message' => '#^Method CodeIgniter\\\\Images\\\\Handlers\\\\BaseHandler\\:\\:__call\\(\\) should return mixed but return statement is missing\\.$#',
-	'count' => 1,
-	'path' => __DIR__ . '/system/Images/Handlers/BaseHandler.php',
-];
 $ignoreErrors[] = [
 	// identifier: missingType.iterableValue
 	'message' => '#^Method CodeIgniter\\\\Images\\\\Handlers\\\\BaseHandler\\:\\:_text\\(\\) has parameter \\$options with no value type specified in iterable type array\\.$#',
@@ -7609,12 +7603,6 @@
 	'count' => 1,
 	'path' => __DIR__ . '/system/Language/Language.php',
 ];
-$ignoreErrors[] = [
-	// identifier: missingType.iterableValue
-	'message' => '#^Method CodeIgniter\\\\Language\\\\Language\\:\\:load\\(\\) return type has no value type specified in iterable type array\\.$#',
-	'count' => 1,
-	'path' => __DIR__ . '/system/Language/Language.php',
-];
 $ignoreErrors[] = [
 	// identifier: missingType.iterableValue
 	'message' => '#^Method CodeIgniter\\\\Language\\\\Language\\:\\:parseLine\\(\\) return type has no value type specified in iterable type array\\.$#',

rector.php

@@ -12,14 +12,13 @@
  */
 
 use Rector\Caching\ValueObject\Storage\FileCacheStorage;
-use Rector\CodeQuality\Rector\Class_\CompleteDynamicPropertiesRector;
 use Rector\CodeQuality\Rector\Empty_\SimplifyEmptyCheckOnEmptyArrayRector;
 use Rector\CodeQuality\Rector\Expression\InlineIfToExplicitIfRector;
 use Rector\CodeQuality\Rector\Foreach_\UnusedForeachValueToArrayKeysRector;
 use Rector\CodeQuality\Rector\FuncCall\ChangeArrayPushToArrayAssignRector;
+use Rector\CodeQuality\Rector\FuncCall\CompactToVariablesRector;
 use Rector\CodeQuality\Rector\FunctionLike\SimplifyUselessVariableRector;
 use Rector\CodeQuality\Rector\Identical\FlipTypeControlToUseExclusiveTypeRector;
-use Rector\CodeQuality\Rector\If_\ShortenElseIfRector;
 use Rector\CodeQuality\Rector\Ternary\TernaryEmptyArrayArrayDimFetchToCoalesceRector;
 use Rector\CodingStyle\Rector\ClassMethod\FuncGetArgsToVariadicParamRector;
 use Rector\CodingStyle\Rector\ClassMethod\MakeInheritedMethodVisibilitySameAsParentRector;
@@ -171,6 +170,8 @@
 
         // Unnecessary (string) is inserted
         NullToStrictStringFuncCallArgRector::class,
+
+        CompactToVariablesRector::class,
     ])
     // auto import fully qualified class names
     ->withImportNames(removeUnusedImports: true)
@@ -185,7 +186,6 @@
         ChangeIfElseValueAssignToEarlyReturnRector::class,
         InlineIfToExplicitIfRector::class,
         PreparedValueToEarlyReturnRector::class,
-        ShortenElseIfRector::class,
         UnusedForeachValueToArrayKeysRector::class,
         ChangeArrayPushToArrayAssignRector::class,
         RemoveErrorSuppressInTryCatchStmtsRector::class,
@@ -196,7 +196,6 @@
         EmptyOnNullableObjectToInstanceOfRector::class,
         DisallowedEmptyRuleFixerRector::class,
         PrivatizeFinalClassPropertyRector::class,
-        CompleteDynamicPropertiesRector::class,
         BooleanInIfConditionRuleFixerRector::class,
         VersionCompareFuncCallToConstantRector::class,
         AddClosureVoidReturnTypeWhereNoReturnRector::class,
@@ -210,4 +209,4 @@
         // keep '\\' prefix string on string '\Foo\Bar'
         StringClassNameToClassConstantRector::SHOULD_KEEP_PRE_SLASH => true,
     ])
-    ->withCodeQualityLevel(24);
+    ->withCodeQualityLevel(34);

system/CLI/Commands.php

@@ -52,12 +52,12 @@ public function __construct($logger = null)
     /**
      * Runs a command given
      *
-     * @return int|void Exit code
+     * @return int Exit code
      */
     public function run(string $command, array $params)
     {
         if (! $this->verifyCommand($command, $this->commands)) {
-            return;
+            return EXIT_ERROR;
         }
 
         // The file would have already been loaded during the

system/CodeIgniter.php

@@ -318,7 +318,7 @@ private function configureKint(): void
      *
      * @param bool $returnResponse Used for testing purposes only.
      *
-     * @return ResponseInterface|void
+     * @return ResponseInterface|null
      */
     public function run(?RouteCollectionInterface $routes = null, bool $returnResponse = false)
     {
@@ -379,6 +379,8 @@ public function run(?RouteCollectionInterface $routes = null, bool $returnRespon
         }
 
         $this->sendResponse();
+
+        return null;
     }
 
     /**
@@ -863,7 +865,7 @@ protected function determinePath()
      * controller method and make the script go. If it's not able to, will
      * show the appropriate Page Not Found error.
      *
-     * @return ResponseInterface|string|void
+     * @return ResponseInterface|string|null
      */
     protected function startController()
     {
@@ -889,6 +891,8 @@ protected function startController()
         ) {
             throw PageNotFoundException::forControllerNotFound($this->controller, $this->method);
         }
+
+        return null;
     }
 
     /**

system/Commands/Database/MigrateRollback.php

@@ -74,7 +74,7 @@ public function run(array $params)
             $force = array_key_exists('f', $params) || CLI::getOption('f');
 
             if (! $force && CLI::prompt(lang('Migrations.rollBackConfirm'), ['y', 'n']) === 'n') {
-                return;
+                return null;
             }
             // @codeCoverageIgnoreEnd
         }
@@ -115,5 +115,7 @@ public function run(array $params)
             $this->showError($e);
             // @codeCoverageIgnoreEnd
         }
+
+        return null;
     }
 }

system/Database/BaseBuilder.php

@@ -3019,7 +3019,7 @@ protected function _delete(string $table): string
      *
      * @param array|string $table The table to inspect
      *
-     * @return string|void
+     * @return string|null
      */
     protected function trackAliases($table)
     {
@@ -3028,7 +3028,7 @@ protected function trackAliases($table)
                 $this->trackAliases($t);
             }
 
-            return;
+            return null;
         }
 
         // Does the string contain a comma?  If so, we need to separate
@@ -3048,6 +3048,8 @@ protected function trackAliases($table)
             // Store the alias, if it doesn't already exist
             $this->db->addTableAlias($alias);
         }
+
+        return null;
     }
 
     /**

system/Filters/CSRF.php

@@ -35,14 +35,14 @@ class CSRF implements FilterInterface
      *
      * @param list<string>|null $arguments
      *
-     * @return RedirectResponse|void
+     * @return RedirectResponse|null
      *
      * @throws SecurityException
      */
     public function before(RequestInterface $request, $arguments = null)
     {
         if (! $request instanceof IncomingRequest) {
-            return;
+            return null;
         }
 
         /** @var Security $security */
@@ -57,16 +57,17 @@ public function before(RequestInterface $request, $arguments = null)
 
             throw $e;
         }
+
+        return null;
     }
 
     /**
      * We don't have anything to do here.
      *
      * @param list<string>|null $arguments
-     *
-     * @return void
      */
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
     {
+        return null;
     }
 }

system/Filters/Cors.php

@@ -48,18 +48,18 @@ public function __construct(array $config = [])
     /**
      * @param list<string>|null $arguments
      *
-     * @return ResponseInterface|string|void
+     * @return ResponseInterface|null
      */
     public function before(RequestInterface $request, $arguments = null)
     {
         if (! $request instanceof IncomingRequest) {
-            return;
+            return null;
         }
 
         $this->createCorsService($arguments);
 
         if (! $this->cors->isPreflightRequest($request)) {
-            return;
+            return null;
         }
 
         /** @var ResponseInterface $response */
@@ -88,12 +88,12 @@ private function createCorsService(?array $arguments): void
     /**
      * @param list<string>|null $arguments
      *
-     * @return ResponseInterface|void
+     * @return ResponseInterface|null
      */
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
     {
         if (! $request instanceof IncomingRequest) {
-            return;
+            return null;
         }
 
         $this->createCorsService($arguments);

system/Filters/DebugToolbar.php

@@ -30,6 +30,7 @@ class DebugToolbar implements FilterInterface
      */
     public function before(RequestInterface $request, $arguments = null)
     {
+        return null;
     }
 
     /**
@@ -41,5 +42,7 @@ public function before(RequestInterface $request, $arguments = null)
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
     {
         service('toolbar')->prepare($request, $response);
+
+        return null;
     }
 }

system/Filters/FilterInterface.php

@@ -33,7 +33,7 @@ interface FilterInterface
      *
      * @param list<string>|null $arguments
      *
-     * @return RequestInterface|ResponseInterface|string|void
+     * @return RequestInterface|ResponseInterface|string|null
      */
     public function before(RequestInterface $request, $arguments = null);
 
@@ -45,7 +45,7 @@ public function before(RequestInterface $request, $arguments = null);
      *
      * @param list<string>|null $arguments
      *
-     * @return ResponseInterface|void
+     * @return ResponseInterface|null
      */
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null);
 }

system/Filters/ForceHTTPS.php

@@ -32,14 +32,14 @@ class ForceHTTPS implements FilterInterface
      *
      * @param array|null $arguments
      *
-     * @return ResponseInterface|void
+     * @return ResponseInterface|null
      */
     public function before(RequestInterface $request, $arguments = null)
     {
         $config = config(App::class);
 
         if ($config->forceGlobalSecureRequests !== true) {
-            return;
+            return null;
         }
 
         $response = service('response');
@@ -49,16 +49,17 @@ public function before(RequestInterface $request, $arguments = null)
         } catch (RedirectException $e) {
             return $e->getResponse();
         }
+
+        return null;
     }
 
     /**
      * We don't have anything to do here.
      *
      * @param array|null $arguments
-     *
-     * @return void
      */
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
     {
+        return null;
     }
 }

system/Filters/Honeypot.php

@@ -36,12 +36,14 @@ class Honeypot implements FilterInterface
     public function before(RequestInterface $request, $arguments = null)
     {
         if (! $request instanceof IncomingRequest) {
-            return;
+            return null;
         }
 
         if (service('honeypot')->hasContent($request)) {
             throw HoneypotException::isBot();
         }
+
+        return null;
     }
 
     /**
@@ -52,5 +54,7 @@ public function before(RequestInterface $request, $arguments = null)
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
     {
         service('honeypot')->attachHoneypot($response);
+
+        return null;
     }
 }

system/Filters/InvalidChars.php

@@ -48,13 +48,11 @@ class InvalidChars implements FilterInterface
      * Check invalid characters.
      *
      * @param list<string>|null $arguments
-     *
-     * @return void
      */
     public function before(RequestInterface $request, $arguments = null)
     {
         if (! $request instanceof IncomingRequest) {
-            return;
+            return null;
         }
 
         $data = [
@@ -69,17 +67,18 @@ public function before(RequestInterface $request, $arguments = null)
             $this->checkEncoding($values);
             $this->checkControl($values);
         }
+
+        return null;
     }
 
     /**
      * We don't have anything to do here.
      *
      * @param list<string>|null $arguments
-     *
-     * @return void
      */
     public function after(RequestInterface $request, ResponseInterface $response, $arguments = null)
     {
+        return null;
     }
 
     /**