Commits

Anonymous 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

Comments (0)

Files changed (2)

File django/db/backends/util.py

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

File tests/regressiontests/transactions_regress/tests.py

 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
         except:
             self.fail("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):