Corrected FilterView behaviour with unbound FilterSet. (carltongibson…

…#1025)

Closes carltongibson#930, carltongibson#987, carltongibson#1007.

django_filters/views.py

@@ -77,7 +77,7 @@ def get(self, request, *args, **kwargs):
         filterset_class = self.get_filterset_class()
         self.filterset = self.get_filterset(filterset_class)
 
-        if self.filterset.is_valid() or not self.get_strict():
+        if not self.filterset.is_bound or self.filterset.is_valid() or not self.get_strict():
             self.object_list = self.filterset.qs
         else:
             self.object_list = self.filterset.queryset.none()

tests/templates/tests/book_filter.html

@@ -1,5 +1,5 @@
 {{ filter.form }}
 
-{% for obj in filter.qs %}
+{% for obj in object_list %}
     {{ obj }}
 {% endfor %}

tests/test_views.py

@@ -123,6 +123,19 @@ class View(FilterView):
         self.assertEqual(message, expected)
         self.assertEqual(len(recorded), 0)
 
+    def test_view_with_unbound_filter_form_returns_initial_queryset(self):
+        factory = RequestFactory()
+        request = factory.get(self.base_url)
+
+        queryset = Book.objects.filter(title='Snowcrash')
+        view = FilterView.as_view(model=Book, queryset=queryset)
+
+        response = view(request)
+        titles = [o.title for o in response.context_data['object_list']]
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(titles, ['Snowcrash'])
+
 
 class GenericFunctionalViewTests(GenericViewTestCase):
     base_url = '/books-legacy/'