Merge branch 'master' of portal.softwarepublico.gov.br:softwarepublico/colab

Sergio Oliveira
2 parents 0d143d6a a3ff3d5d
Showing 18 changed files with 259 additions and 119 deletions Show diff stats
colab/accounts/forms.py
colab/accounts/models.py
colab/accounts/templates/accounts/user_detail.html
colab/accounts/utils/mailman.py
colab/accounts/views.py
colab/home/views.py
colab/plugins/gitlab/migrations/0005_auto_20150806_1230.py
colab/plugins/gitlab/models.py
colab/plugins/gitlab/tests/test_gitlab.py
colab/search/utils.py
colab/settings.py
colab/super_archives/fixtures/mailinglistdata.json
colab/super_archives/models.py
colab/super_archives/templates/superarchives/emails/email_verification.txt
colab/super_archives/tests/test_privatelist.py
colab/super_archives/utils/email.py
colab/super_archives/views.py
colab/utils/conf.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,
@@ -60,7 +60,7 @@ class User(AbstractUser):
         return mailman.user_lists(self)
     def update_subscription(self, email, lists):
-        mailman.update_subscription(email, lists)
+        return mailman.update_subscription(email, lists)
     def save(self, *args, **kwargs):
@@ -84,7 +84,6 @@
           {% endif %}
         {% endif %}
       </ul>
-
       {% if user_.mailinglists %}
         <b>{% trans 'Groups: ' %}</b>
         {% for list in user_.mailinglists %}
@@ -4,66 +4,99 @@ import requests
 import logging
 from django.conf import settings
+from django.contrib import messages
 TIMEOUT = 1
 LOGGER = logging.getLogger('colab.mailman')
-
-def get_url(listname=None):
+S = 'success'
+I = 'info'
+E = 'error'
+
+MAILMAN_MSGS = {
+    0: (S, '%s: Success!'),
+    1: (S, '%s: An email confirmation was sent to you, please check your inbox.'),
+    2: (I, '%s: Your subscription was sent successfully! Please wait for the list\'s admin approval.'),
+    3: (I, '%s: You are already a member of this list.'),
+    4: (E, '%s: You are banned from this list!'),
+    5: (E, '%s: You appear to have an invalid email address.'),
+    6: (E, '%s: Your email address is considered to be hostile.'),
+    7: (E, '%s: You are not a member of this list.'),
+    8: (E, 'Missing information: `email_from`, `subject` and `body` are mandatory.'),
+}
+
+
+def get_url(path, listname=None):
+    url = urlparse.urljoin(settings.MAILMAN_API_URL, path)
     if listname:
-        return urlparse.urljoin(settings.MAILMAN_API_URL, '/' + listname)
-
-    return settings.MAILMAN_API_URL
+        return urlparse.urljoin(url, listname)
+    return url
 def subscribe(listname, address):
-    url = get_url(listname)
+    url = get_url('subscribe/', listname=listname)
     try:
-        requests.put(url, timeout=TIMEOUT, data={'address': address})
+        result = requests.put(url, timeout=TIMEOUT, data={'address': address})
+        msg_type, message = MAILMAN_MSGS[result.json()]
+        return msg_type, message % listname
     except:
         LOGGER.exception('Unable to subscribe user')
-        return False
-    return True
+        return E, 'Error: Unable to subscribe user'
 def unsubscribe(listname, address):
-    url = get_url(listname)
+    url = get_url('subscribe/', listname)
     try:
-        requests.delete(url, timeout=TIMEOUT, data={'address': address})
+        result = requests.delete(url, timeout=TIMEOUT, data={'address': address})
+        msg_type, message = MAILMAN_MSGS[result.json()]
+        return msg_type, message % listname
     except:
         LOGGER.exception('Unable to unsubscribe user')
-        return False
-    return True
+        return E, 'Error: Unable to subscribe user'
 def update_subscription(address, lists):
-    current_lists = mailing_lists(address=address)
+    current_lists = mailing_lists(address=address, names_only=True)
+    info_messages = []
     for maillist in current_lists:
         if maillist not in lists:
-            unsubscribe(maillist, address)
+            info_messages.append(unsubscribe(maillist, address))
     for maillist in lists:
         if maillist not in current_lists:
