flake8 --max-line-length=119 applied.

Author: jawsper

src/barsystem/admin.py

@@ -2,34 +2,43 @@
 from .models import Person, Product, ProductCategory, Journal, Token, VendingMachineProduct
 from .functions import money_display
 
+
 @admin.register(Person)
 class PersonAdmin(admin.ModelAdmin):
-    list_display = ('id', 'nick_name', 'active', 'member', 'special', 'first_name', 'last_name', 'amount', 'balance_limit', 'type')
+    list_display = ('id', 'nick_name', 'active', 'member', 'special', 'first_name',
+                    'last_name', 'amount', 'balance_limit', 'type')
     ordering = ('id',)
 
+
 @admin.register(Product)
 class ProductAdmin(admin.ModelAdmin):
-    list_display = ('name', 'active', 'special', 'type', 'category', 'member_price_', 'standard_price_')
+    list_display = ('name', 'active', 'special', 'type', 'category', 'member_price_',
+                    'standard_price_')
     ordering = ('id',)
 
     def member_price_(self, obj):
         return money_display(obj.member_price)
+
     def standard_price_(self, obj):
         return money_display(obj.standard_price)
 
+
 @admin.register(ProductCategory)
 class ProductCategoryAdmin(admin.ModelAdmin):
     pass
 
+
 @admin.register(Journal)
 class JournalAdmin(admin.ModelAdmin):
     list_display = ('moment', 'sender', 'recipient', 'product', 'items', 'amount')
     date_hierarchy = 'moment'
 
+
 @admin.register(Token)
 class TokenAdmin(admin.ModelAdmin):
     list_display = ('person', 'type', 'value')
 
+
 @admin.register(VendingMachineProduct)
 class VendingMachineProductAdmin(admin.ModelAdmin):
     list_display = ('code', 'product', 'virtual')

src/barsystem/cart.py

@@ -4,6 +4,7 @@
 from django.utils import timezone
 from django.db import transaction
 
+
 class Cart(dict):
     def __init__(self, data=None, person=None):
         if data:
@@ -34,6 +35,7 @@ def can_checkout(self):
             return self.person.balance - total >= self.person.balance_limit
 
         return True
+
     def checkout(self, test=False):
         if not self.can_checkout():
             return False

src/barsystem/functions.py

@@ -2,19 +2,21 @@
 from django.core.mail import send_mail
 from django.conf import settings
 
+
 def money_display(value):
-	if value is None:
-		return str(None)
-	min_digits = 2
-	max_digits = 4
-	negative = False
-	if value < 0:
-		value *= -1
-		negative = True
-	var = floatformat(value, 4)
-	if var[-1] == '0': var = var[:-1]
-	if var[-1] == '0': var = var[:-1]
-	return '{}{}{}'.format('-' if negative else '', '€ ', var)
+    if value is None:
+        return str(None)
+    negative = False
+    if value < 0:
+        value *= -1
+        negative = True
+    # Make sure precision is 4 digits, but strip up to 2 trailing zeroes.
+    var = floatformat(value, 4)
+    if var[-1] == '0':
+        var = var[:-1]
+    if var[-1] == '0':
+        var = var[:-1]
+    return '{}{}{}'.format('-' if negative else '', '€ ', var)
 
 
 EMAIL_OVERDRAWN_SUBJECT = 'Barsystem: {nickname} is over spending limit!'
@@ -33,12 +35,12 @@ def money_display(value):
 
 
 def send_overdrawn_mail(naughty_person, transaction_amount):
-	person_name = '{} ({} {})'.format(naughty_person.nick_name, naughty_person.first_name, naughty_person.last_name)
-	values = {
-		'nickname': naughty_person.nick_name,
-		'transaction_amount': money_display(transaction_amount),
-		'balance': money_display(naughty_person.amount),
-	}
-	subject = EMAIL_OVERDRAWN_SUBJECT.format(**values)
-	message = EMAIL_OVERDRAWN_BODY.format(**values)
-	send_mail(subject, message, settings.EMAIL_HOST_USER, settings.EMAIL_TARGET, fail_silently=True)
+    # person_name = '{} ({} {})'.format(naughty_person.nick_name, naughty_person.first_name, naughty_person.last_name)
+    values = {
+        'nickname': naughty_person.nick_name,
+        'transaction_amount': money_display(transaction_amount),
+        'balance': money_display(naughty_person.amount),
+    }
+    subject = EMAIL_OVERDRAWN_SUBJECT.format(**values)
+    message = EMAIL_OVERDRAWN_BODY.format(**values)
+    send_mail(subject, message, settings.EMAIL_HOST_USER, settings.EMAIL_TARGET, fail_silently=True)

src/barsystem/gui/__init__.py

@@ -1,3 +0,0 @@
-def persons_import(filename):
-    with open(filename, 'r') as f:
-        lines = f.readlines()

src/barsystem/gui/templatetags/money_display.py

@@ -1,9 +1,9 @@
 from django import template
-from django.template.defaultfilters import floatformat
 import barsystem.functions
 
 register = template.Library()
 
+
 @register.filter
 def money_display(value, *args, **kwargs):
-	return barsystem.functions.money_display(value)
+    return barsystem.functions.money_display(value)

src/barsystem/gui/urls.py

@@ -1,16 +1,17 @@
-from django.conf.urls import include, url
+from django.conf.urls import url
 
 from barsystem.gui import views
 
+
 urlpatterns = [
-	url(r'^people/$', views.PeopleView.as_view(), name='people'),
-	url(r'^people/set/(?P<person_id>\d+)/$', views.PeopleSetView.as_view(), name='people_set'),
-	url(r'^cart/add/', views.AddToCartView.as_view(), name='add_to_cart'),
-	url(r'^products/$', views.ProductsView.as_view(), name='products'),
-	url(r'^products/get/$', views.ProductsGetView.as_view(), name='products_get'),
-	url(r'^products/confirm/$', views.ProductsConfirmView.as_view(), name='products_confirm'),
-	url(r'^transactions/$', views.TransactionsView.as_view(), name='transactions'),
-	url(r'^create_account/$', views.CreateAccountView.as_view(), name='create_account'),
-	url(r'^delete_account/$', views.DeleteAccountView.as_view(), name='delete_account'),
-	url(r'^$', views.IndexView.as_view(), name='index'),
-]
+    url(r'^people/$', views.PeopleView.as_view(), name='people'),
+    url(r'^people/set/(?P<person_id>\d+)/$', views.PeopleSetView.as_view(), name='people_set'),
+    url(r'^cart/add/', views.AddToCartView.as_view(), name='add_to_cart'),
+    url(r'^products/$', views.ProductsView.as_view(), name='products'),
+    url(r'^products/get/$', views.ProductsGetView.as_view(), name='products_get'),
+    url(r'^products/confirm/$', views.ProductsConfirmView.as_view(), name='products_confirm'),
+    url(r'^transactions/$', views.TransactionsView.as_view(), name='transactions'),
+    url(r'^create_account/$', views.CreateAccountView.as_view(), name='create_account'),
+    url(r'^delete_account/$', views.DeleteAccountView.as_view(), name='delete_account'),
+    url(r'^$', views.IndexView.as_view(), name='index'),
+]