Run code-quality tools

Author: rambo

project/creditor/management/commands/update_membershipfees.py

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 import datetime
-import dateutil.parser
 
+import dateutil.parser
 from creditor.models import RecurringTransaction, TransactionTag
 from creditor.tests.fixtures.recurring import MembershipfeeFactory
 from django.core.management.base import BaseCommand, CommandError
@@ -26,7 +26,7 @@ def handle(self, *args, **options):
             end=None,
             start__lt=cutoff_dt,
             amount=options['oldamount']
-            ):
+        ):
             rt.end = end_dt
             rt.save()
             newrt = MembershipfeeFactory.create(amount=options['newamount'], start=cutoff_dt, end=None, owner=rt.owner)

project/creditor/tests/fixtures/recurring.py

@@ -9,11 +9,13 @@
 
 from .tags import TransactionTagFactory
 
+
 def get_tag():
     if TransactionTag.objects.count():
         return factory.fuzzy.FuzzyChoice(TransactionTag.objects.all())
     return factory.SubFactory(TransactionTagFactory, label='Membership fee', tmatch='1')
 
+
 class RecurringTransactionFactory(factory.django.DjangoModelFactory):
 
     class Meta:

project/examples/handlers.py

@@ -166,9 +166,9 @@ def import_generic_transaction(self, at, lt):
         return lt
 
     def import_tmatch_transaction(self, at, lt):
-        if len(at.reference) < 2: # To avoid indexerrors
+        if len(at.reference) < 2:  # To avoid indexerrors
             return None
-        if at.reference[0:2] == "RF": # ISO references, our lookup won't work with them, even worse: there will be exceptions
+        if at.reference[0:2] == "RF":  # ISO references, our lookup won't work with them, even worse: there will be exceptions
             return None
         # In  this example the last meaningful number (last number is checksum) of the reference is used to recognize the TransactionTag
         try:

project/ndaparser/models.py

@@ -1,23 +1,20 @@
 # -*- coding: utf-8 -*-
 import datetime
-import slugify as unicodeslugify
 
-from django.db import models, transaction
+import slugify as unicodeslugify
 from django.conf import settings
 from django.contrib.auth import get_user_model
+from django.db import models, transaction
 from django.utils.translation import ugettext_lazy as _
 
-
 from asylum.models import AsylumModel
 
 
-
 def get_sentinel_user():
     """Gets a "sentinel" user ("deleted") and for assigning as uploader"""
     return get_user_model().objects.get_or_create(username='deleted')[0]
 
 
-
 def datestamped_and_normalized(instance, filename):
     """Normalized filename and places in datestamped path"""
     file_parts = filename.split('.')
@@ -34,7 +31,6 @@ def datestamped_and_normalized(instance, filename):
     return datetime.datetime.now().strftime("ndaparser/%Y/%m/%d/{}").format(filename_normalized)
 
 
-
 class UploadedTransaction(AsylumModel):
     """Track uploaded transaction files"""
     user = models.ForeignKey(settings.AUTH_USER_MODEL, on_delete=models.SET(get_sentinel_user))
@@ -45,4 +41,4 @@ class UploadedTransaction(AsylumModel):
     class Meta:
         verbose_name = _('Uploaded transaction')
         verbose_name_plural = _('Uploaded transaction')
-        ordering = [ '-stamp' ]
+        ordering = ['-stamp']

project/ndaparser/views.py

@@ -7,8 +7,8 @@
 
 from .forms import UploadForm
 from .importer import NDAImporter
-from .parser import parseLine
 from .models import UploadedTransaction
+from .parser import parseLine
 
 
 class NordeaUploadView(FormView):
@@ -38,7 +38,7 @@ def form_valid(self, form):
         last_stamp = None
         with open(tmp.name) as fp:
             for line in fp:
-                nt =  parseLine(line)
+                nt = parseLine(line)
                 if not nt:
                     continue
                 if not last_stamp:
@@ -47,9 +47,9 @@ def form_valid(self, form):
                     last_stamp = nt.timestamp
 
         UploadedTransaction(
-            last_transaction = last_stamp,
-            file = self.request.FILES['ndafile'],
-            user = self.request.user
+            last_transaction=last_stamp,
+            file=self.request.FILES['ndafile'],
+            user=self.request.user
         ).save()
 
         # Done with the temp file, get rid of it