From e2e2d1969c94e36bd47a92a16beb840608e306a9 Mon Sep 17 00:00:00 2001
From: florimondmanca <florimond.manca@gmail.com>
Date: Sun, 11 Nov 2018 09:08:07 +0100
Subject: [PATCH] refactor participation user link display field

---
 visits/admin.py | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/visits/admin.py b/visits/admin.py
index 2b0fcb5..20c48dd 100644
--- a/visits/admin.py
+++ b/visits/admin.py
@@ -113,14 +113,16 @@ reject_selected_participations.short_description = (
 class ParticipationAdmin(admin.ModelAdmin):
     """Admin panel for visit participations."""
 
-    list_display = ('submitted', 'visit', 'user_link',
-                    'accepted', 'present')
+    list_display = ('submitted', 'visit', 'user_link', 'accepted', 'present')
     list_filter = ('submitted', 'accepted', 'present')
     actions = [accept_selected_participations, reject_selected_participations]
-    def user_link (self,book):
-        url=reverse("admin:users_user_change", args=[book.user.id]) 
-        link = '<a href="%s">%s</a>' % (url, book.user.email)
+
+    def user_link(self, participation: Participation):
+        """Return a link to the participation's user."""
+        url = reverse("admin:users_user_change", args=[participation.user.id])
+        link = f'<a href="{url}">{participation.user}</a>'
         return mark_safe(link)
+
     user_link.short_description = 'Utilisateur'
 
 
-- 
GitLab