Merge pull request #613 from praekelt/feature/search-query-trimming

strip leading and trailing empty spaces in search query

molo/core/tests/test_search.py

@@ -106,6 +106,27 @@ def test_search(self):
         response = self.client.get(reverse('search'))
         self.assertContains(response, 'No search results for None')
 
+    def test_search_empty_values(self):
+        self.backend = get_search_backend('default')
+        self.backend.reset_index()
+
+        self.mk_article(
+            self.english_section, title="Site 1 article")
+        self.mk_article(
+            self.yourmind2, title="Site 2 article")
+        self.backend.refresh_index()
+
+        response = self.client.get(reverse('search'), {
+            'q': ' '
+        })
+        self.assertEqual(response.status_code, 200)
+
+        response = self.client.get(reverse('search'), {
+            'q': ' article '
+        })
+        self.assertContains(response, 'Site 1 article')
+        self.assertNotContains(response, 'Site 2 article')
+
     def test_search_works_with_multisite(self):
         self.backend = get_search_backend('default')
         self.backend.reset_index()

molo/core/views.py

@@ -50,6 +50,7 @@ def csrf_failure(request, reason=""):
 
 def search(request, results_per_page=10, load_more=False):
     search_query = request.GET.get('q', None)
+    search_query = search_query.strip() if search_query else search_query
     page = request.GET.get('p', 1)
     locale = get_locale_code(get_language_from_request(request))