adding cumulative filters to thread lists #24

Luan
1 parent 2d13baf3
Showing 3 changed files with 15 additions and 10 deletions Show diff stats
src/super_archives/templates/message-list.html
src/super_archives/utils/url.py
src/super_archives/views.py
@@ -27,9 +27,12 @@
       <h4>{% trans "Lists" %}</h4>
       <ul class="unstyled-list">
         {% for list in lists %}
-          <li {% if list.name == selected_list %} title="{% trans "Remove filter" %}" class="selected" {% endif %}>
-            <span class="glyphicon {% if list.name == selected_list %}glyphicon-remove{% else %}glyphicon-chevron-right{% endif %}"></span> <a href="{% ifnotequal list.name selected_list %} {% append_to_get list=list.name p=1 %} {% else %} {% append_to_get list="" p=1 %}
-            {% endifnotequal %}">{{ list.name }}</a></li>
+          {% with list.name|add:" "|add:selected_lists as list_name %}
+            <li {% if list.name in selected_lists %} title="{% trans "Remove filter" %}" class="selected" {% endif %}>
+              <span class="glyphicon {% if list.name in selected_lists %}glyphicon-remove{% else %}glyphicon-chevron-right{% endif %}"></span>
+              <a href="{% if not list.name in selected_lists %}{% append_to_get list=list_name p=1 %}{% else %}{% pop_from_get list=list.name %}{% endif %}">{{ list.name }}</a>
+            </li>
+          {% endwith %}
         {% endfor %}
       </ul>
     </div>
@@ -19,6 +19,7 @@ def pop_from_get(path, query=None, **kwargs):
         if query_dict[key] == value:
             del query_dict[key]
             continue
-    if not query_dict:
-        return u'{}?q='.format(path)
+        if value in query_dict[key]:
+            aux = query_dict[key].split(value)
+            query_dict[key] = u''.join(aux).strip()
     return u'{}?{}'.format(path, urllib.urlencode(query_dict))
@@ -60,8 +60,9 @@ def thread(request, mailinglist, thread_token):
 def list_messages(request):
-
-    selected_list = request.GET.get('list')
+    selected_lists = request.GET.get('list', [])
+    if selected_lists:
+        selected_lists = selected_lists.split()
     order_by = request.GET.get('order')
     if order_by == 'hottest':
@@ -69,9 +70,9 @@ def list_messages(request):
     else:
         threads = queries.get_latest_threads()
-    mail_list = request.GET.get('list')
+    mail_list = selected_lists
     if mail_list:
-        threads = threads.filter(mailinglist__name=mail_list)
+        threads = threads.filter(mailinglist__name__in=mail_list)
     paginator = Paginator(threads, 16)
     try:
@@ -86,7 +87,7 @@ def list_messages(request):
         'lists': lists,
         'n_results': paginator.count,
         'threads': threads,
-        'selected_list': selected_list,
+        'selected_lists': ' '.join(selected_lists) if selected_lists else '',
         'order_data': settings.ORDERING_DATA,
     }
     return render(request, 'message-list.html', template_data)