Fixed bug with execution of a PL/SQL block containing at least one

output bind variable immediately following a query that returned
multiple duplicate rows.

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -26,6 +26,9 @@ Thin Mode Changes
     value of the attribute :attr:`DeqOptions.deliverymode`.
 #)  Fixed bug with detection of when a connection has been closed by the
     database without notification.
+#)  Fixed bug with execution of a PL/SQL block containing at least one output
+    bind variable immediately following a query that returned multiple
+    duplicate rows.
 
 Thick Mode Changes
 ++++++++++++++++++

src/oracledb/impl/thin/messages/base.pyx

@@ -806,7 +806,7 @@ cdef class MessageWithData(Message):
 
                 # retain last raw value when not fetching Arrow (for handling
                 # duplicate rows)
-                if not self.cursor_impl.fetching_arrow:
+                if self.in_fetch and not self.cursor_impl.fetching_arrow:
                     var_impl._last_raw_value = \
                             var_impl._values[self.cursor_impl._last_row_index]
 

tests/test_4300_cursor_other.py

@@ -997,6 +997,19 @@ def test_4369(self):
             cursor = conn.cursor()
         self.assertEqual(cursor.rowcount, -1)
 
+    def test_4370(self):
+        "4370 - execute PL/SQL with out vars after query with duplicate data"
+        self.cursor.execute("truncate table TestTempTable")
+        self.cursor.executemany(
+            "insert into TestTempTable (IntCol, StringCol1) values (:1, :2)",
+            [(i + 1, "test_4370") for i in range(20)],
+        )
+        self.conn.commit()
+        self.cursor.execute("select IntCol, StringCol1 from TestTempTable")
+        var = self.cursor.var(int)
+        self.cursor.execute("begin :1 := 4370; end;", [var])
+        self.assertEqual(var.getvalue(), 4370)
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()

tests/test_6300_cursor_other_async.py

@@ -913,6 +913,21 @@ async def test_6353(self):
             cursor = conn.cursor()
         self.assertEqual(cursor.rowcount, -1)
 
+    async def test_6354(self):
+        "6354 - execute PL/SQL with out vars after query with duplicate data"
+        await self.cursor.execute("truncate table TestTempTable")
+        await self.cursor.executemany(
+            "insert into TestTempTable (IntCol, StringCol1) values (:1, :2)",
+            [(i + 1, "test_4370") for i in range(20)],
+        )
+        await self.conn.commit()
+        await self.cursor.execute(
+            "select IntCol, StringCol1 from TestTempTable"
+        )
+        var = self.cursor.var(int)
+        await self.cursor.execute("begin :1 := 4370; end;", [var])
+        self.assertEqual(var.getvalue(), 4370)
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()