-            subscribe(maillist, address)
+            info_messages.append(subscribe(maillist, address))
+
+    return info_messages
 def mailing_lists(**kwargs):
-    url = get_url()
+    url = get_url('lists/')
     try:
-        lists = requests.get(url, timeout=TIMEOUT, params=kwargs)
+        lists = requests.get(url, timeout=TIMEOUT, params=kwargs).json()
+        if not isinstance(lists, (list, tuple)):
+            raise
     except:
         LOGGER.exception('Unable to list mailing lists')
         return []
-    return lists.json()
+    if kwargs.get('names_only'):
+        names_only = []
+        for l in lists:
+            names_only.append(l['listname'])
+        return names_only
+    else:
+        return lists
 def is_private_list(name):
     try:
-        return dict(all_lists(private=True))[name]
+        privacy = {}
+        privacy.update({mlist.get('listname'): mlist.get('archive_private')
+                        for mlist in all_lists()})
+        return privacy[name]
     except KeyError:
         return []
@@ -76,22 +109,25 @@ 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)
 def get_list_description(listname, lists=None):
     if not lists:
-        lists = dict(all_lists(description=True))
-    elif not isinstance(lists, dict):
-        lists = dict(lists)
+        lists = all_lists()
+
+    desc = "".join(mlist.get('description') for mlist in lists
+                   if isinstance(mlist, dict) and
+                   mlist.get('listname') == listname)
-    return lists.get(listname)
+    return desc
 def list_users(listname):
-    url = get_url(listname)
+    url = get_url('members/', listname)
     params = {}
@@ -114,3 +150,10 @@ def get_user_mailinglists(user):
         lists_for_user.extend(mailing_lists(address=email))
     return lists_for_user
+
+
+def extract_listname_from_list(lists):
+    try:
+        return [mlist.get('listname') for mlist in lists]
+    except ValueError:
+        return []
@@ -103,7 +103,12 @@ def signup(request):
     user.is_active = False
     user.save()
-    EmailAddressValidation.create(user.email, user)
+    email = EmailAddressValidation.create(user.email, user)
+
+    location = reverse('archive_email_view',
+                       kwargs={'key': email.validation_key})
+    verification_url = request.build_absolute_uri(location)
+    EmailAddressValidation.verify_email(email, verification_url)
     # Check if the user's email have been used previously
     #   in the mainling lists to link the user to old messages
@@ -139,7 +144,11 @@ class ManageUserSubscriptionsView(UserProfileBaseMixin, DetailView):
         user = self.get_object()
         for email in user.emails.values_list('address', flat=True):
             lists = self.request.POST.getlist(email)
-            user.update_subscription(email, lists)
+            info_messages = user.update_subscription(email, lists)
+            for msg_type, message in info_messages:
+                show_message = getattr(messages, msg_type)
+                show_message(request, _(message))
+
         return redirect('user_profile', username=user.username)
@@ -149,18 +158,19 @@ class ManageUserSubscriptionsView(UserProfileBaseMixin, DetailView):
         user = self.get_object()
         emails = user.emails.values_list('address', flat=True)
-        all_lists = mailman.all_lists(description=True)
+        all_lists = mailman.all_lists()
         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:
+            lists_for_address = mailman.mailing_lists(address=email, names_only=True)
+            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
                 ))
@@ -10,9 +10,10 @@ from colab.accounts.models import User
 def get_user_threads(threads, lists_for_user, key):
     visible_threads = []
+    listnames_for_user = mailman.extract_listname_from_list(lists_for_user)
     for t in threads:
         if not t.mailinglist.is_private or \
-           t.mailinglist.name in lists_for_user:
+           t.mailinglist.name in listnames_for_user:
                 visible_threads.append(key(t))
     return visible_threads
@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+from __future__ import unicode_literals
+
+from django.db import models, migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('gitlab', '0004_auto_20150630_1149'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='gitlabcomment',
+            name='iid',
+        ),
+        migrations.AddField(
+            model_name='gitlabissue',
+            name='iid',
+            field=models.IntegerField(null=True),
+            preserve_default=True,
+        ),
+    ]
@@ -64,6 +64,7 @@ class GitlabMergeRequest(Collaboration):
 class GitlabIssue(Collaboration):
     id = models.IntegerField(primary_key=True)
