Running the black code formatter on the codebase

piratar · May 31, 2022 · f5b8551 · f5b8551
1 parent d843f92
commit f5b8551
Show file tree

Hide file tree

Showing 129 changed files with 4,065 additions and 1,440 deletions.
diff --git a/Makefile b/Makefile
@@ -82,6 +82,11 @@ update-dependencies:
 	@make freeze-dependencies
 
 
+.PHONY: format
+format: setup
+	@. .venv/bin/activate && black .
+
+
 .PHONY: test
 test: setup  ## Runs the unit tests
 	@. .venv/bin/activate && ./manage.py test

diff --git a/core/admin.py b/core/admin.py
@@ -7,6 +7,7 @@
 def getDerivedAdmin(base_admin, **kwargs):
     class DerivedAdmin(base_admin):
         pass
+
     derived = DerivedAdmin
     for k, v in kwargs.iteritems():
         setattr(derived, k, getattr(base_admin, k, []) + v)
@@ -26,7 +27,7 @@ class UserProfileInline(admin.StackedInline):
 
 
 class UserAdmin(auth.admin.UserAdmin):
-    inlines = (UserProfileInline, )
+    inlines = (UserProfileInline,)
 
 
 # Register the admins

diff --git a/core/ajax/utils.py b/core/ajax/utils.py
@@ -1,4 +1,3 @@
-
 from django.http import HttpResponse
 import json
 

diff --git a/core/apps.py b/core/apps.py
@@ -2,6 +2,7 @@
 
 from core.django_mdmail import convert_md_templates
 
+
 class CoreConfig(AppConfig):
     name = 'core'
     verbose_name = 'Wasa2il Core'

diff --git a/core/authentication.py b/core/authentication.py
@@ -21,6 +21,7 @@ def authenticate(self, request, username=None, password=None):
 
 class EmailAuthenticationBackend(CustomAuthenticationBackend):
     """Allow users to log in using their e-mail address"""
+
     def custom_get_user(self, email):
         try:
             return User.objects.get(email=email)
@@ -30,6 +31,7 @@ def custom_get_user(self, email):
 
 class SSNAuthenticationBackend(CustomAuthenticationBackend):
     """Allow users to log in using their SSN"""
+
     def custom_get_user(self, ssn):
         # FIXME: This may be Iceland specific; we ignore dashes.
         ssn = ssn.replace('-', '').strip()

diff --git a/core/contextprocessors.py b/core/contextprocessors.py
@@ -13,7 +13,7 @@ def globals(request):
         'INSTANCE_VERSION': settings.WASA2IL_VERSION,
         'FEATURES': settings.FEATURES,
         'GCM_APP_ID': settings.GCM_APP_ID,
-        'settings': settings
+        'settings': settings,
     }
 
     # Get global variables from GlobalsMiddleWare.
@@ -25,7 +25,9 @@ def globals(request):
 def auto_logged_out(request):
     if hasattr(request, 'auto_logged_out') and request.auto_logged_out:
         return {
-            'splash_message': _('For security reasons, you have been automatically logged out due to inactivity.')
+            'splash_message': _(
+                'For security reasons, you have been automatically logged out due to inactivity.'
+            )
         }
 
     return {}
diff --git a/core/dataviews.py b/core/dataviews.py
@@ -3,90 +3,91 @@
 from election.models import Election
 from issue.models import Issue
 
+
 @jsonize
 def recent_activity(request):
 
     # Names prefixed with "q_" to distinguish them as queries. We will be
     # returning the same data in JSON format, which we will call "issues" and
     # "elections".
-    q_issues = Issue.objects.select_related('polity').recent().order_by('polity__id', '-deadline_votes')
-    q_elections = Election.objects.prefetch_related(
-        'electionvote_set',
-        'candidate_set'
-    ).select_related(
-        'result'
-    ).recent()
+    q_issues = (
+        Issue.objects.select_related('polity')
+        .recent()
+        .order_by('polity__id', '-deadline_votes')
+    )
+    q_elections = (
+        Election.objects.prefetch_related('electionvote_set', 'candidate_set')
+        .select_related('result')
+        .recent()
+    )
 
     issues = []
     for q_issue in q_issues:
