Release v4.3.6

Author: actions-user

system/Autoloader/FileLocator.php

@@ -33,8 +33,13 @@ public function __construct(Autoloader $autoloader)
      * Attempts to locate a file by examining the name for a namespace
      * and looking through the PSR-4 namespaced files that we know about.
      *
-     * @param string      $file   The namespaced file to locate
-     * @param string|null $folder The folder within the namespace that we should look for the file.
+     * @param string      $file   The relative file path or namespaced file to
+     *                            locate. If not namespaced, search in the app
+     *                            folder.
+     * @param string|null $folder The folder within the namespace that we should
+     *                            look for the file. If $file does not contain
+     *                            this value, it will be appended to the namespace
+     *                            folder.
      * @param string      $ext    The file extension the file should have.
      *
      * @return false|string The path to the file, or false if not found.

system/CodeIgniter.php

@@ -47,7 +47,7 @@ class CodeIgniter
     /**
      * The current version of CodeIgniter Framework
      */
-    public const CI_VERSION = '4.3.5';
+    public const CI_VERSION = '4.3.6';
 
     /**
      * App startup time.
@@ -310,8 +310,6 @@ private function configureKint(): void
      * makes all of the pieces work together.
      *
      * @return ResponseInterface|void
-     *
-     * @throws RedirectException
      */
     public function run(?RouteCollectionInterface $routes = null, bool $returnResponse = false)
     {

system/Common.php

@@ -744,11 +744,7 @@ function is_windows(?bool $mock = null): bool
             $mocked = $mock;
         }
 
-        if (isset($mocked)) {
-            return $mocked;
-        }
-
-        return DIRECTORY_SEPARATOR === '\\';
+        return $mocked ?? DIRECTORY_SEPARATOR === '\\';
     }
 }
 

system/Config/BaseConfig.php

@@ -86,7 +86,7 @@ public function __construct()
     /**
      * Initialization an environment-specific configuration setting
      *
-     * @param mixed $property
+     * @param array|bool|float|int|string|null $property
      *
      * @return void
      */

system/Config/Factories.php

@@ -24,6 +24,7 @@
  * instantiation checks.
  *
  * @method static BaseConfig|null config(...$arguments)
+ * @method static Model|null      models(string $name, array $options = [], ?ConnectionInterface &$conn = null)
  */
 class Factories
 {
@@ -69,23 +70,6 @@ class Factories
      */
     protected static $instances = [];
 
-    /**
-     * This method is only to prevent PHPStan error.
-     * If we have a solution, we can remove this method.
-     * See https://github.com/codeigniter4/CodeIgniter4/pull/5358
-     *
-     * @template T of Model
-     *
-     * @phpstan-param class-string<T> $name
-     *
-     * @return Model
-     * @phpstan-return T
-     */
-    public static function models(string $name, array $options = [], ?ConnectionInterface &$conn = null)
-    {
-        return self::__callStatic('models', [$name, $options, $conn]);
-    }
-
     /**
      * Loads instances based on the method component name. Either
      * creates a new instance or returns an existing shared instance.

system/Database/BaseBuilder.php

@@ -1997,7 +1997,7 @@ protected function _upsertBatch(string $table, array $keys, array $values): stri
         }
 
         if (isset($this->QBOptions['setQueryAsData'])) {
-            $data = $this->QBOptions['setQueryAsData'];
+            $data = $this->QBOptions['setQueryAsData'] . "\n";
         } else {
             $data = 'VALUES ' . implode(', ', $this->formatValues($values)) . "\n";
         }

system/Database/Config.php

@@ -37,7 +37,7 @@ class Config extends BaseConfig
     protected static $factory;
 
     /**
-     * Creates the default
+     * Returns the database connection
      *
      * @param array|BaseConnection|string|null $group The name of the connection group to use,
      *                                                or an array of configuration settings.

system/Database/Database.php

@@ -16,7 +16,7 @@
 /**
  * Database Connection Factory
  *
- * Creates and returns an instance of the appropriate DatabaseConnection
+ * Creates and returns an instance of the appropriate Database Connection.
  */
 class Database
 {
@@ -32,8 +32,7 @@ class Database
     protected $connections = [];
 
     /**
-     * Parses the connection binds and returns an instance of the driver
-     * ready to go.
+     * Parses the connection binds and creates a Database Connection instance.
      *
      * @return BaseConnection
      *
@@ -83,7 +82,7 @@ public function loadUtils(ConnectionInterface $db): BaseUtils
     }
 
     /**
-     * Parse universal DSN string
+     * Parses universal DSN string
      *
      * @throws InvalidArgumentException
      */
@@ -121,21 +120,20 @@ protected function parseDSN(array $params): array
     }
 
     /**
-     * Initialize database driver.
+     * Creates a database object.
      *
      * @param string       $driver   Driver name. FQCN can be used.
-     * @param array|object $argument
+     * @param string       $class    'Connection'|'Forge'|'Utils'
+     * @param array|object $argument The constructor parameter.
      *
      * @return BaseConnection|BaseUtils|Forge
      */
     protected function initDriver(string $driver, string $class, $argument): object
     {
-        $class = $driver . '\\' . $class;
+        $classname = (strpos($driver, '\\') === false)
+            ? "CodeIgniter\\Database\\{$driver}\\{$class}"
+            : $driver . '\\' . $class;
 
-        if (strpos($driver, '\\') === false) {
-            $class = "CodeIgniter\\Database\\{$class}";
-        }
-
-        return new $class($argument);
+        return new $classname($argument);
     }
 }