+    iid = models.IntegerField(null=True)
     project = models.ForeignKey(GitlabProject, null=True,
                                 on_delete=models.SET_NULL)
     title = models.TextField()
@@ -82,7 +83,7 @@ class GitlabIssue(Collaboration):
     @property
     def url(self):
         return u'/gitlab/{}/issues/{}'.format(
-            self.project.path_with_namespace, self.id)
+            self.project.path_with_namespace, self.iid)
     class Meta:
         verbose_name = _('Gitlab Issue')
@@ -92,7 +93,6 @@ class GitlabIssue(Collaboration):
 class GitlabComment(Collaboration):
     id = models.IntegerField(primary_key=True)
-    iid = models.IntegerField(null=True)
     body = models.TextField()
     created_at = models.DateTimeField(blank=True, null=True)
     issue_comment = models.BooleanField(default=True)
@@ -139,7 +139,7 @@ class GitlabComment(Collaboration):
             url_str = u'/gitlab/{}/merge_requests/{}#notes_{}'
         return url_str.format(self.project.path_with_namespace,
-                              self.parent_id, self.iid)
+                              self.parent_id, self.id)
     class Meta:
         verbose_name = _('Gitlab Comments')
@@ -24,9 +24,9 @@ class GitlabTest(TestCase):
         pass
     def test_data_integrity(self):
-        self.assertEqual(GitlabProject.objects.all().count(), 1)
-        self.assertEqual(GitlabMergeRequest.objects.all().count(), 1)
-        self.assertEqual(GitlabIssue.objects.all().count(), 1)
+        self.assertEqual(GitlabProject.objects.all().count(), 2)
+        self.assertEqual(GitlabMergeRequest.objects.all().count(), 2)
+        self.assertEqual(GitlabIssue.objects.all().count(), 2)
         self.assertEqual(GitlabComment.objects.all().count(), 2)
     def test_project_url(self):
@@ -34,55 +34,92 @@ class GitlabTest(TestCase):
                          '/gitlab/softwarepublico/colab')
     def test_merge_request_url(self):
