Changed SearchView to allow more control over how many results are shown per page. Thanks to simonw for the suggestion.

Author: toastdriven

docs/views_and_forms.rst

@@ -129,8 +129,8 @@ The functional view provides an example of how Haystack can be used in more
 traditional settings or as an example of how to write a more complex custom
 view. It is also thread-safe.
 
-``SearchView(template=None, load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext)``
----------------------------------------------------------------------------------------------------------------------------
+``SearchView(template=None, load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext, results_per_page=None)``
+--------------------------------------------------------------------------------------------------------------------------------------------------
 
 The ``SearchView`` is designed to be easy/flexible enough to override common
 changes as well as being internally abstracted so that only altering a specific
@@ -241,8 +241,8 @@ creates the context and renders the response for all the aforementioned
 processing.
 
 
-``basic_search(request, template='search/search.html', load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext, extra_context=None)``
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
+``basic_search(request, template='search/search.html', load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext, extra_context=None, results_per_page=None)``
+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 
 The ``basic_search`` tries to provide most of the same functionality as the
 class-based views but resembles a more traditional generic view. It's both a

haystack/views.py

@@ -17,13 +17,17 @@ class SearchView(object):
     results = EmptySearchQuerySet()
     request = None
     form = None
+    results_per_page = RESULTS_PER_PAGE
 
-    def __init__(self, template=None, load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext):
+    def __init__(self, template=None, load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext, results_per_page=None):
         self.load_all = load_all
         self.form_class = form_class
         self.context_class = context_class
         self.searchqueryset = searchqueryset
 
+        if not results_per_page is None:
+            self.results_per_page = results_per_page
+        
         if template:
             self.template = template
 
@@ -93,7 +97,7 @@ def build_page(self):
         should be a simple matter to override this method to do what they would
         like.
         """
-        paginator = Paginator(self.results, RESULTS_PER_PAGE)
+        paginator = Paginator(self.results, self.results_per_page)
 
         try:
             page = paginator.page(self.request.GET.get('page', 1))
@@ -142,7 +146,7 @@ def extra_context(self):
         return extra
 
 
-def basic_search(request, template='search/search.html', load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext, extra_context=None):
+def basic_search(request, template='search/search.html', load_all=True, form_class=ModelSearchForm, searchqueryset=None, context_class=RequestContext, extra_context=None, results_per_page=None):
     """
     A more traditional view that also demonstrate an alternative
     way to use Haystack.
@@ -175,7 +179,7 @@ def basic_search(request, template='search/search.html', load_all=True, form_cla
     else:
         form = form_class(searchqueryset=searchqueryset, load_all=load_all)
 
-    paginator = Paginator(results, RESULTS_PER_PAGE)
+    paginator = Paginator(results, results_per_page or RESULTS_PER_PAGE)
 
     try:
         page = paginator.page(int(request.GET.get('page', 1)))

tests/core/tests/results_per_page_urls.py

@@ -0,0 +1,12 @@
+from django.conf.urls.defaults import *
+from haystack.views import SearchView
+
+
+class CustomPerPage(SearchView):
+    results_per_page = 1
+
+
+urlpatterns = patterns('haystack.views',
+    url(r'^search/$', CustomPerPage(load_all=False), name='haystack_search'),
+    url(r'^search2/$', CustomPerPage(load_all=False, results_per_page=2), name='haystack_search'),
+)

tests/core/tests/views.py

@@ -100,6 +100,21 @@ def __call__(self, request):
         self.assertNotEqual(foo, bar)
 
 
+class ResultsPerPageTestCase(TestCase):
+    urls = 'core.tests.results_per_page_urls'
+    
+    def test_custom_results_per_page(self):
+        response = self.client.get('/search/', {'q': 'hello world'})
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(len(response.context[-1]['page'].object_list), 1)
+        self.assertEqual(response.context[-1]['paginator'].per_page, 1)
+        
+        response = self.client.get('/search2/', {'q': 'hello world'})
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(len(response.context[-1]['page'].object_list), 1)
+        self.assertEqual(response.context[-1]['paginator'].per_page, 2)
+
+
 class FacetedSearchViewTestCase(TestCase):
     def setUp(self):
         super(FacetedSearchViewTestCase, self).setUp()