Updated mailing_lists to mailman-api v2

Signed-off-by: Carlos Oliveira <carlospecter@gmail.com>

Updated mailing_lists to mailman-api v2
Signed-off-by: Carlos Oliveira <carlospecter@gmail.com>
Carlos Coêlho
1 parent 8d1f236c
Showing 4 changed files with 11 additions and 8 deletions Show diff stats
colab/accounts/forms.py
colab/accounts/templates/accounts/user_detail.html
colab/accounts/utils/mailman.py
colab/accounts/views.py
@@ -141,8 +141,9 @@ class UserUpdateForm(UserForm):
 class ListsForm(forms.Form):
     LISTS_NAMES = ((
-        listname, u'{} ({})'.format(listname, description)
-    ) for listname, description in mailman.all_lists(description=True))
+        mlist.get('listname'), u'{} ({})'.format(mlist.get('listname'),
+                                                 mlist.get('description'))
+    ) for mlist in mailman.all_lists())
     lists = forms.MultipleChoiceField(label=_(u'Mailing lists'),
                                       required=False,
@@ -84,7 +84,6 @@
           {% endif %}
         {% endif %}
       </ul>
-
       {% if user_.mailinglists %}
         <b>{% trans 'Groups: ' %}</b>
         {% for list in user_.mailinglists %}
@@ -73,9 +73,10 @@ def update_subscription(address, lists):
 def mailing_lists(**kwargs):
     url = get_url()
+    path = 'lists/'
     try:
-        lists = requests.get(url, timeout=TIMEOUT, params=kwargs)
+        lists = requests.get(url + path, timeout=TIMEOUT, params=kwargs)
     except:
         LOGGER.exception('Unable to list mailing lists')
         return []
@@ -98,7 +99,8 @@ def user_lists(user):
     list_set = set()
     for email in user.emails.values_list('address', flat=True):
-        list_set.update(mailing_lists(address=email))
+        mlists = mailing_lists(address=email)
+        list_set.update(mlist.get('listname') for mlist in mlists)
     return tuple(list_set)
@@ -158,13 +158,14 @@ class ManageUserSubscriptionsView(UserProfileBaseMixin, DetailView):
         for email in emails:
             lists = []
             lists_for_address = mailman.mailing_lists(address=email)
-            for listname, description in all_lists:
-                if listname in lists_for_address:
+            for mlist in all_lists:
+                if mlist.get('listname') in lists_for_address:
                     checked = True
                 else:
                     checked = False
                 lists.append((
-                    {'listname': listname, 'description': description},
+                    {'listname': mlist.get('listname'),
+                      'description': mlist.get('description')},
                     checked
                 ))