-        self.assertEqual(GitlabMergeRequest.objects.get(iid=1).url,
+        self.assertEqual(GitlabMergeRequest.objects.get(id=1).url,
                          '/gitlab/softwarepublico/colab/merge_requests/1')
+        self.assertEqual(GitlabMergeRequest.objects.get(id=2).url,
+                         '/gitlab/softwarepublico/colabinc/merge_requests/1')
     def test_issue_url(self):
         self.assertEqual(GitlabIssue.objects.get(id=1).url,
                          '/gitlab/softwarepublico/colab/issues/1')
+        self.assertEqual(GitlabIssue.objects.get(id=2).url,
+                         '/gitlab/softwarepublico/colabinc/issues/1')
     def test_comment_on_mr_url(self):
         url = '/gitlab/softwarepublico/colab/merge_requests/1#notes_1'
-        self.assertEqual(GitlabComment.objects.get(iid=1).url, url)
+        self.assertEqual(GitlabComment.objects.get(id=1).url, url)
     def test_comment_on_issue_url(self):
         self.assertEqual(GitlabComment.objects.get(id=2).url,
                          '/gitlab/softwarepublico/colab/issues/1#notes_2')
     def create_gitlab_data(self):
-        g = GitlabProject()
-        g.id = 1
-        g.name = "colab"
-        g.name_with_namespace = "Software Public / Colab"
-        g.path_with_namespace = "softwarepublico/colab"
-        g.created_at = datetime.now()
-        g.last_activity_at = datetime.now()
-        g.save()
-
-        mr = GitlabMergeRequest()
-        mr.iid = 1
-        mr.project = g
-        mr.title = "Include plugin support"
-        mr.description = "Merge request for plugin support"
-        mr.state = "Closed"
-        mr.created_at = datetime.now()
-        mr.update_user(self.user.username)
-        mr.save()
-
-        i = GitlabIssue()
-        i.id = 1
-        i.project = g
-        i.title = "Issue for colab"
-        i.description = "Issue reported to colab"
-        i.created_at = datetime.now()
-        i.state = "Open"
-        i.update_user(self.user.username)
-        i.save()
+        g1 = GitlabProject()
+        g1.id = 1
+        g1.name = "colab"
+        g1.name_with_namespace = "Software Public / Colab"
+        g1.path_with_namespace = "softwarepublico/colab"
+        g1.created_at = datetime.now()
+        g1.last_activity_at = datetime.now()
+        g1.save()
+
+        g2 = GitlabProject()
+        g2.id = 2
+        g2.name = "colabinc"
+        g2.name_with_namespace = "Software Public / ColabInc"
+        g2.path_with_namespace = "softwarepublico/colabinc"
+        g2.created_at = datetime.now()
+        g2.last_activity_at = datetime.now()
+        g2.save()
+
+        mr1 = GitlabMergeRequest()
+        mr1.id = 1
+        mr1.iid = 1
+        mr1.project = g1
+        mr1.title = "Include plugin support"
+        mr1.description = "Merge request for plugin support"
+        mr1.state = "Closed"
+        mr1.created_at = datetime.now()
+        mr1.update_user(self.user.username)
+        mr1.save()
+
+        mr2 = GitlabMergeRequest()
+        mr2.id = 2
+        mr2.iid = 1
+        mr2.project = g2
+        mr2.title = "Include test support"
+        mr2.description = "Merge request for test support"
+        mr2.state = "Closed"
+        mr2.created_at = datetime.now()
+        mr2.update_user(self.user.username)
+        mr2.save()
+        
+        i1 = GitlabIssue()
+        i1.id = 1
+        i1.iid = 1
+        i1.project = g1
+        i1.title = "Issue for colab"
+        i1.description = "Issue reported to colab"
+        i1.created_at = datetime.now()
+        i1.state = "Open"
+        i1.update_user(self.user.username)
+        i1.save()
+
+        i2 = GitlabIssue()
+        i2.id = 2
+        i2.iid = 1
+        i2.project = g2
+        i2.title = "Issue for colab"
+        i2.description = "Issue reported to colab"
+        i2.created_at = datetime.now()
+        i2.state = "Open"
+        i2.update_user(self.user.username)
+        i2.save()
         c1 = GitlabComment()
-        c1.iid = 1
-        c1.parent_id = mr.iid
-        c1.project = g
+        c1.id = 1
+        c1.parent_id = mr1.iid
+        c1.project = g1
         c1.body = "Comment to merge request"
         c1.created_at = datetime.now()
         c1.issue_comment = False
@@ -90,9 +127,9 @@ class GitlabTest(TestCase):
         c1.save()
         c2 = GitlabComment()
-        c2.iid = 2
-        c2.parent_id = i.id
-        c2.project = g
+        c2.id = 2
+        c2.parent_id = i1.id
+        c2.project = g1
         c2.body = "Comment to issue"
         c2.created_at = datetime.now()
         c2.issue_comment = True
@@ -15,11 +15,12 @@ from colab.accounts.utils import mailman
 def get_visible_threads_queryset(logged_user):
     queryset = Thread.objects
-    lists_for_user = []
+    listnames_for_user = []
     if logged_user:
         lists_for_user = mailman.get_user_mailinglists(logged_user)
+        listnames_for_user = mailman.extract_listname_from_list(lists_for_user)
-    user_lists = Condition(mailinglist__name__in=lists_for_user)
+    user_lists = Condition(mailinglist__name__in=listnames_for_user)
     public_lists = Condition(mailinglist__is_private=False)
     queryset = Thread.objects.filter(user_lists | public_lists)
@@ -28,6 +29,7 @@ def get_visible_threads_queryset(logged_user):
 def get_visible_threads(logged_user, filter_by_user=None):
     thread_qs = get_visible_threads_queryset(logged_user)
+
     if filter_by_user:
         message_qs = Message.objects.filter(thread__in=thread_qs)
         messages = message_qs.filter(
@@ -57,10 +59,8 @@ def get_collaboration_data(logged_user, filter_by_user=None):
     latest_results.extend(messages)
-    app_names = settings.COLAB_APPS.keys()
-
-    for app_name in app_names:
-        module = importlib.import_module('{}.models'.format(app_name))
+    for app in settings.COLAB_APPS.values():
+        module = importlib.import_module('{}.models'.format(app.get('name')))
         for module_item_name in dir(module):
             module_item = getattr(module, module_item_name)
@@ -239,7 +239,7 @@ SUPER_ARCHIVES_EXCLUDE = []
 SUPER_ARCHIVES_LOCK_FILE = '/var/lock/colab/import_emails.lock'
 # Mailman API settings
-MAILMAN_API_URL = 'http://localhost:8124'
+MAILMAN_API_URL = 'http://localhost:8124/v2/'
 LOGIN_URL = '/user/login'
 LOGIN_REDIRECT_URL = '/'
@@ -266,8 +266,8 @@ for app_name, app in COLAB_APPS.items():
             if dep not in INSTALLED_APPS:
                 INSTALLED_APPS += (dep,)
-    if app_name not in INSTALLED_APPS:
-        INSTALLED_APPS += (app_name,)
+    if app.get('name') not in INSTALLED_APPS:
+        INSTALLED_APPS += (app.get('name'),)
     if 'middlewares' in app:
         for middleware in app.get('middlewares'):
@@ -6,23 +6,23 @@
       "twitter": null,
       "is_staff": false,
       "user_permissions": [
-        
+
       ],
       "date_joined": "2015-02-24T21:10:35.004Z",
       "google_talk": null,
-      "first_name": "Gust",
+      "first_name": "John",
       "is_superuser": false,
       "last_login": "2015-02-26T17:56:13.378Z",
       "verification_hash": null,
       "role": null,
-      "email": "gustmax@hotmail.com",
-      "username": "gustmax",
+      "email": "johndoe@example.com",
+      "username": "johndoe",
       "bio": null,
       "needs_update": false,
       "is_active": true,
       "facebook": null,
       "groups": [
-        
+
       ],
       "password": "pbkdf2_sha256$12000$ez83ccNOUQZk$vYT/QcYMukXZ7D7L1qQPyYlzCUEEEF20J7/Xjef0Rqg=",
       "institution": null,
@@ -37,7 +37,7 @@
       "real_name": "",
       "user": 1,
       "md5": "ed8f47ae6048f8d4456c0554578f53ff",
-      "address": "gustmax@hotmail.com"
+      "address": "johndoe@example.com"
     },
     "model": "super_archives.emailaddress",
     "pk": 1
@@ -41,11 +41,17 @@ class EmailAddressValidation(models.Model):
     def create(cls, address, user):
         email_address_validation = cls.objects.create(address=address,
                                                       user=user)
-        email.send_verification_email(email_address_validation.address,
-                                      email_address_validation.user,
-                                      email_address_validation.validation_key)
         return email_address_validation
+    @classmethod
+    def verify_email(cls, email_address_validation, verification_url):
+        return email.send_verification_email(
+            email_address_validation.address,
+            email_address_validation.user,
+            email_address_validation.validation_key,
+            verification_url
+            )
+
 class EmailAddress(models.Model):
     user = models.ForeignKey(settings.AUTH_USER_MODEL, null=True,
 {% load i18n %}
 {% blocktrans with fullname=user.get_full_name|title username=user.username|lower %}Hey, we want to verify that you are indeed "{{ fullname }} ({{ username }})".  If that's the case, please follow the link below:{% endblocktrans %}
-{{ SITE_URL }}{% url 'archive_email_view' key %}
+{{ verification_url }}
 {% blocktrans with username=user.username %}If you're not {{ username }} or didn't request verification you can ignore this email.{% endblocktrans %}
@@ -2,7 +2,7 @@
 import mock
 from colab.accounts.utils import mailman
-from django.test import TestCase,  Client
+from django.test import TestCase, Client
 class ArchivesViewTest(TestCase):
@@ -13,11 +13,14 @@ class ArchivesViewTest(TestCase):
         self.client = Client()
     def authenticate_user(self):
-        self.client.login(username='gustmax', password='1234')
+        self.client.login(username='johndoe', password='1234')
     def test_see_only_private_list_if_member(self):
         mailman.get_user_mailinglists = mock.Mock(
+            return_value="[{'listname': 'privatelist'}]")
+        mailman.extract_listname_from_list = mock.Mock(
             return_value="['privatelist']")
+        mailman.list_users = mock.Mock(return_value="['johndoe@example.com']")
         self.authenticate_user()
         request = self.client.get('/archives/thread/')
@@ -26,7 +29,7 @@ class ArchivesViewTest(TestCase):
         self.assertEqual('lista', list_data[0][0])
         self.assertEqual('privatelist', list_data[1][0])
-        self.assertEqual(2,  len(list_data))
+        self.assertEqual(2, len(list_data))
     def test_see_only_public_if_not_logged_in(self):
         request = self.client.get('/archives/thread/')
@@ -34,10 +37,12 @@ class ArchivesViewTest(TestCase):
         list_data = request.context['lists']
         self.assertEqual('lista', list_data[0][0])
-        self.assertEqual(1,  len(list_data))
+        self.assertEqual(1, len(list_data))
     def test_see_private_thread_in_dashboard_if_member(self):
         mailman.get_user_mailinglists = mock.Mock(
+            return_value="[{'listname': 'privatelist'}]")
+        mailman.extract_listname_from_list = mock.Mock(
             return_value="['privatelist']")
         self.authenticate_user()
@@ -59,7 +64,7 @@ class ArchivesViewTest(TestCase):
         self.assertEqual(1, len(hottest_threads))
     def test_dont_see_private_threads_in_profile_if_logged_out(self):
-        request = self.client.get('/account/gustmax')
+        request = self.client.get('/account/johndoe')
         emails = request.context['emails']
@@ -10,11 +10,11 @@ def colab_send_email(subject, message, to):
     return mail.send_mail(subject, message, from_email, [to])
-def send_verification_email(to, user, validation_key):
+def send_verification_email(to, user, validation_key, verification_url):
     subject = _('Please verify your email ') + u'{}'.format(to)
     msg_tmpl = \
         loader.get_template('superarchives/emails/email_verification.txt')
     message = msg_tmpl.render(Context({'to': to, 'user': user,
-                                       'key': validation_key,
-                                       'SITE_URL': settings.SITE_URL}))
+                                       'verification_url': verification_url
+                                       }))
     return colab_send_email(subject, message, to)
