diff --git a/web/webapp/admin.py b/web/webapp/admin.py index 178abdd..e596778 100644 --- a/web/webapp/admin.py +++ b/web/webapp/admin.py @@ -70,7 +70,7 @@ def admin_ldap_sync(): changes['starving']['add'] = form.starving_to_add.data changes['starving']['remove'] = form.starving_to_remove.data - directory.update_member_groups(g.ldap, changes) + directory.update_member_groups(g.ldap, changes) return render_template("admin_ldap_sync.html", form=form) @@ -107,7 +107,7 @@ def admin_member(membername): @bp.route("/admin/member//policy:") @login_required @admin_required -def admin_member_set_policy(membername,policy): +def admin_member_set_policy(membername, policy): member = models.Member.query.filter_by(username=membername).first() member.payment_policy = models.PaymentPolicy[policy].value db.session.add(member) @@ -154,12 +154,13 @@ def match_auto(): for transfer in transfers_unmatched: matchability, member, months = transfer.get_matchability() try: - print "[i] Matching transfer {} for {:.2f}PLN by member {}, {} months".format(transfer.id, transfer.amount/100, member.username, months) + print "[i] Matching transfer {} for {:.2f}PLN by member {}, {} months".format( + transfer.id, transfer.amount/100, member.username, months) except AttributeError: print "[e] Member data invalid, WTF - {}".format(repr(member)) continue if matchability == models.Transfer.MATCH_OK: - if len(member.transfers) > 0: + if member.transfers: year, month = member.get_next_unpaid() if None in (year, month): print "[w] next_unpaid borked, skipping" @@ -288,7 +289,8 @@ def sendspam(): if form.dry_run.data: readable = [ - msg.as_string().split('\n\n')[0] + '\n\n' + msg.get_payload(decode=True) for msg in spam] + msg.as_string().split('\n\n')[0] + '\n\n' + + msg.get_payload(decode=True) for msg in spam] return Response('\n====\n'.join(readable), mimetype='text/text') for msg in spam: