fix: N+1 queries on old club members view

club/templates/club/club_old_members.jinja

@@ -5,20 +5,22 @@
   <h2>{% trans %}Club old members{% endtrans %}</h2>
   <table>
     <thead>
+      <tr>
         <td>{% trans %}User{% endtrans %}</td>
         <td>{% trans %}Role{% endtrans %}</td>
         <td>{% trans %}Description{% endtrans %}</td>
         <td>{% trans %}From{% endtrans %}</td>
         <td>{% trans %}To{% endtrans %}</td>
+      </tr>
     </thead>
     <tbody>
-      {% for m in club.members.exclude(end_date=None).order_by('-role', 'description', '-end_date').all() %}
+      {% for member in old_members %}
         <tr>
-          <td>{{ user_profile_link(m.user) }}</td>
+          <td>{{ user_profile_link(member.user) }}</td>
-          <td>{{ settings.SITH_CLUB_ROLES[m.role] }}</td>
+          <td>{{ settings.SITH_CLUB_ROLES[member.role] }}</td>
-          <td>{{ m.description }}</td>
+          <td>{{ member.description }}</td>
-          <td>{{ m.start_date }}</td>
+          <td>{{ member.start_date }}</td>
-          <td>{{ m.end_date }}</td>
+          <td>{{ member.end_date }}</td>
         </tr>
       {% endfor %}
     </tbody>

club/views.py

@@ -342,6 +342,15 @@ class ClubOldMembersView(ClubTabsMixin, PermissionRequiredMixin, DetailView):
     current_tab = "elderlies"
     permission_required = "club.view_club"
 
+    def get_context_data(self, **kwargs):
+        return super().get_context_data(**kwargs) | {
+            "old_members": (
+                self.object.members.exclude(end_date=None)
+                .order_by("-role", "description", "-end_date")
+                .select_related("user")
+            )
+        }
+
 
 class ClubSellingView(ClubTabsMixin, CanEditMixin, DetailFormView):
     """Sellings of a club."""