@@ -9,6 +9,7 @@ import requests
 from django import http
 from django.conf import settings
 from django.contrib import messages
+from django.core.urlresolvers import reverse
 from django.db import IntegrityError
 from django.views.generic import View
 from django.utils.translation import ugettext as _
@@ -32,17 +33,23 @@ class ThreadView(View):
         thread = get_object_or_404(Thread, subject_token=thread_token,
                                    mailinglist__name=mailinglist)
-        # TODO: Refactor this code
-        # Use local flag is_private instead of always check the API!!
-        all_privates = dict(mailman.all_lists(private=True))
-        if all_privates[thread.mailinglist.name]:
+        all_privates = []
+        all_privates.extend(
+            [mlist.get('listname')
+                for mlist in mailman.all_lists()
+                if mlist.get('archive_private')]
+        )
+
+        if all_privates.count(thread.mailinglist.name):
             if not request.user.is_authenticated():
                 raise PermissionDenied
             else:
                 user = User.objects.get(username=request.user)
                 emails = user.emails.values_list('address', flat=True)
                 lists_for_user = mailman.get_user_mailinglists(user)
-                if thread.mailinglist.name not in lists_for_user:
+                listnames_for_user = mailman.extract_listname_from_list(
+                    lists_for_user)
+                if thread.mailinglist.name not in listnames_for_user:
                     raise PermissionDenied
         thread.hit(request)
