Django 1.8 compatible ModelAdmin.get_urls

This replaces code that was removed in bb3a835. Closes #26.
RealGeeks · Nov 17, 2016 · ace1ba9 · ace1ba9
1 parent bdd8014
commit ace1ba9
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 6 deletions.
diff --git a/modelclone/admin.py b/modelclone/admin.py
@@ -46,11 +46,19 @@ def get_urls(self):
             self.model._meta.app_label,
             getattr(self.model._meta, 'module_name', getattr(self.model._meta, 'model_name', '')))
 
-        new_urlpatterns = [
-            url(r'^(.+)/change/clone/$',
-                self.admin_site.admin_view(self.clone_view),
-                name=url_name)
-        ]
+        if VERSION[1] < 9:
+            from django.conf.urls import patterns
+            new_urlpatterns = patterns('',
+                url(r'^(.+)/clone/$',
+                    self.admin_site.admin_view(self.clone_view),
+                    name=url_name)
+                )
+        else:
+            new_urlpatterns = [
+                url(r'^(.+)/change/clone/$',
+                    self.admin_site.admin_view(self.clone_view),
+                    name=url_name)
+            ]
 
         original_urlpatterns = super(ClonableModelAdmin, self).get_urls()
 

diff --git a/tests/test_admin.py b/tests/test_admin.py
@@ -85,7 +85,10 @@ def test_clone_view_is_wrapped_as_admin_view(self):
 
     def test_clone_view_url_name(self):
         post_id = self.post.id
-        expected_url = '/admin/posts/post/{0}/change/clone/'.format(post_id)
+        if django.VERSION[1] < 9:
+            expected_url = '/admin/posts/post/{0}/clone/'.format(post_id)
+        else:
+            expected_url = '/admin/posts/post/{0}/change/clone/'.format(post_id)
 
         assert reverse('admin:posts_post_clone', args=(post_id,)) == expected_url