diff --git a/src/postorius/context_processors.py b/src/postorius/context_processors.py
index 9dfff0f..934651f 100644
--- a/src/postorius/context_processors.py
+++ b/src/postorius/context_processors.py
@@ -49,6 +49,6 @@
'request': request,
'hyperkitty_url': hyperkitty_url,
# Resolve the login and logout URLs from the settings
- 'login_url': resolve_url(settings.LOGIN_URL),
+ 'login_url': resolve_url(settings.LOGIN_URL),
'logout_url': resolve_url(settings.LOGOUT_URL),
}
diff --git a/src/postorius/forms.py b/src/postorius/forms.py
index 14638f4..2a1af6e 100644
--- a/src/postorius/forms.py
+++ b/src/postorius/forms.py
@@ -192,15 +192,15 @@
"""
email = forms.ChoiceField(
- label=_('Your email address'),
- validators=[validate_email],
- widget=forms.Select(),
- error_messages={
- 'required': _('Please enter an email address.'),
- 'invalid': _('Please enter a valid email address.')})
+ label=_('Your email address'),
+ validators=[validate_email],
+ widget=forms.Select(),
+ error_messages={
+ 'required': _('Please enter an email address.'),
+ 'invalid': _('Please enter a valid email address.')})
- display_name = forms.CharField(label=_('Your name (optional)'),
- required=False)
+ display_name = forms.CharField(
+ label=_('Your name (optional)'), required=False)
def __init__(self, user_emails, *args, **kwargs):
super(ListSubscribe, self).__init__(*args, **kwargs)
@@ -826,14 +826,14 @@
# Check if the address belongs to someone else
if User.objects.filter(email=email).exists():
raise forms.ValidationError(
- _('This email is in use. Please choose another or contact'
- ' the administrator'), 'error')
+ _('This email is in use. Please choose another or contact'
+ ' the administrator'), 'error')
# Check if the email is attached to a user in Mailman
try:
utils.get_client().get_user(email)
raise forms.ValidationError(
- _('This email already belongs to a user'), 'error')
+ _('This email already belongs to a user'), 'error')
except HTTPError:
pass
return email
diff --git a/src/postorius/lib/scrub.py b/src/postorius/lib/scrub.py
index f0995df..86952a6 100644
--- a/src/postorius/lib/scrub.py
+++ b/src/postorius/lib/scrub.py
@@ -170,4 +170,4 @@
if ctype == 'message/rfc822':
submsg = part.get_payload()
decodedpayload = str(submsg)
- return (counter, filebase+ext, ctype, charset, decodedpayload)
+ return (counter, filebase + ext, ctype, charset, decodedpayload)
diff --git a/src/postorius/models.py b/src/postorius/models.py
index f57403c..854dcc8 100644
--- a/src/postorius/models.py
+++ b/src/postorius/models.py
@@ -276,8 +276,8 @@
# the setting or the default
if not template_path:
template_path = getattr(
- settings, 'EMAIL_CONFIRMATION_TEMPLATE',
- 'postorius/user/address_confirmation_message.txt')
+ settings, 'EMAIL_CONFIRMATION_TEMPLATE',
+ 'postorius/user/address_confirmation_message.txt')
# Create a template context (if there is none) containing
# the activation_link and the host_url.
if not template_context:
diff --git a/src/postorius/tests/mailman_api_tests/test_address_activation.py b/src/postorius/tests/mailman_api_tests/test_address_activation.py
index 2e4ddf0..90c620a 100644
--- a/src/postorius/tests/mailman_api_tests/test_address_activation.py
+++ b/src/postorius/tests/mailman_api_tests/test_address_activation.py
@@ -38,15 +38,15 @@
super(TestAddressActivationForm, self).setUp()
# Create a user and profile.
self.user = User.objects.create_user(
- 'testuser', 'les@example.org', 'testpass')
+ 'testuser', 'les@example.org', 'testpass')
self.profile = AddressConfirmationProfile.objects.create(
- email='les2@example.org', user=self.user)
+ email='les2@example.org', user=self.user)
self.expired = AddressConfirmationProfile.objects.create(
- email='expired@example.org', user=self.user)
+ email='expired@example.org', user=self.user)
self.expired.created -= timedelta(weeks=100)
self.expired.save()
self.mm_user = self.mm_client.create_user(
- 'subscribed@example.org', 'password')
+ 'subscribed@example.org', 'password')
def test_valid_email_is_valid(self):
form = AddressActivationForm({'email': 'very_new_email@example.org'})
@@ -82,7 +82,7 @@
username=u'ler_mm', email=u'ler@mailman.mostdesirable.org',
password=u'pwd')
self.profile = AddressConfirmationProfile.objects.create(
- email=u'les@example.org', user=self.user)
+ email=u'les@example.org', user=self.user)
# Create a test request object
self.request = RequestFactory().get('/')
@@ -156,7 +156,7 @@
password='pwd')
self.mm_user = self.mm_client.create_user('ler@example.org', None)
self.profile = AddressConfirmationProfile.objects.create(
- email=u'les@example.org', user=self.user)
+ email=u'les@example.org', user=self.user)
self.profile.save()
def test_add_address(self):
diff --git a/src/postorius/tests/mailman_api_tests/test_list_bans.py b/src/postorius/tests/mailman_api_tests/test_list_bans.py
index afca78a..a071b7a 100644
--- a/src/postorius/tests/mailman_api_tests/test_list_bans.py
+++ b/src/postorius/tests/mailman_api_tests/test_list_bans.py
@@ -17,7 +17,7 @@
"""Tests for ban lists"""
-from __future__ import absolute_import, print_function, unicode_literals
+from __future__ import absolute_import, print_function, unicode_literals
from django.contrib.auth.models import User
from django.core.urlresolvers import reverse
@@ -59,11 +59,11 @@
self.assertEqual(response.status_code, 200)
self.assertTrue('addban_form' in response.context)
self.assertContains(
- response, '')
+ response, '')
self.assertContains(
- response, '')
+ response, '')
def test_context_contains_delete_forms(self):
banned = ['banned{}@example.com'.format(i) for i in range(1, 10)]
@@ -73,11 +73,12 @@
self.assertEqual(response.status_code, 200)
for ban in banned:
self.assertContains(
- response, ''
- % ban)
+ response,
+ '' % ban)
self.assertContains(
- response, '