Ticket #17415: 17415.diff

django/contrib/sites/management.py

@@ -3 +3 @@
 """
 
 from django.db.models import signals
+from django.db import connections
 from django.db import router
 from django.contrib.sites.models import Site
 from django.contrib.sites import models as site_app
+from django.core.management.color import no_style
 
 def create_default_site(app, created_models, verbosity, db, **kwargs):
     # Only create the default sites in databases where Django created the table
@@ -19 +21 @@
         # can also crop up outside of tests - see #15346.
         s = Site(pk=1, domain="example.com", name="example.com")
         s.save(using=db)
+        # Reset the sequence. Above the pk is set explicitly for autofield, so
+        # make sure the sequence doesn't collide with the above defined ID.
+        reset_sql = connections[db].ops.sequence_reset_sql(no_style(), [Site])
+        cursor = connections[db].cursor()
+        for command in reset_sql:
+            cursor.execute(command)
     Site.objects.clear_cache()
 
 signals.post_syncdb.connect(create_default_site, sender=site_app)

django/contrib/sites/tests.py

@@ -15 +15 @@
     def tearDown(self):
         Site._meta.installed = self.old_Site_meta_installed
 
+    def test_save_another(self):
+        """
+        On some backends the sequence needs reset after save with explicit ID.
+
+        Test that there is no sequence collisions by saving another site.
+        """
+        Site(domain="example2.com", name="example2.com").save()
+
     def test_site_manager(self):
         # Make sure that get_current() does not return a deleted Site object.
         s = Site.objects.get_current()