Merge pull request #5278 from paulbalandan/pgsql-result

Fix compatibility of Postgres result for PHP 8.1
codeigniter4 · Nov 3, 2021 · 4995720 · 4995720
2 parents af247f4 + e94394a
commit 4995720
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/system/Database/BaseResult.php b/system/Database/BaseResult.php
@@ -28,7 +28,7 @@ abstract class BaseResult implements ResultInterface
     /**
      * Result ID
      *
-     * @var bool|object|resource
+     * @var false|object|resource
      */
     public $resultID;
 

diff --git a/system/Database/Postgre/Result.php b/system/Database/Postgre/Result.php
@@ -68,7 +68,7 @@ public function getFieldData(): array
      */
     public function freeResult()
     {
-        if (is_resource($this->resultID)) {
+        if ($this->resultID !== false) {
             pg_free_result($this->resultID);
             $this->resultID = false;
         }