Ticket #17300: 17300-3.diff

django/core/management/commands/createcachetable.py

@@ -3 +3 @@
 from django.core.cache.backends.db import BaseDatabaseCache
 from django.core.management.base import LabelCommand
 from django.db import connections, router, transaction, models, DEFAULT_DB_ALIAS
+from django.db.utils import DatabaseError
 
 class Command(LabelCommand):
     help = "Creates the table needed to use the SQL cache backend."
@@ -51 +52 @@
             full_statement.append('    %s%s' % (line, i < len(table_output)-1 and ',' or ''))
         full_statement.append(');')
         curs = connection.cursor()
-        curs.execute("\n".join(full_statement))
-        for statement in index_output:
-            curs.execute(statement)
-        transaction.commit_unless_managed(using=db)
+        try:
+            curs.execute("\n".join(full_statement))
+        except DatabaseError, e:
+            self.stderr.write(
+                self.style.ERROR("Cache database '%s' could not be created.\nThe error was: %s.\n" %
+                    (tablename, e)))
+            transaction.rollback_unless_managed(using=db)
+        else:
+            for statement in index_output:
+                curs.execute(statement)
+            transaction.commit_unless_managed(using=db)

tests/regressiontests/cache/tests.py

@@ -817 +817 @@
         self.cache = get_cache('db://%s?max_entries=30&cull_frequency=0' % self._table_name)
         self.perform_cull_test(50, 18)
 
+    def test_second_call_dont_crash(self):
+        from StringIO import StringIO
+        out = StringIO()
+        management.call_command('createcachetable', self._table_name, verbosity=0, interactive=False, stderr=out)
+        self.assertTrue("Cache database 'test cache table' could not be created" in out.getvalue())
+
 
 DBCacheWithTimeZoneTests = override_settings(USE_TZ=True)(DBCacheTests)