Fixed bug in the queryset_factory function because it did not always return a queryset

Author: Ernesto Perez Amigo

graphene_django_extras/__init__.py

@@ -7,7 +7,7 @@
 from .pagination import LimitOffsetGraphqlPagination, PageGraphqlPagination, CursorGraphqlPagination
 from .types import DjangoObjectType, DjangoInputObjectType, DjangoListObjectType
 
-VERSION = (0, 0, 1, 'beta', 9)
+VERSION = (0, 0, 1, 'beta', 10)
 
 __version__ = get_version(VERSION)
 

graphene_django_extras/fields.py

@@ -87,7 +87,7 @@ def list_resolver1(manager, filterset_class, filtering_args, root, info, **kwarg
         qs = find_field(info.field_asts[0], root._prefetched_objects_cache)
 
         if not qs:
-            qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs).get_queryset()
+            qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs)
             qs = filterset_class(data=filter_kwargs, queryset=qs).qs
 
             if root and is_valid_django_model(root._meta.model):
@@ -117,7 +117,7 @@ def list_resolver(manager, filterset_class, filtering_args, root, info, **kwargs
                 qs = None
 
         if not qs:
-            qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs).get_queryset()
+            qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs)
             qs = filterset_class(data=filter_kwargs, queryset=qs).qs
 
             if root and is_valid_django_model(root._meta.model):
@@ -173,7 +173,7 @@ def model(self):
     def list_resolver(self, manager, filterset_class, filtering_args,
                       root, info, **kwargs):
 
-        qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs).get_queryset()
+        qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs)
         qs = filterset_class(data=filter_kwargs, queryset=qs).qs
 
         if root and is_valid_django_model(root._meta.model):
@@ -229,7 +229,7 @@ def model(self):
 
     def list_resolver(self, manager, filterset_class, filtering_args, root, info, **kwargs):
 
-        qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs).get_queryset()
+        qs = queryset_factory(manager, info.field_asts, info.fragments, **kwargs)
 
         filter_kwargs = {k: v for k, v in kwargs.items() if k in filtering_args}
 

graphene_django_extras/utils.py

@@ -168,4 +168,4 @@ def queryset_factory(manager, fields_asts=None, fragments=None, **kwargs):
         return manager.prefetch_related(*prefetch_related)
     elif select_related and not prefetch_related:
         return manager.select_related(*select_related)
-    return manager
+    return manager.get_queryset()