Fix guardian import

rest_framework/compat.py

@@ -207,7 +207,6 @@ def value_from_object(field, obj):
 try:
     if 'guardian' in settings.INSTALLED_APPS:
         import guardian
-        import guardian.shortcuts  # Fixes #1624
 except ImportError:
     pass
 

rest_framework/filters.py

@@ -289,6 +289,11 @@ def __init__(self):
     perm_format = '%(app_label)s.view_%(model_name)s'
 
     def filter_queryset(self, request, queryset, view):
+        # We want to defer this import until run-time, rather than import-time.
+        # See https://github.com/tomchristie/django-rest-framework/issues/4608
+        # (Also see #1624 for why we need to make this import explicitly)
+        from guardian.shortcuts import get_objects_for_user
+
         extra = {}
         user = request.user
         model_cls = queryset.model
@@ -302,4 +307,4 @@ def filter_queryset(self, request, queryset, view):
             extra = {'accept_global_perms': False}
         else:
             extra = {}
-        return guardian.shortcuts.get_objects_for_user(user, permission, queryset, **extra)
+        return get_objects_for_user(user, permission, queryset, **extra)

rest_framework/schemas.py

@@ -1,4 +1,3 @@
-import os
 import re
 from collections import OrderedDict
 from importlib import import_module
@@ -37,6 +36,18 @@
 })
 
 
+def common_path(paths):
+    split_paths = [path.strip('/').split('/') for path in paths]
+    s1 = min(split_paths)
+    s2 = max(split_paths)
+    common = s1
+    for i, c in enumerate(s1):
+        if c != s2[i]:
+            common = s1[:i]
+            break
+    return '/' + '/'.join(common)
+
+
 def get_pk_name(model):
     meta = model._meta.concrete_model._meta
     return _get_pk(meta).name
@@ -292,7 +303,7 @@ def determine_path_prefix(self, paths):
                 # one URL that doesn't have a path prefix.
                 return '/'
             prefixes.append('/' + prefix + '/')
-        return os.path.commonprefix(prefixes)
+        return common_path(prefixes)
 
     def create_view(self, callback, method, request=None):
         """

tests/test_schemas.py

@@ -335,3 +335,14 @@ def test_schema_for_regular_views(self):
             }
         )
         self.assertEqual(schema, expected)
+
+
+@unittest.skipUnless(coreapi, 'coreapi is not installed')
+class Test4605Regression(TestCase):
+    def test_4605_regression(self):
+        generator = SchemaGenerator()
+        prefix = generator.determine_path_prefix([
+            '/api/v1/items/',
+            '/auth/convert-token/'
+        ])
+        assert prefix == '/'