system/Database/Postgre/Connection.php

@@ -64,14 +64,11 @@ public function connect(bool $persistent = false)
             $this->buildDSN();
         }
 
-        // Strip pgsql if exists
+        // Convert DSN string
         if (mb_strpos($this->DSN, 'pgsql:') === 0) {
-            $this->DSN = mb_substr($this->DSN, 6);
+            $this->convertDSN();
         }
 
-        // Convert semicolons to spaces.
-        $this->DSN = str_replace(';', ' ', $this->DSN);
-
         $this->connID = $persistent === true ? pg_pconnect($this->DSN) : pg_connect($this->DSN);
 
         if ($this->connID !== false) {
@@ -92,6 +89,44 @@ public function connect(bool $persistent = false)
         return $this->connID;
     }
 
+    /**
+     * Converts the DSN with semicolon syntax.
+     */
+    private function convertDSN()
+    {
+        // Strip pgsql
+        $this->DSN = mb_substr($this->DSN, 6);
+
+        // Convert semicolons to spaces in DSN format like:
+        // pgsql:host=localhost;port=5432;dbname=database_name
+        // https://www.php.net/manual/en/function.pg-connect.php
+        $allowedParams = ['host', 'port', 'dbname', 'user', 'password', 'connect_timeout', 'options', 'sslmode', 'service'];
+
+        $parameters = explode(';', $this->DSN);
+
+        $output            = '';
+        $previousParameter = '';
+
+        foreach ($parameters as $parameter) {
+            [$key, $value] = explode('=', $parameter, 2);
+            if (in_array($key, $allowedParams, true)) {
+                if ($previousParameter !== '') {
+                    if (array_search($key, $allowedParams, true) < array_search($previousParameter, $allowedParams, true)) {
+                        $output .= ';';
+                    } else {
+                        $output .= ' ';
+                    }
+                }
+                $output .= $parameter;
+                $previousParameter = $key;
+            } else {
+                $output .= ';' . $parameter;
+            }
+        }
+
+        $this->DSN = $output;
+    }
+
     /**
      * Keep or establish the connection if no queries have been sent for
      * a length of time exceeding the server's idle timeout.

system/Encryption/Encryption.php

@@ -149,7 +149,7 @@ public static function createKey($length = 32)
      *
      * @param string $key Property name
      *
-     * @return mixed
+     * @return array|string|null
      */
     public function __get($key)
     {