Merge pull request #9 from philb61/master

Don't override manager if it's in the arguments.
justinmayer · May 5, 2016 · a16fd5e · a16fd5e
2 parents fb62460 + f6d1a12
commit a16fd5e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/autoslug/fields.py b/autoslug/fields.py
@@ -244,6 +244,9 @@ def deconstruct(self):
         if self.always_update:
             kwargs['always_update'] = self.always_update
 
+        if 'manager' in kwargs:
+            del kwargs['manager']
+
         return name, path, args, kwargs
 
     def pre_save(self, instance, add):

diff --git a/autoslug/tests/tests.py b/autoslug/tests/tests.py
@@ -241,6 +241,12 @@ def test_autoslug_with_manager_name(self):
         b = NonDeletableModelWithUniqueSlug.objects.create(name='My name')
         self.assertEqual(b.slug, u'my-name-2')
 
+    def test_deconstruct_with_manager(self):
+        a = SharedSlugSpace(name='TestName')
+        a.save()
+        _, _, _, kwargs = a._meta.get_field('slug').deconstruct()
+        self.assertNotIn('manager', kwargs)
+
 
 class AutoSlugModelTranslationTestCase(TestCase):