-        issues.append({
-            # Web location for further info on the issue.
-            'url': request.build_absolute_uri(
-                reverse('issue', args=(q_issue.polity_id, q_issue.id))
-            ),
-
-            # The polity to which this issue belongs.
-            'polity': q_issue.polity.name,
-
-            # The polity's short name, if available.
-            'polity_shortname': q_issue.polity.name_short,
-
-            # A unique identifier for formal reference. Example: 6/2019
-            'log_number': '%d/%d' % (q_issue.issue_num, q_issue.issue_year),
-
-            # The issue's name or title.
-            'name': q_issue.name,
-
-            # Options are: concluded/voting/accepting_proposals/discussion
-            # Note that the state does not give the *result*, i.e. whether the
-            # proposal was accepted or rejected, but rather where the issue is
-            # currently in the decision-making process. Therefore "concluded"
-            # only means that the issue has concluded, but does not tell us
-            # *how* it concluded.
-            'state': q_issue.issue_state(),
-
-            # Translated, human-readable version of the issue state.
-            'state_human_readable': q_issue.get_issue_state_display(),
-
-            # A boolean indicating whether the issue has been approved or not.
-            'majority_reached': q_issue.majority_reached(),
-
-            # Translated, human-readable version of the result.
-            'majority_reached_human_readable': q_issue.get_majority_reached_display(),
-
-            # When the issue's fate is not determined by vote from within
-            # Wasa2il, for example when a vote is made outside of Wasa2il but
-            # still placed here for reference or historical reasons, or when
-            # an issue is retracted without ever coming to a vote.
-            #
-            # Consider displaying only this value if it is non-null, and the
-            # `majority_reached` value only if this is null.
-            'special_process': q_issue.special_process,
-
-            # Translated, human-readable version of the result.
-            'special_process_human_readable': q_issue.get_special_process_display(),
-
-            # Comment count.
-            'comment_count': q_issue.comment_count,
-
-            # Vote count.
-            'vote_count': q_issue.votecount,
-        })
+        issues.append(
+            {
+                # Web location for further info on the issue.
+                'url': request.build_absolute_uri(
+                    reverse('issue', args=(q_issue.polity_id, q_issue.id))
+                ),
+                # The polity to which this issue belongs.
+                'polity': q_issue.polity.name,
+                # The polity's short name, if available.
+                'polity_shortname': q_issue.polity.name_short,
+                # A unique identifier for formal reference. Example: 6/2019
+                'log_number': '%d/%d'
+                % (q_issue.issue_num, q_issue.issue_year),
+                # The issue's name or title.
+                'name': q_issue.name,
+                # Options are: concluded/voting/accepting_proposals/discussion
+                # Note that the state does not give the *result*, i.e. whether the
+                # proposal was accepted or rejected, but rather where the issue is
+                # currently in the decision-making process. Therefore "concluded"
+                # only means that the issue has concluded, but does not tell us
+                # *how* it concluded.
+                'state': q_issue.issue_state(),
+                # Translated, human-readable version of the issue state.
+                'state_human_readable': q_issue.get_issue_state_display(),
+                # A boolean indicating whether the issue has been approved or not.
+                'majority_reached': q_issue.majority_reached(),
+                # Translated, human-readable version of the result.
+                'majority_reached_human_readable': q_issue.get_majority_reached_display(),
+                # When the issue's fate is not determined by vote from within
+                # Wasa2il, for example when a vote is made outside of Wasa2il but
+                # still placed here for reference or historical reasons, or when
+                # an issue is retracted without ever coming to a vote.
+                #
+                # Consider displaying only this value if it is non-null, and the
+                # `majority_reached` value only if this is null.
+                'special_process': q_issue.special_process,
+                # Translated, human-readable version of the result.
+                'special_process_human_readable': q_issue.get_special_process_display(),
+                # Comment count.
+                'comment_count': q_issue.comment_count,
+                # Vote count.
+                'vote_count': q_issue.votecount,
+            }
+        )
 
     elections = []
     for q_election in q_elections:
         # See comments for issue above, which are more detailed but are mostly
         # applicable to this section as well.
