refactor: fix minor type inaccuracies found via PHPStan bleeding edge

Author: paulbalandan

system/Database/Database.php

@@ -58,7 +58,7 @@ public function load(array $params = [], string $alias = '')
             throw new InvalidArgumentException('You have not selected a database type to connect to.');
         }
 
-        assert($this->checkDbExtension($params['DBDriver']));
+        $this->checkDbExtension($params['DBDriver']);
 
         $this->connections[$alias] = $this->initDriver($params['DBDriver'], 'Connection', $params);
 
@@ -158,11 +158,11 @@ protected function initDriver(string $driver, string $class, $argument): object
      *
      * @param string $driver DB driver or FQCN for custom driver
      */
-    private function checkDbExtension(string $driver): bool
+    private function checkDbExtension(string $driver): void
     {
         if (str_contains($driver, '\\')) {
             // Cannot check a fully qualified classname for a custom driver.
-            return true;
+            return;
         }
 
         $extensionMap = [
@@ -183,7 +183,7 @@ private function checkDbExtension(string $driver): bool
         }
 
         if (extension_loaded($extension)) {
-            return true;
+            return;
         }
 
         $message = 'The required PHP extension "' . $extension . '" is not loaded.'

system/HTTP/Files/FileCollection.php

@@ -189,7 +189,7 @@ protected function createFileObject(array $array)
 
         return new UploadedFile(
             $array['tmp_name'] ?? null,
-            $array['name'] ?? null,
+            $array['name'],
             $array['type'] ?? null,
             ($array['size'] ?? null) === null ? null : (int) $array['size'],
             $array['error'] ?? null,

system/HTTP/SiteURIFactory.php

@@ -227,7 +227,7 @@ private function createURIFromRoutePath(string $routePath): SiteURI
      */
     private function getHost(): ?string
     {
-        $httpHostPort = $this->superglobals->server('HTTP_HOST') ?? null;
+        $httpHostPort = $this->superglobals->server('HTTP_HOST');
 
         if ($httpHostPort !== null) {
             [$httpHost] = explode(':', $httpHostPort, 2);

system/Helpers/Array/ArrayHelper.php

@@ -324,7 +324,7 @@ public static function recursiveCount(array $array, int $counter = 0): int
      * @param list<int|list<int|string>|string> $array
      * @param int|string|null                   $sortByIndex
      */
-    public static function sortValuesByNatural(array &$array, $sortByIndex = null): bool
+    public static function sortValuesByNatural(array &$array, $sortByIndex = null): true
     {
         return usort($array, static function ($currentValue, $nextValue) use ($sortByIndex): int {
             if ($sortByIndex !== null) {

system/Router/RouteCollection.php

@@ -1520,7 +1520,7 @@ private function checkSubdomains($subdomains): bool
 
         // Routes can be limited to any sub-domain. In that case, though,
         // it does require a sub-domain to be present.
-        if (! in_array($this->currentSubdomain, [null, ''], true) && in_array('*', $subdomains, true)) {
+        if ($this->currentSubdomain !== '' && in_array('*', $subdomains, true)) {
             return true;
         }