codeigniter4 · paulbalandan · Aug 12, 2026 · Aug 12, 2026
diff --git a/system/CLI/CLI.php b/system/CLI/CLI.php
@@ -741,7 +741,7 @@ public static function getWidth(int $default = 80): int
             static::generateDimensions();
         }
 
-        return static::$width ?: $default;
+        return (static::$width === null || static::$width === 0) ? $default : static::$width;
     }
 
     /**
@@ -753,7 +753,7 @@ public static function getHeight(int $default = 32): int
             static::generateDimensions();
         }
 
-        return static::$height ?: $default;
+        return (static::$height === null || static::$height === 0) ? $default : static::$height;
     }
 
     /**

diff --git a/system/Commands/Utilities/Namespaces.php b/system/Commands/Utilities/Namespaces.php
@@ -145,7 +145,8 @@ private function outputCINamespaces(array $params): array
                     $pathOutput = $this->truncate(clean_path($path), $maxLength);
                 }
 
-                $path = realpath($path) ?: $path;
+                $realPath = realpath($path);
+                $path     = $realPath === false ? $path : $realPath;
 
                 $tbody[] = [
                     $ns,

diff --git a/system/Common.php b/system/Common.php
@@ -95,7 +95,8 @@ function clean_path(string $path): string
     {
         // Resolve relative paths
         try {
-            $path = realpath($path) ?: $path;
+            $realPath = realpath($path);
+            $path     = $realPath === false ? $path : $realPath;
         } catch (ErrorException|ValueError) {
             $path = 'error file path: ' . urlencode($path);
         }
@@ -1347,7 +1348,11 @@ function class_uses_recursive($class)
      */
     function trait_uses_recursive($trait)
     {
-        $traits = class_uses($trait) ?: [];
+        $traits = class_uses($trait);
+
+        if ($traits === false) {
+            return [];
+        }
 
         foreach ($traits as $trait) {
             $traits += trait_uses_recursive($trait);

diff --git a/system/Cookie/Cookie.php b/system/Cookie/Cookie.php
@@ -254,11 +254,11 @@ final public function __construct(string $name, string $value = '', array $optio
 
         // to preserve backward compatibility with array-based cookies in previous CI versions
         $prefix = ($options['prefix'] === '') ? self::$defaults['prefix'] : $options['prefix'];
-        $path   = $options['path'] ?: self::$defaults['path'];
-        $domain = $options['domain'] ?: self::$defaults['domain'];
+        $path   = ($options['path'] === '') ? self::$defaults['path'] : $options['path'];
+        $domain = ($options['domain'] === '') ? self::$defaults['domain'] : $options['domain'];
 
         // empty string SameSite should use the default for browsers
-        $samesite = $options['samesite'] ?: self::$defaults['samesite'];
+        $samesite = ($options['samesite'] === '') ? self::$defaults['samesite'] : $options['samesite'];
 
         $raw      = $options['raw'];
         $secure   = $options['secure'];
@@ -429,7 +429,7 @@ public function getOptions(): array
             'domain'   => $this->domain,
             'secure'   => $this->secure,
             'httponly' => $this->httponly,
-            'samesite' => $this->samesite ?: ucfirst(self::SAMESITE_LAX),
+            'samesite' => ($this->samesite === '') ? ucfirst(self::SAMESITE_LAX) : $this->samesite,
         ];
     }
 

diff --git a/system/Debug/Toolbar/Collectors/Database.php b/system/Debug/Toolbar/Collectors/Database.php
@@ -88,7 +88,7 @@ public static function collect(Query $query)
         $config = config(Toolbar::class);
 
         // Provide default in case it's not set
-        $max = $config->maxQueries ?: 100;
+        $max = ($config->maxQueries === 0) ? 100 : $config->maxQueries;
 
         if (count(static::$queries) < $max) {
             $queryString = $query->getQuery();

diff --git a/system/Files/File.php b/system/Files/File.php
@@ -111,7 +111,8 @@ public function getSizeByUnit(string $unit = 'b')
     public function guessExtension(): ?string
     {
         // naively get the path extension using pathinfo
-        $pathinfo = pathinfo($this->getRealPath() ?: $this->__toString()) + ['extension' => ''];
+        $realPath = $this->getRealPath();
+        $pathinfo = pathinfo($realPath === false ? $this->__toString() : $realPath) + ['extension' => ''];
 
         $proposedExtension = $pathinfo['extension'];
 
@@ -131,9 +132,10 @@ public function getMimeType(): string
             return $this->originalMimeType ?? 'application/octet-stream'; // @codeCoverageIgnore
         }
 
-        $finfo = finfo_open(FILEINFO_MIME_TYPE);
+        $finfo    = finfo_open(FILEINFO_MIME_TYPE);
+        $realPath = $this->getRealPath();
 
-        return finfo_file($finfo, $this->getRealPath() ?: $this->__toString());
+        return finfo_file($finfo, $realPath === false ? $this->__toString() : $realPath);
     }
 
     /**
@@ -159,7 +161,8 @@ public function move(string $targetPath, ?string $name = null, bool $overwrite =
         $name ??= $this->getBasename();
         $destination = $overwrite ? $targetPath . $name : $this->getDestination($targetPath . $name);
 
-        $oldName = $this->getRealPath() ?: $this->__toString();
+        $realPath = $this->getRealPath();
+        $oldName  = $realPath === false ? $this->__toString() : $realPath;
 
         if (! @rename($oldName, $destination)) {
             $error = error_get_last();

diff --git a/system/HTTP/CURLRequest.php b/system/HTTP/CURLRequest.php
@@ -587,7 +587,8 @@ private function applySslOptions(array $curlOptions, array $config): array
         // SSL Verification
         if (isset($config['verify'])) {
             if (is_string($config['verify'])) {
-                $file = realpath($config['verify']) ?: $config['verify'];
+                $realPath = realpath($config['verify']);
+                $file     = $realPath === false ? $config['verify'] : $realPath;
 
                 if (! is_file($file)) {
                     throw HTTPException::forInvalidSSLKey($config['verify']);

diff --git a/system/HTTP/ResponseTrait.php b/system/HTTP/ResponseTrait.php
@@ -195,7 +195,7 @@ public function getJSON()
             $body = service('format')->getFormatter('application/json')->format($body);
         }
 
-        return $body ?: null;
+        return ($body === null || $body === '') ? null : $body;
     }
 
     /**
@@ -541,10 +541,12 @@ public function setCookie(
 
         if (is_numeric($expire)) {
             $expire = $expire > 0 ? Time::now()->getTimestamp() + $expire : 0;
+        } else {
+            $expire = 0;
         }
 
         $cookie = new Cookie($name, $value, [
-            'expires'  => $expire ?: 0,
+            'expires'  => $expire,
             'domain'   => $domain,
             'path'     => $path,
             'prefix'   => $prefix,

diff --git a/system/Helpers/filesystem_helper.php b/system/Helpers/filesystem_helper.php
@@ -155,8 +155,9 @@ function write_file(string $path, string $data, string $mode = 'wb'): bool
      */
     function delete_files(string $path, bool $delDir = false, bool $htdocs = false, bool $hidden = false): bool
     {
-        $path = realpath($path) ?: $path;
-        $path = rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
+        $realPath = realpath($path);
+        $path     = $realPath === false ? $path : $realPath;
+        $path     = rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
 
         try {
             foreach (new RecursiveIteratorIterator(
@@ -208,7 +209,8 @@ function get_filenames(
     ): array {
         $files = [];
 
-        $sourceDir = realpath($sourceDir) ?: $sourceDir;
+        $realPath  = realpath($sourceDir);
+        $sourceDir = $realPath === false ? $sourceDir : $realPath;
         $sourceDir = rtrim($sourceDir, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
 
         try {

diff --git a/system/I18n/TimeTrait.php b/system/I18n/TimeTrait.php
@@ -87,7 +87,7 @@ public function __construct(?string $time = null, $timezone = null, ?string $loc
             }
         }
 
-        $timezone       = $timezone ?: date_default_timezone_get();
+        $timezone       = ($timezone === null || $timezone === '') ? date_default_timezone_get() : $timezone;
         $this->timezone = $timezone instanceof DateTimeZone ? $timezone : new DateTimeZone($timezone);
 
         // If the time string was a relative string (i.e. 'next Tuesday')

diff --git a/system/Pager/Pager.php b/system/Pager/Pager.php
@@ -216,7 +216,9 @@ public function getCurrentPage(string $group = 'default'): int
     {
         $this->ensureGroup($group);
 
-        return $this->groups[$group]['currentPage'] ?: 1;
+        $currentPage = $this->groups[$group]['currentPage'];
+
+        return ($currentPage === 0) ? 1 : $currentPage;
     }
 
     /**

diff --git a/system/Router/AutoRouter.php b/system/Router/AutoRouter.php
@@ -86,7 +86,8 @@ public function getRoute(string $uri, string $httpVerb): array
         // If it doesn't, no biggie - the default method name
         // has already been set.
         if ($segments !== []) {
-            $this->method = array_shift($segments) ?: $this->method;
+            $method       = array_shift($segments);
+            $this->method = $method === '' ? $this->method : $method;
         }
 
         // Prevent access to initController method

diff --git a/system/Test/Mock/MockCache.php b/system/Test/Mock/MockCache.php
@@ -144,7 +144,7 @@ public function deleteMatching(string $pattern): int
     public function increment(string $key, int $offset = 1): bool
     {
         $key  = static::validateKey($key, $this->prefix);
-        $data = $this->cache[$key] ?: null;
+        $data = $this->cache[$key] ?? null;
 
         if ($data === null) {
             $data = 0;
@@ -162,7 +162,7 @@ public function decrement(string $key, int $offset = 1): bool
     {
         $key = static::validateKey($key, $this->prefix);
 
-        $data = $this->cache[$key] ?: null;
+        $data = $this->cache[$key] ?? null;
 
         if ($data === null) {
             $data = 0;

diff --git a/system/Validation/Validation.php b/system/Validation/Validation.php
@@ -985,7 +985,8 @@ protected function splitRules(string $rules): array
             ) {
                 // the pipe is inside the brackets causing the closing bracket to
                 // not be included. so, we adjust the rule to include that portion.
-                $pos  = strpos($string, '|', $cursor + strlen($rule) + 1) ?: $length;
+                $next = strpos($string, '|', $cursor + strlen($rule) + 1);
+                $pos  = $next === false ? $length : $next;
                 $rule = substr($string, $cursor, $pos - $cursor);
             }
 

diff --git a/system/View/View.php b/system/View/View.php
@@ -239,7 +239,7 @@ public function render(string $view, ?array $options = null, ?bool $saveData = n
             ob_start();
             include $this->renderVars['file'];
 
-            return ob_get_clean() ?: '';
+            return (string) ob_get_clean();
         })();
 
         // Get back current vars
@@ -331,7 +331,7 @@ public function renderString(string $view, ?array $options = null, ?bool $saveDa
             ob_start();
             eval('?>' . $view);
 
-            return ob_get_clean() ?: '';
+            return (string) ob_get_clean();
         })($view);
 
         $this->logPerformance($start, microtime(true), $this->excerpt($view));

diff --git a/tests/system/Commands/Generators/CellGeneratorTest.php b/tests/system/Commands/Generators/CellGeneratorTest.php
@@ -45,7 +45,7 @@ protected function getFileContents(string $filepath): string
             return '';
         }
 
-        return file_get_contents($filepath) ?: '';
+        return (string) file_get_contents($filepath);
     }
 
     public function testGenerateCell(): void

diff --git a/tests/system/Commands/Generators/CommandGeneratorTest.php b/tests/system/Commands/Generators/CommandGeneratorTest.php
@@ -51,7 +51,7 @@ protected function getFileContents(string $filepath): string
             return '';
         }
 
-        return file_get_contents($filepath) ?: '';
+        return (string) file_get_contents($filepath);
     }
 
     public function testGenerateCommand(): void

diff --git a/tests/system/Commands/Generators/ControllerGeneratorTest.php b/tests/system/Commands/Generators/ControllerGeneratorTest.php
@@ -40,7 +40,7 @@ protected function getFileContents(string $filepath): string
             return '';
         }
 
-        return file_get_contents($filepath) ?: '';
+        return (string) file_get_contents($filepath);
     }
 
     public function testGenerateController(): void

diff --git a/tests/system/Commands/Generators/ModelGeneratorTest.php b/tests/system/Commands/Generators/ModelGeneratorTest.php
@@ -43,7 +43,7 @@ private function getFileContent(string $filepath): string
             return '';
         }
 
-        return file_get_contents($filepath) ?: '';
+        return (string) file_get_contents($filepath);
     }
 
     public function testGenerateModel(): void

diff --git a/tests/system/Commands/Generators/ScaffoldGeneratorTest.php b/tests/system/Commands/Generators/ScaffoldGeneratorTest.php
@@ -73,7 +73,7 @@ protected function getFileContents(string $filepath): string
             return '';
         }
 
-        return file_get_contents($filepath) ?: '';
+        return (string) file_get_contents($filepath);
     }
 
     public function testCreateComponentProducesManyFiles(): void

diff --git a/tests/system/Publisher/PublisherSupportTest.php b/tests/system/Publisher/PublisherSupportTest.php
@@ -158,7 +158,8 @@ public function testWipe(): void
     {
         $directory = rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . bin2hex(random_bytes(6));
         mkdir($directory, 0700);
-        $directory = realpath($directory) ?: $directory;
+        $realPath  = realpath($directory);
+        $directory = $realPath === false ? $directory : $realPath;
         $this->assertDirectoryExists($directory);
         config('Publisher')->restrictions[$directory] = ''; // Allow the directory
 

diff --git a/utils/phpstan-baseline/loader.neon b/utils/phpstan-baseline/loader.neon
@@ -1,4 +1,4 @@
-# total 1560 errors
+# total 1527 errors
 
 includes:
     - argument.type.neon
@@ -20,4 +20,3 @@ includes:
     - property.phpDocType.neon
     - return.type.neon
     - staticMethod.notFound.neon
-    - ternary.shortNotAllowed.neon