Implemented FR #53466 (SQLite3Result::columnType() should return false after all...

diff --git a/NEWS b/NEWS
index acb6e25cbf38e4538c88d1a8852375e6d1ab815d..e022f5ce3be268cfcee61ff545d56b609b2f935a 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -85,6 +85,7 @@
 - SQLite3 extension:
   . Fixed memory leaked introduced by the NULL poisoning patch (Mateusz Kocielski, Pierre)
   . Add SQlite3_Stmt::readonly() for checking if a statement is read only. (Scott)
+  . Implemented FR #53466 (SQLite3Result::columnType() should return false after all of the rows have been fetched). (Scott)
 
 - Streams:
   . Implemented FR #26158 (open arbitrary file descriptor with fopen). (Gustavo)

diff --git a/ext/sqlite3/sqlite3.c b/ext/sqlite3/sqlite3.c
index 2e13918a1155e0cf84946eb941dc55a94e249e57..541c13f6c58d68c1b15571fad0b30ce688fbddb5 100644 (file)
--- a/ext/sqlite3/sqlite3.c
+++ b/ext/sqlite3/sqlite3.c
@@ -1585,6 +1585,10 @@ PHP_METHOD(sqlite3result, columnType)
                return;
        }
 
+       if (result_obj->complete) {
+               RETURN_FALSE;
+       }
+
        RETURN_LONG(sqlite3_column_type(result_obj->stmt_obj->stmt, column));
 }
 /* }}} */
@@ -1634,6 +1638,7 @@ PHP_METHOD(sqlite3result, fetchArray)
                        break;
 
                case SQLITE_DONE:
+                       result_obj->complete = 1;
                        RETURN_FALSE;
                        break;