@@ -144,13 +151,16 @@ class ThreadDashboardView(View):
         context['lists'] = []
-        lists_for_user = []
+        listnames_for_user = []
         if request.user.is_authenticated():
             user = User.objects.get(username=request.user)
             lists_for_user = mailman.get_user_mailinglists(user)
+            listnames_for_user = mailman.extract_listname_from_list(
+                lists_for_user)
         for list_ in MailingList.objects.order_by('name'):
-            if list_.name not in all_privates or list_.name in lists_for_user:
+            if list_.name not in all_privates\
+                    or list_.name in listnames_for_user:
                 context['lists'].append((
                     list_.name,
                     mailman.get_list_description(list_.name),
@@ -177,7 +187,7 @@ class EmailView(View):
         except EmailAddressValidation.DoesNotExist:
             messages.error(request, _('The email address you are trying to '
                                       'verify either has already been verified'
-                                      'or does not exist.'))
+                                      ' or does not exist.'))
             return redirect('/')
         try:
@@ -289,8 +299,11 @@ class EmailValidationView(View):
             raise http.Http404
         try:
-            send_verification_email(email_addr, email.user,
-                                    email.validation_key)
+            location = reverse('archive_email_view',
+                               kwargs={'key': email.validation_key})
+            verification_url = request.build_absolute_uri(location)
+            send_verification_email(request, email_addr, email.user,
+                                    email.validation_key, verification_url)
         except smtplib.SMTPException:
             logging.exception('Error sending validation email')
             return http.HttpResponseServerError()
