From 436334f23dcf60e73bcb894a5e657da0fcb99428 Mon Sep 17 00:00:00 2001
From: Christian Merten <christian@merten.dev>
Date: Sun, 15 Dec 2024 00:46:56 +0100
Subject: [PATCH] members/admin: show new members section also to
 non-waitinglist managers

---
 jdav_web/members/admin.py                       |  5 +++--
 jdav_web/members/models.py                      |  3 +--
 jdav_web/templates/admin/members/app_index.html | 11 ++++++++++-
 3 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/jdav_web/members/admin.py b/jdav_web/members/admin.py
index a4a1f95..45e6868 100644
--- a/jdav_web/members/admin.py
+++ b/jdav_web/members/admin.py
@@ -407,7 +407,7 @@ class MemberUnconfirmedAdmin(CommonAdminMixin, admin.ModelAdmin):
                 'comments',
                 'legal_guardians',
                 'dav_badge_no',
-                'active', 'echoed',
+                'echoed',
                 'user',
              ]
          }
@@ -440,7 +440,7 @@ class MemberUnconfirmedAdmin(CommonAdminMixin, admin.ModelAdmin):
     search_fields = ('prename', 'lastname', 'email')
     list_filter = ('group', 'confirmed_mail', 'confirmed_alternative_mail')
     readonly_fields = ['confirmed_mail', 'confirmed_alternative_mail',
-                       'good_conduct_certificate_valid']
+                       'good_conduct_certificate_valid', 'echoed']
     actions = ['request_mail_confirmation', 'confirm', 'demote_to_waiter_action']
     inlines = [EmergencyContactInline]
     change_form_template = "members/change_member_unconfirmed.html"
@@ -454,6 +454,7 @@ class MemberUnconfirmedAdmin(CommonAdminMixin, admin.ModelAdmin):
 
     field_change_permissions = {
         'user': 'members.may_set_auth_user',
+        'group': 'members.may_change_member_group',
         'good_conduct_certificate_presented_date': 'members.may_change_organizationals',
         'has_key': 'members.may_change_organizationals',
         'has_free_ticket_gym': 'members.may_change_organizationals',
diff --git a/jdav_web/members/models.py b/jdav_web/members/models.py
index 8c3d27b..ad497fd 100644
--- a/jdav_web/members/models.py
+++ b/jdav_web/members/models.py
@@ -465,8 +465,7 @@ class Member(Person):
     def registration_ready(self):
         """Returns if the member is currently unconfirmed and all email addresses
         are confirmed."""
-        return not self.confirmed and self.confirmed_alternative_mail and self.confirmed_mail and\
-            all([emc.confirmed_mail for emc in self.emergencycontact_set.all()])
+        return not self.confirmed and self.confirmed_alternative_mail and self.confirmed_mail
 
     def confirm_mail(self, key):
         ret = super().confirm_mail(key)
diff --git a/jdav_web/templates/admin/members/app_index.html b/jdav_web/templates/admin/members/app_index.html
index 9773569..6721797 100644
--- a/jdav_web/templates/admin/members/app_index.html
+++ b/jdav_web/templates/admin/members/app_index.html
@@ -45,11 +45,12 @@ Hier siehst du alle von dir geleiteten Ausfahrten und für dich sichtbare Teilne
         <td></td>
     </tr>
 </table>
-{% if perms.members.may_manage_waiting_list %}
+{% if perms.members.view_memberunconfirmedproxy %}
 <br>
 <div class="app-members module current-app">
 <h2>Neue Mitglieder</h2>
 <p>
+{% if perms.member.may_manage_waiting_list %}
 Hier werden neue Mitglieder verwaltet. Um ein neues Mitglied anzulegen, muss sich die Person
 <a href="{% url 'members:register_waiting_list' %}">anmelden</a>. Daraufhin landet
 sie auf der <a href="{% url 'admin:members_memberwaitinglist_changelist' %}">Warteliste</a>. Eine
@@ -57,8 +58,15 @@ Person auf der Warteliste kannst du dann zu einer Schnupperstunde einer gewählt
 Diese Einladung enthält einen Registrierungslink zu einem Formular in dem die Person alle ihre
 Stammdaten eingbit. Diese Daten landen dann unter
 <a href="{% url 'admin:members_memberunconfirmedproxy_changelist' %}">Unbestätigte Registrierungen</a>.
+{% else %}
+Hier werden neue Mitglieder verwaltet. Ob über die Warteliste oder über ein Registrierungspasswort,
+liegt eine neue Registrierung für eine von dir geleitete Jugendgruppe vor, kannst du die hier einsehen
+und die Daten prüfen. Falls die Daten vollständig sind, bestätige die Registrierung um die Person in deine
+Jugendgruppe aufzunehmen.
+{% endif %}
 </p>
 <table>
+    {% if perms.member.may_manage_waiting_list %}
     <tr>
         <th scope="row">
             <a href="{% url 'admin:members_memberwaitinglist_changelist' %}">Warteliste</a>
@@ -66,6 +74,7 @@ Stammdaten eingbit. Diese Daten landen dann unter
         <td></td>
         <td></td>
     </tr>
+    {% endif %}
     <tr>
         <th scope="row">
             <a href="{% url 'admin:members_memberunconfirmedproxy_changelist' %}">Unbestätigte Registrierungen</a>