Merge branch 'non-BDE-members' into 'main'

Allow non-BDE members to use the note

See merge request bde/nk20!268

apps/permission/fixtures/initial.json

@@ -3752,6 +3752,54 @@
             "description": "Modifier bouffe"
         }
     },
+    {
+        "model": "permission.permission",
+        "pk": 239,
+        "fields": {
+            "model": [
+                "note",
+                "alias"
+            ],
+            "query": "[\"AND\", {\"note__noteuser__user__memberships__club\": [\"club\"], \"note__noteuser__user__memberships__date_start__lte\": [\"today\"], \"note__noteuser__user__memberships__date_end__gte\": [\"today\"]}, {\"note__is_active\": true}]",
+            "type": "view",
+            "mask": 2,
+            "field": "",
+            "permanent": false,
+            "description": "Voir les alias des notes des adhérent⋅es du club"
+        }
+    },
+    {
+        "model": "permission.permission",
+        "pk": 240,
+        "fields": {
+            "model": [
+                "note",
+                "alias"
+            ],
+            "query": "[\"AND\", {\"note__noteuser__user__memberships__club\": [\"club\", \"parent_club\"], \"note__noteuser__user__memberships__date_start__lte\": [\"today\"], \"note__noteuser__user__memberships__date_end__gte\": [\"today\"]}, {\"note__is_active\": true}]",
+            "type": "view",
+            "mask": 2,
+            "field": "",
+            "permanent": false,
+            "description": "Voir les alias des notes des adhérent⋅es du club parent"
+        }
+    },
+    {
+        "model": "permission.permission",
+        "pk": 241,
+        "fields": {
+            "model": [
+                "auth",
+                "user"
+            ],
+            "query": "[\"AND\", {\"memberships__club\": [\"club\", \"parent_club\"], \"memberships__date_start__lte\": [\"today\"], \"memberships__date_end__gte\": [\"today\"]}, {\"note__is_active\": true}]",
+            "type": "view",
+            "mask": 2,
+            "field": "",
+            "permanent": false,
+            "description": "Voir les utilisateurs adhérents au club parent"
+        }
+    },
     {
         "model": "permission.role",
         "pk": 1,
@@ -3851,6 +3899,19 @@
             "for_club": null,
             "name": "Membre de club",
             "permissions": [
+                1,
+                2,
+                3,
+                4,
+                5,
+                7,
+                8,
+                9,
+                10,
+                11,
+                12,
+                13,
+                14,
                 22
             ]
         }
@@ -3913,7 +3974,10 @@
                 142,
                 182,
                 184,
-				185
+                185,
+                239,
+                240,
+                241
             ]
         }
     },

apps/registration/views.py

@@ -300,9 +300,13 @@ class FutureUserDetailView(ProtectQuerysetMixin, LoginRequiredMixin, FormMixin,
 #            join_bde = True
 #            join_kfet = True
 
-        if not join_bde:
+        if not (join_bde or any(b for _, b in join_clubs)):
             # This software belongs to the BDE.
-            form.add_error('join_bde', _("You must join the BDE."))
+            form.add_error('join_bde', _("You must join a club."))
+            return super().form_invalid(form)
+
+        if join_kfet and not join_bde:
+            form.add_error('join_bde', _("You must also join the parent club BDE."))
             return super().form_invalid(form)
 
         # Calculate required registration fee

note_kfet/templates/base.html

@@ -72,7 +72,7 @@ SPDX-License-Identifier: GPL-3.0-or-later
                             <a class="nav-link {% if request.path_info == url %}active{% endif %}" href="{{ url }}"><i class="fa fa-cutlery"></i> {% trans 'Food' %}</a>
                         </li>
                     {% endif %}
-                    {% if user.is_authenticated and user|is_member:"Kfet" %}
+                    {% if user.is_authenticated %}
                         <li class="nav-item">
                             {% url 'note:transfer' as url %}
                             <a class="nav-link {% if request.path_info == url %}active{% endif %}" href="{{ url }}"><i class="fa fa-exchange"></i> {% trans 'Transfer' %}</a>