@@ -95,6 +95,7 @@ def load_colab_apps():
         return {'COLAB_APPS': COLAB_APPS}
     for file_name in os.listdir(plugins_dir):
+        app_name = ""
         file_module = file_name.split('.')[0]
         logger.info('Loaded plugin settings: %s%s', plugins_dir, file_name)
@@ -104,7 +105,7 @@ def load_colab_apps():
             app_name = file_name
         elif file_name.endswith('.py'):
-            app_name = py_settings_d.get('name')
+            app_name = py_settings_d.get('name').split('.')[-1]
         if not app_name:
             logger.warning("Plugin missing name variable (%s)", file_name)
@@ -121,7 +122,7 @@ def load_colab_apps():
         fields = ['verbose_name', 'upstream', 'urls',
                   'menu_urls', 'middlewares', 'dependencies',
-                  'context_processors', 'private_token']
+                  'context_processors', 'private_token', 'name']
         for key in fields:
             value = py_settings_d.get(key)
	@@ -60,7 +60,7 @@ class User(AbstractUser):		@@ -60,7 +60,7 @@ class User(AbstractUser):
60	return mailman.user_lists(self)	60	return mailman.user_lists(self)
61		61
62	def update_subscription(self, email, lists):	62	def update_subscription(self, email, lists):
63	- mailman.update_subscription(email, lists)	63	+ return mailman.update_subscription(email, lists)
64		64
65	def save(self, args, *kwargs):	65	def save(self, args, *kwargs):
66		66
@@ -0,0 +1,24 @@		@@ -0,0 +1,24 @@
	1	+# -- coding: utf-8 --
	2	+from __future__ import unicode_literals
	3	+
	4	+from django.db import models, migrations
	5	+
	6	+
	7	+class Migration(migrations.Migration):
	8	+
	9	+ dependencies = [
	10	+ ('gitlab', '0004_auto_20150630_1149'),
	11	+ ]
	12	+
	13	+ operations = [
	14	+ migrations.RemoveField(
	15	+ model_name='gitlabcomment',
	16	+ name='iid',
	17	+ ),
	18	+ migrations.AddField(
	19	+ model_name='gitlabissue',
	20	+ name='iid',
	21	+ field=models.IntegerField(null=True),
	22	+ preserve_default=True,
	23	+ ),
	24	+ ]
1	{% load i18n %}	1	{% load i18n %}
2	{% blocktrans with fullname=user.get_full_name\|title username=user.username\|lower %}Hey, we want to verify that you are indeed "{{ fullname }} ({{ username }})". If that's the case, please follow the link below:{% endblocktrans %}	2	{% blocktrans with fullname=user.get_full_name\|title username=user.username\|lower %}Hey, we want to verify that you are indeed "{{ fullname }} ({{ username }})". If that's the case, please follow the link below:{% endblocktrans %}
3		3
4	-{{ SITE_URL }}{% url 'archive_email_view' key %}	4	+{{ verification_url }}
5		5
6	{% blocktrans with username=user.username %}If you're not {{ username }} or didn't request verification you can ignore this email.{% endblocktrans %}	6	{% blocktrans with username=user.username %}If you're not {{ username }} or didn't request verification you can ignore this email.{% endblocktrans %}