fix online register

tournaments/models/enums.py

@@ -15,6 +15,8 @@ class FederalCategory(models.IntegerChoices):
 
     @staticmethod
     def female_in_male_assimilation_addition(rank: int) -> int:
+        if rank is None:
+            return 0
         if 1 <= rank <= 10:
             return 400
         elif 11 <= rank <= 30:

tournaments/repositories.py

@@ -23,7 +23,7 @@ class TournamentRegistrationRepository:
             is_captain = False
             player_licence_id = player_data['licence_id']
             if player_licence_id and stripped_license:
-                if player_licence_id.startswith(stripped_license):
+                if stripped_license.lower() in player_licence_id.lower():
                     is_captain = True
 
             sex, rank, computed_rank = TournamentRegistrationRepository._compute_rank_and_sex(
@@ -31,6 +31,7 @@ class TournamentRegistrationRepository:
                 player_data
             )
 
+            print("create_player_registrations", player_data.get('last_name'), sex, rank, computed_rank)
             data_source = None
             if player_data.get('found_in_french_federation', False) == True:
                 data_source = PlayerDataSource.FRENCH_FEDERATION
@@ -66,14 +67,18 @@ class TournamentRegistrationRepository:
     @staticmethod
     def _compute_rank_and_sex(tournament, player_data):
         is_woman = player_data.get('is_woman', False)
-        rank = player_data.get('rank', 0)
+        rank = player_data.get('rank', None)
+        if rank is None:
+            computed_rank = 100000
+        else:
             computed_rank = rank
-        sex = PlayerSexType.MALE
 
+        sex = PlayerSexType.MALE
         if is_woman:
             sex = PlayerSexType.FEMALE
             if tournament.federal_category == FederalCategory.MEN:
                 computed_rank = str(int(computed_rank) +
                     FederalCategory.female_in_male_assimilation_addition(int(rank)))
 
+        print("_compute_rank_and_sex", sex, rank, computed_rank)
         return sex, rank, computed_rank

tournaments/services/tournament_registration.py

@@ -31,9 +31,18 @@ class TournamentRegistrationService:
 
         if 'add_player' in self.request.POST:
             self.handle_add_player()
+        if 'remove_player' in self.request.POST:
+            self.handle_remove_player()
         elif 'register_team' in self.request.POST:
             self.handle_team_registration()
 
+    def handle_remove_player(self):
+        team_registration = self.request.session.get('team_registration', [])
+        if team_registration:  # Check if list is not empty
+            team_registration.pop()  # Remove last element
+            self.request.session['team_registration'] = team_registration
+        self.context['current_players'] = team_registration
+
     def handle_add_player(self):
         if not self.context['add_player_form'].is_valid():
             return
@@ -106,6 +115,7 @@ class TournamentRegistrationService:
         self.initialize_session_data()
 
     def add_player_to_session(self, player_data):
+        print("add_player_to_session", player_data)
         if not self.request.session.get('team_registration'):
             self.request.session['team_registration'] = []
 
@@ -216,6 +226,7 @@ class TournamentRegistrationService:
         return False
 
     def _handle_valid_names(self, player_data):
+        print("_handle_valid_names", player_data)
         if player_data.get('rank') is None:
             self._set_default_rank(player_data)
 
@@ -224,8 +235,8 @@ class TournamentRegistrationService:
         self.context['add_player_form'].first_tournament = False
 
     def _handle_invalid_names(self, licence_id, player_data):
-        if not self.context['add_player_form'].first_tournament:
         data, found = get_player_name_from_csv(self.tournament.federal_category, licence_id)
+        print("_handle_invalid_names get_player_name_from_csv", data, found)
         if found and data:
             self._update_player_data_from_csv(player_data, data)
             player_check = self._player_check(player_data)
@@ -235,6 +246,7 @@ class TournamentRegistrationService:
             else:
                 return
         else:
+            print("_handle_first_tournament_case")
             self._handle_first_tournament_case(data)
 
     def _set_default_rank(self, player_data):
@@ -245,7 +257,7 @@ class TournamentRegistrationService:
                 self.request.session['is_woman'] = data['is_woman']
                 self.request.session.modified = True
 
-        player_data['rank'] = self.request.session.get('last_rank', 0)
+        player_data['rank'] = self.request.session.get('last_rank', None)
         player_data['is_woman'] = self.request.session.get('is_woman', False)
 
     def _update_user_license(self, licence_id):
@@ -261,6 +273,7 @@ class TournamentRegistrationService:
             self.context['add_player_form'].first_tournament = False
 
     def _update_player_data_from_csv(self, player_data, csv_data):
+        print("_update_player_data_from_csv", player_data, csv_data)
         player_data.update({
             'first_name': csv_data['first_name'],
             'last_name': csv_data['last_name'],
@@ -278,6 +291,7 @@ class TournamentRegistrationService:
         })
 
     def _handle_first_tournament_case(self, data):
