participation/views.py

@@ -313,7 +313,6 @@ class TeamUpdateView(LoginRequiredMixin, UpdateView):
                                                           instance=self.object.participation)
         if not self.request.user.registration.is_volunteer:
             del context["participation_form"].fields['final']
-            context["participation_form"].helper.layout.remove('final')
         context["title"] = _("Update team {trigram}").format(trigram=self.object.trigram)
         return context
 
@@ -322,7 +321,6 @@ class TeamUpdateView(LoginRequiredMixin, UpdateView):
         participation_form = ParticipationForm(data=self.request.POST or None, instance=self.object.participation)
         if not self.request.user.registration.is_volunteer:
             del participation_form.fields['final']
-            participation_form.helper.layout.remove('final')
         if not participation_form.is_valid():
             return self.form_invalid(form)
 

registration/forms.py

@@ -7,6 +7,8 @@ from django.contrib.auth.forms import UserCreationForm
 from django.contrib.auth.models import User
 from django.core.exceptions import ValidationError
 from django.forms import FileInput
+from django.utils import timezone
+from django.utils.text import format_lazy
 from django.utils.translation import gettext_lazy as _
 
 from .models import CoachRegistration, ParticipantRegistration, Payment, \

registration/templates/registration/tex/Autorisation_parentale.tex

@@ -70,11 +70,7 @@ ses propres moyens et sous la responsabilité du/de la représentant\cdt{}e lég
 
 \vspace{8ex}
 
-Fait à \vrule width 10cm height 0pt depth 0.4pt, le \phantom{232323}/\phantom{XXX}/{% now "Y" %}
+Fait à \vrule width 10cm height 0pt depth 0.4pt, le \phantom{232323}/\phantom{XXX}/{% now "Y" %},
-
-\vspace{4ex}
-
-Signature :
 
 \vfill
 \vfill

registration/views.py

@@ -18,7 +18,7 @@ from django.http import FileResponse, Http404
 from django.shortcuts import redirect, resolve_url
 from django.template.loader import render_to_string
 from django.urls import reverse_lazy
-from django.utils import timezone, translation
+from django.utils import translation
 from django.utils.crypto import get_random_string
 from django.utils.http import urlsafe_base64_decode
 from django.utils.text import format_lazy
@@ -60,10 +60,9 @@ class SignupView(CreateView):
 
         return context
 
-    def get_form(self, form_class=None):
+    @transaction.atomic
-        form = super().get_form(form_class)
+    def form_valid(self, form):
-        if self.request.method in ("POST", "PUT") \
+        if not self.request.user.registration.is_admin:
-                and (not self.request.user.is_authenticated or not self.request.user.registration.is_admin):
             # Check that registrations are opened
             now = timezone.now()
             if now < settings.REGISTRATION_DATES['open']:
@@ -74,10 +73,9 @@ class SignupView(CreateView):
                 form.add_error(None, format_lazy(_("Registrations for this year are closed since "
                                                 "{closing_date:%Y-%m-%d %H:%M}."),
                                                 closing_date=settings.REGISTRATION_DATES['close']))
-        return form
+            if not form.is_valid():
+                return self.form_invalid(form)
 
-    @transaction.atomic
-    def form_valid(self, form):
         role = form.cleaned_data["role"]
         if role == "participant":
             registration_form = StudentRegistrationForm(self.request.POST)