From 0875ef1278c5a7555ca81c3342e20f006df88137 Mon Sep 17 00:00:00 2001 From: Ludovic Stephan Date: Fri, 28 Aug 2020 18:00:54 +0200 Subject: [PATCH] Black v20 --- bda/admin.py | 2 +- bda/algorithm.py | 8 ++++---- bda/migrations/0010_spectaclerevente_shotgun.py | 5 +++-- bda/models.py | 4 ++-- bds/migrations/0003_staff_permission.py | 5 ++++- .../0005_remove_bdsprofile_certificate_file.py | 5 ++++- bds/mixins.py | 4 ++-- events/migrations/0004_unique_constraints.py | 15 ++++++++++----- events/tests/test_views.py | 3 ++- gestioncof/decorators.py | 16 ++++++++-------- gestioncof/tests/test_views.py | 3 ++- kfet/management/commands/loadkfetdevdata.py | 4 +++- kfet/tests/test_views.py | 6 ++++-- 13 files changed, 49 insertions(+), 31 deletions(-) diff --git a/bda/admin.py b/bda/admin.py index 7f626c7a..7a981d1c 100644 --- a/bda/admin.py +++ b/bda/admin.py @@ -100,7 +100,7 @@ class ParticipantAdminForm(forms.ModelForm): class ParticipantPaidFilter(admin.SimpleListFilter): """ - Permet de filtrer les participants sur s'ils ont payé leurs places ou pas + Permet de filtrer les participants sur s'ils ont payé leurs places ou pas """ title = "A payé" diff --git a/bda/algorithm.py b/bda/algorithm.py index add09335..3d4b4f51 100644 --- a/bda/algorithm.py +++ b/bda/algorithm.py @@ -10,10 +10,10 @@ class Algorithm(object): def __init__(self, shows, members, choices): """Initialisation : - - on aggrège toutes les demandes pour chaque spectacle dans - show.requests - - on crée des tables de demandes pour chaque personne, afin de - pouvoir modifier les rankings""" + - on aggrège toutes les demandes pour chaque spectacle dans + show.requests + - on crée des tables de demandes pour chaque personne, afin de + pouvoir modifier les rankings""" self.max_group = 2 * max(choice.priority for choice in choices) self.shows = [] showdict = {} diff --git a/bda/migrations/0010_spectaclerevente_shotgun.py b/bda/migrations/0010_spectaclerevente_shotgun.py index da5c014c..9acd1878 100644 --- a/bda/migrations/0010_spectaclerevente_shotgun.py +++ b/bda/migrations/0010_spectaclerevente_shotgun.py @@ -12,8 +12,9 @@ def forwards_func(apps, schema_editor): for revente in SpectacleRevente.objects.all(): is_expired = timezone.now() > revente.date_tirage() - is_direct = revente.attribution.spectacle.date >= revente.date and timezone.now() > revente.date + timedelta( - minutes=15 + is_direct = ( + revente.attribution.spectacle.date >= revente.date + and timezone.now() > revente.date + timedelta(minutes=15) ) revente.shotgun = is_expired or is_direct revente.save() diff --git a/bda/models.py b/bda/models.py index f4a0fac6..ffc203ed 100644 --- a/bda/models.py +++ b/bda/models.py @@ -172,8 +172,8 @@ class Attribution(models.Model): class ParticipantPaidQueryset(models.QuerySet): """ - Un manager qui annote le queryset avec un champ `paid`, - indiquant si un participant a payé toutes ses attributions. + Un manager qui annote le queryset avec un champ `paid`, + indiquant si un participant a payé toutes ses attributions. """ def annotate_paid(self): diff --git a/bds/migrations/0003_staff_permission.py b/bds/migrations/0003_staff_permission.py index 1f038eaa..4e61981c 100644 --- a/bds/migrations/0003_staff_permission.py +++ b/bds/migrations/0003_staff_permission.py @@ -18,5 +18,8 @@ class Migration(migrations.Migration): "verbose_name_plural": "Profils BDS", }, ), - migrations.RemoveField(model_name="bdsprofile", name="is_buro",), + migrations.RemoveField( + model_name="bdsprofile", + name="is_buro", + ), ] diff --git a/bds/migrations/0005_remove_bdsprofile_certificate_file.py b/bds/migrations/0005_remove_bdsprofile_certificate_file.py index 514c2d08..faafb3dc 100644 --- a/bds/migrations/0005_remove_bdsprofile_certificate_file.py +++ b/bds/migrations/0005_remove_bdsprofile_certificate_file.py @@ -10,5 +10,8 @@ class Migration(migrations.Migration): ] operations = [ - migrations.RemoveField(model_name="bdsprofile", name="certificate_file",), + migrations.RemoveField( + model_name="bdsprofile", + name="certificate_file", + ), ] diff --git a/bds/mixins.py b/bds/mixins.py index 4f1f8038..43607055 100644 --- a/bds/mixins.py +++ b/bds/mixins.py @@ -9,7 +9,7 @@ class StaffRequiredMixin(PermissionRequiredMixin): class MultipleFormMixin(ContextMixin): - """ Mixin pour gérer plusieurs formulaires dans la même vue. + """Mixin pour gérer plusieurs formulaires dans la même vue. Le fonctionnement est relativement identique à celui de FormMixin, dont la documentation est disponible ici : https://docs.djangoproject.com/en/3.0/ref/class-based-views/mixins-editing/ @@ -97,7 +97,7 @@ class MultipleFormMixin(ContextMixin): class ProcessMultipleFormView(View): - """ Équivalent de `ProcessFormView` pour plusieurs forms. + """Équivalent de `ProcessFormView` pour plusieurs forms. Note : il faut que *tous* les formulaires soient valides pour qu'ils soient sauvegardés ! """ diff --git a/events/migrations/0004_unique_constraints.py b/events/migrations/0004_unique_constraints.py index 3d69f99c..fa3ab610 100644 --- a/events/migrations/0004_unique_constraints.py +++ b/events/migrations/0004_unique_constraints.py @@ -13,18 +13,23 @@ class Migration(migrations.Migration): operations = [ migrations.AlterUniqueTogether( - name="extrafield", unique_together={("event", "name")}, + name="extrafield", + unique_together={("event", "name")}, ), migrations.AlterUniqueTogether( - name="extrafieldcontent", unique_together={("field", "registration")}, + name="extrafieldcontent", + unique_together={("field", "registration")}, ), migrations.AlterUniqueTogether( - name="option", unique_together={("event", "name")}, + name="option", + unique_together={("event", "name")}, ), migrations.AlterUniqueTogether( - name="optionchoice", unique_together={("option", "choice")}, + name="optionchoice", + unique_together={("option", "choice")}, ), migrations.AlterUniqueTogether( - name="registration", unique_together={("event", "user")}, + name="registration", + unique_together={("event", "user")}, ), ] diff --git a/events/tests/test_views.py b/events/tests/test_views.py index b6251ae9..ef3eda31 100644 --- a/events/tests/test_views.py +++ b/events/tests/test_views.py @@ -25,7 +25,8 @@ def make_user(name): def make_staff_user(name): view_event_perm = Permission.objects.get( - codename="view_event", content_type__app_label="events", + codename="view_event", + content_type__app_label="events", ) user = make_user(name) user.user_permissions.add(view_event_perm) diff --git a/gestioncof/decorators.py b/gestioncof/decorators.py index 28a67331..e227ed45 100644 --- a/gestioncof/decorators.py +++ b/gestioncof/decorators.py @@ -11,10 +11,10 @@ logger = logging.getLogger(__name__) def cof_required(view_func): """Décorateur qui vérifie que l'utilisateur est connecté et membre du COF. - - Si l'utilisteur n'est pas connecté, il est redirigé vers la page de - connexion - - Si l'utilisateur est connecté mais pas membre du COF, il obtient une - page d'erreur lui demandant de s'inscrire au COF + - Si l'utilisteur n'est pas connecté, il est redirigé vers la page de + connexion + - Si l'utilisateur est connecté mais pas membre du COF, il obtient une + page d'erreur lui demandant de s'inscrire au COF """ def is_cof(user): @@ -36,10 +36,10 @@ def cof_required(view_func): def buro_required(view_func): """Décorateur qui vérifie que l'utilisateur est connecté et membre du burô. - - Si l'utilisateur n'est pas connecté, il est redirigé vers la page de - connexion - - Si l'utilisateur est connecté mais pas membre du burô, il obtient une - page d'erreur 403 Forbidden + - Si l'utilisateur n'est pas connecté, il est redirigé vers la page de + connexion + - Si l'utilisateur est connecté mais pas membre du burô, il obtient une + page d'erreur 403 Forbidden """ def is_buro(user): diff --git a/gestioncof/tests/test_views.py b/gestioncof/tests/test_views.py index e5a4e033..ac18573e 100644 --- a/gestioncof/tests/test_views.py +++ b/gestioncof/tests/test_views.py @@ -589,7 +589,8 @@ class ExportMegaParticipantsViewTests(MegaHelperMixin, ViewTestCaseMixin, TestCa self.assertEqual(r.status_code, 200) self.assertCSVEqual( - r, [["u2", "", "", "", "", str(self.u2.pk), "", ""]], + r, + [["u2", "", "", "", "", str(self.u2.pk), "", ""]], ) diff --git a/kfet/management/commands/loadkfetdevdata.py b/kfet/management/commands/loadkfetdevdata.py index 43154d6e..9c1425b4 100644 --- a/kfet/management/commands/loadkfetdevdata.py +++ b/kfet/management/commands/loadkfetdevdata.py @@ -41,7 +41,9 @@ class Command(MyBaseCommand): group_chef.save() group_boy.save() - permissions_chef = Permission.objects.filter(content_type__app_label="kfet",) + permissions_chef = Permission.objects.filter( + content_type__app_label="kfet", + ) permissions_boy = Permission.objects.filter( content_type__app_label="kfet", codename__in=["is_team", "perform_deposit"] ) diff --git a/kfet/tests/test_views.py b/kfet/tests/test_views.py index e08575a3..9a9ad871 100644 --- a/kfet/tests/test_views.py +++ b/kfet/tests/test_views.py @@ -1778,7 +1778,8 @@ class KPsulPerformOperationsViewTests(ViewTestCaseMixin, TestCase): # Another Article, price=2.5, stock=20, no COF reduction self.article_no_reduction = Article.objects.create( category=ArticleCategory.objects.create( - name="Category_no_reduction", has_reduction=False, + name="Category_no_reduction", + has_reduction=False, ), name="Article_no_reduction", price=Decimal("2.5"), @@ -3799,7 +3800,8 @@ class KPsulCancelOperationsViewTests(ViewTestCaseMixin, TestCase): self.assertEqual(self.checkout.balance, Decimal("100.00")) self.kpsul_consumer_mock.group_send.assert_called_with( - "kfet.kpsul", {"checkouts": [], "articles": []}, + "kfet.kpsul", + {"checkouts": [], "articles": []}, ) @mock.patch("django.utils.timezone.now")