+        print("_handle_first_tournament_case", data)
         if data:
             self.request.session['last_rank'] = data['rank']
             self.request.session['is_woman'] = data['is_woman']
@@ -307,6 +321,6 @@ class TournamentRegistrationService:
     def _license_already_registered(self, stripped_license):
         return PlayerRegistration.objects.filter(
             team_registration__tournament=self.tournament,
-            licence_id__startswith=stripped_license,
+            licence_id__icontains=stripped_license,
             team_registration__walk_out=False
         ).exists()

tournaments/services/tournament_unregistration.py

@@ -53,7 +53,7 @@ class TournamentUnregistrationService:
 
     def _find_player_registration(self):
         self.player_registration = PlayerRegistration.objects.filter(
-            licence_id__startswith=self.request.user.licence_id,
+            licence_id__icontains=self.request.user.licence_id,
             team_registration__tournament_id=self.tournament.id,
         ).first()
 

tournaments/templates/register_tournament.html

@@ -23,7 +23,7 @@
   {% if registration_successful %}
     <p>Merci, l'inscription a bien été envoyée au juge-arbitre.</p>
     <p style="text-align: justify;">
-    Un email de confirmation a été envoyé à {{ user.email }} pour confirmer votre inscription. Pensez à vérifier vos spams si vous ne recevez pas l'email. En cas de problème, contactez le juge-arbitre.
+    Un email de confirmation a été envoyé à l'adresse associée à votre compte Padel Club ({{ user.email }}). Pensez à vérifier vos spams si vous ne recevez pas l'email. En cas de problème, contactez le juge-arbitre.
     </p>
   {% else %}
   <form method="post">
@@ -48,7 +48,24 @@
       </p>
       <ul>
               {% for player in current_players %}
-                  <li>{{ player.first_name }} {{ player.last_name }}{% if player.licence_id %} ({{ player.licence_id }}){% endif %}</li>
+                  <li>
+                      <div>
+                          {{ player.first_name }} {{ player.last_name }}{% if player.licence_id %} ({{ player.licence_id }}){% endif %}
+                      </div>
+                      <div>
+                          {{ player.club_name }}
+                      </div>
+                      <div>
+                          Classement à ce jour : {{ player.rank }}
+                      </div>
+                      {% if not forloop.first %}  <!-- Only show remove button if not the first player -->
+                      <div>
+                          <button type="submit" name="remove_player" class="btn small-button">
+                              modifier
+                          </button>
+                      </div>
+                      {% endif %}
+                  </li>
               {% endfor %}
           </ul>
       {% endif %}
@@ -73,6 +90,9 @@
           {% endif %}
           {% if add_player_form.first_tournament or add_player_form.user_without_licence or tournament.license_is_required is False %}
             {% if not add_player_form.user_without_licence and tournament.license_is_required is True %}
+            <div class="semibold">
+                Padel Club n'a pas trouvé votre partenaire, il se peut qu'il s'agisse de son premier tournoi. Contacter le juge-arbitre après l'inscription si ce n'est pas le cas.
+            </div>
             <div class="semibold">
                 Précisez les informations du joueur :
             </div>

tournaments/utils/player_search.py

@@ -9,7 +9,7 @@ from tournaments.models.enums import FederalCategory
 def clean_licence_id(licence_id):
     # This regex matches the trailing letters (non-digits) and removes them
     cleaned_licence_id = re.sub(r'\D+$', '', str(licence_id))  # \D+ matches non-digits at the end
-    return cleaned_licence_id
+    return cleaned_licence_id.lstrip("0")
 
 def get_player_name_from_csv(category, licence_id, base_folder=None):
     """
@@ -29,6 +29,8 @@ def get_player_name_from_csv(category, licence_id, base_folder=None):
     else:
         cleaned_licence_id = None
 
+    print("get_player_name_from_csv", cleaned_licence_id)
+
     def extract_date(file_name):
         """
         Extract the date (MM-YYYY) from the file name and return it as a datetime object.

tournaments/views.py

@@ -170,7 +170,7 @@ def tournament_info(request, tournament_id):
             stripped_license = validator.stripped_license
             # Check if there is a PlayerRegistration for this user in this tournament
             registered_user = PlayerRegistration.objects.filter(
-                licence_id__startswith=stripped_license,
+                licence_id__icontains=stripped_license,
                 team_registration__tournament=tournament,
                 team_registration__walk_out=False,
             ).first()
@@ -747,7 +747,7 @@ def my_tournaments(request):
 
     def filter_user_tournaments(tournaments):
         return [t for t in tournaments if t.teamregistration_set.filter(
-            playerregistration__licence_id__startswith=stripped_license,
+            playerregistration__licence_id__icontains=stripped_license,
             walk_out=False
         ).exists()]