aaug...@bcc190cf-cafb-0310-a4f2-bffc1f526a37  committed a3ac7ba

Fixed #6669 -- Ensured database connections are marked as dirty by CursorDebugWrapper.execute/executemany. Refs #9964. Thanks james at 10gic net for the report and Claude Paroz for the patch.

  • Participants
  • Parent commits bbd9e23
  • Branches default

Comments (0)

Files changed (2)

File django/db/backends/

         self.cursor = cursor
         self.db = db
-    def __getattr__(self, attr):
+    def set_dirty(self):
         if self.db.is_managed():
+    def __getattr__(self, attr):
+        self.set_dirty()
         if attr in self.__dict__:
             return self.__dict__[attr]
 class CursorDebugWrapper(CursorWrapper):
     def execute(self, sql, params=()):
+        self.set_dirty()
         start = time()
             return self.cursor.execute(sql, params)
     def executemany(self, sql, param_list):
+        self.set_dirty()
         start = time()
             return self.cursor.executemany(sql, param_list)

File tests/regressiontests/transactions_regress/

 from django.db import connection, transaction
 from django.db.transaction import commit_on_success, commit_manually, TransactionManagementError
 from django.test import TransactionTestCase, skipUnlessDBFeature
+from django.test.utils import override_settings
 from django.utils.unittest import skipIf
 from .models import Mod, M2mA, M2mB
   "A transaction consisting of a failed operation was not closed.")
+    @override_settings(DEBUG=True)
+    def test_failing_query_transaction_closed_debug(self):
+        """
+        Regression for #6669. Same test as above, with DEBUG=True.
+        """
+        self.test_failing_query_transaction_closed()
 class TestManyToManyAddTransaction(TransactionTestCase):
     def test_manyrelated_add_commit(self):