Merge pull request #161 from colab/fix_default_ordering

Made latest default ordering for list view

colab/super_archives/tests/test_mailinglist_view.py

@@ -17,16 +17,6 @@ def setUp(self):
     def authenticate_user(self):
         self.client.login(username=self.username, password='admin')
 
-    def test_get_query_set_with_no_order(self):
-        response = self.client.get('/archives/mailinglist/lista')
-
-        self.assertEqual(response.status_code, 200)
-        self.assertEqual(len(response.context['thread_list']), 1)
-        self.assertEqual(response.context['thread_list'][0].mailinglist.name,
-                         'lista')
-        self.assertEqual(response.context['thread_list'][0].subject_token,
-                         'Subject2')
-
     def test_get_query_set_with_latest_order(self):
         response = self.client.get(
             '/archives/mailinglist/publiclist?order=latest'

colab/super_archives/views.py

@@ -369,13 +369,10 @@ def get_queryset(self):
 
         query = Q(mailinglist__name__iexact=mailinglist_name)
 
-        order = self.request.GET.get('order')
+        order = self.request.GET.get('order', 'latest')
         order = self.order_data.get(order)
 
-        if order:
-            result = Thread.objects.filter(query).order_by(order['field'])
-        else:
-            result = Thread.objects.filter(query)
+        result = Thread.objects.filter(query).order_by(order['field'])
 
         return result
 
@@ -385,7 +382,7 @@ def get_context_data(self, **kwargs):
 
         context['mailinglist'] = mailinglist
         context['order_data'] = self.order_data
-        context['selected'] = self.request.GET.get('order')
+        context['selected'] = self.request.GET.get('order', 'latest')
 
         return context