-        elections.append({
-            'url': request.build_absolute_uri(reverse('election', args=(q_election.polity_id, q_election.id))),
-            'polity': q_election.polity.name,
-            'polity_shortname': q_election.polity.name_short,
-            'name': q_election.name,
-            'state': q_election.election_state(),
-            'state_human_readable': q_election.get_election_state_display(),
-            'candidate_count': q_election.candidate_set.count(),
-            'vote_count': q_election.get_vote_count(),
-        })
+        elections.append(
+            {
+                'url': request.build_absolute_uri(
+                    reverse(
+                        'election', args=(q_election.polity_id, q_election.id)
+                    )
+                ),
+                'polity': q_election.polity.name,
+                'polity_shortname': q_election.polity.name_short,
+                'name': q_election.name,
+                'state': q_election.election_state(),
+                'state_human_readable': q_election.get_election_state_display(),
+                'candidate_count': q_election.candidate_set.count(),
+                'vote_count': q_election.get_vote_count(),
+            }
+        )
 
     return {
         'elections': elections,

diff --git a/core/django_mdmail.py b/core/django_mdmail.py
@@ -46,6 +46,7 @@
 
 from django.conf import settings
 from django.core.mail import EmailMultiAlternatives
+
 if sys.version_info[0] == 3:
     from email.mime.image import MIMEImage
 else:
@@ -57,17 +58,24 @@
 OVERRIDE_WARNING = 'WARNING! THIS FILE IS AUTO-GENERATED by django_mdmail upon Django startup. Changes to this file WILL be overwritten. In the same directory, there should be a file with the same name, except an ".md" ending (for Markdown). Edit that instead and restart Django.'
 
 
-def send_mail(subject, message, from_email, recipient_list, fail_silently=False, auth_user=None, auth_password=None, connection=None, html_message=None):
+def send_mail(
+    subject,
+    message,
+    from_email,
+    recipient_list,
+    fail_silently=False,
+    auth_user=None,
+    auth_password=None,
+    connection=None,
+    html_message=None,
+):
 
     # Have `mdmail` do its Markdown magic.
     content = EmailContent(message)
 
     # Create the email message and fill it with the relevant data.
     email = EmailMultiAlternatives(
-        subject,
-        content.text,
-        from_email,
-        recipient_list
+        subject, content.text, from_email, recipient_list
     )
     email.attach_alternative(html_message or content.html, 'text/html')
     email.mixed_subtype = 'related'
@@ -115,7 +123,9 @@ def ready(self):
                 convert_md_templates()
     '''
 
-    override_comment = '{%% comment %%}%s{%% endcomment %%}\n' % OVERRIDE_WARNING
+    override_comment = (
+        '{%% comment %%}%s{%% endcomment %%}\n' % OVERRIDE_WARNING
+    )
 
     # Find all the template directories we'll need to process and put them
     # in a flat list.

diff --git a/core/forms.py b/core/forms.py
@@ -21,11 +21,25 @@ class EmailWantedField(ChoiceWidget):
 
 
 class UserProfileForm(Wasa2ilForm):
-    email = EmailField(label=_("E-mail"), help_text=_("You can change your email address, but will then need to verify it."))
+    email = EmailField(
+        label=_("E-mail"),
+        help_text=_(
+            "You can change your email address, but will then need to verify it."
+        ),
+    )
 
     class Meta:
         model = UserProfile
-        fields = ('displayname', 'email', 'phone', 'picture', 'bio', 'declaration_of_interests', 'language', 'email_wanted')
+        fields = (
+            'displayname',
+            'email',
+            'phone',
+            'picture',
+            'bio',
+            'declaration_of_interests',
+            'language',
+            'email_wanted',
+        )
 
     # We need to keep the 'request' object for certain kinds of validation ('picture' in this case)
     def __init__(self, *args, **kwargs):
@@ -38,15 +52,20 @@ def clean_picture(self):
         picture = self.request.FILES.get('picture')
         if picture:
             if picture.name.find('.') == -1:
-                raise ValidationError(_('Filename must contain file extension'))
+                raise ValidationError(
+                    _('Filename must contain file extension')
+                )
             else:
                 ext = picture.name.split('.')[-1].lower()
                 allowed_exts = ['jpg', 'jpeg', 'png', 'gif']
                 if ext not in allowed_exts:
-                    raise ValidationError(u'%s: %s' % (
-                        _('Only the following file endings are allowed'),
-                        ', '.join(allowed_exts)
-                    ))
+                    raise ValidationError(
+                        u'%s: %s'
+                        % (
+                            _('Only the following file endings are allowed'),
+                            ', '.join(allowed_exts),
+                        )
+                    )
 
         return data
 
@@ -55,13 +74,16 @@ class Wasa2ilRegistrationForm(RegistrationForm):
     username = UsernameField(
         widget=TextInput(attrs={'autofocus': True}),
         label=_('Username'),
-        help_text=_('Only letters, numbers and the symbols @/./+/-/_ are allowed.') #_('Aðeins er leyfilegt að nota bókstafi, tölustafi og táknin @/./+/-/_')
+        help_text=_(
+            'Only letters, numbers and the symbols @/./+/-/_ are allowed.'
+        ),  # _('Aðeins er leyfilegt að nota bókstafi, tölustafi og táknin @/./+/-/_')
     )
     email_wanted = TypedChoiceField(
         choices=((True, _('Yes')), (False, _('No'))),
         widget=EmailWantedField,
-        label=_('Consent for sending email')
+        label=_('Consent for sending email'),
     )
 
+
 class PushNotificationForm(Form):
     text = CharField(label=_('Message'))
diff --git a/core/management/commands/closeissues.py b/core/management/commands/closeissues.py
@@ -5,8 +5,8 @@
 
 from core.models import *
 
-class Command(BaseCommand):
 
+class Command(BaseCommand):
     def handle(self, *args, **options):
 
         now = datetime.now()
@@ -15,7 +15,9 @@ def handle(self, *args, **options):
         for i in issues:
             if i.deadline_votes > now:
 
-                stdout.write("Setting closing time of issue '%s' to now..." % i.name)
+                stdout.write(
+                    "Setting closing time of issue '%s' to now..." % i.name
+                )
                 stdout.flush()
 
                 i.deadline_votes = now
@@ -27,4 +29,3 @@ def handle(self, *args, **options):
                 i.save()
 
                 stdout.write(" done\n")
-