[Oracle] Fixes #52678 transaction creation

troopa81 · nyalldawson · commit 2fa79220b1e1 · 2023-04-19T06:28:31.000+10:00
diff --git a/src/providers/oracle/qgsoracleconn.cpp b/src/providers/oracle/qgsoracleconn.cpp
@@ -526,7 +526,7 @@ bool QgsOracleConn::execLogged( const QString &query, bool logError, QString *er
 
   QSqlQuery qry( mDatabase );
 
-  const bool res { !exec( qry, query, QVariantList() ) };
+  const bool res { exec( qry, query, QVariantList() ) };
 
   logWrapper.setQuery( qry.lastQuery() );
 
diff --git a/tests/src/python/test_provider_oracle.py b/tests/src/python/test_provider_oracle.py
@@ -734,6 +734,15 @@ def testNestedInsert(self):
         self.vl.addFeature(f)  # Should not deadlock during an active iteration
         f = next(it)
 
+    def testTransactionEditing(self):
+        tg = QgsTransactionGroup()
+        tg.addLayer(self.vl)
+        self.vl.startEditing()
+        feat = QgsFeature(self.vl.fields())
+        feat.setAttribute("pk", 6)
+        self.assertTrue(self.vl.addFeature(feat))
+        self.vl.rollBack()
+
     def testTimeout(self):
         """
         Asserts that we will not deadlock if more iterators are opened in parallel than
