Use clearer labels for structure membership

Following cf47b66, use the helper for
more memberships.

itou/approvals/admin.py

@@ -20,7 +20,7 @@
     ItouTabularInline,
     PkSupportRemarkInline,
     get_admin_view_link,
-    get_company_view_link,
+    get_structure_view_link,
 )
 from itou.utils.templatetags.str_filters import pluralizefr
 
@@ -60,7 +60,7 @@ def has_add_permission(self, request, obj=None):
 
     @admin.display(description="Entreprise destinataire")
     def to_company_link(self, obj):
-        return get_company_view_link(obj.to_company)
+        return get_structure_view_link(obj.to_company, display_attr="display_name")
 
     # Custom read-only fields as workaround :
     # there is no direct relation between approvals and employee records

itou/users/admin.py

@@ -37,7 +37,7 @@
     PkSupportRemarkInline,
     add_support_remark_to_obj,
     get_admin_view_link,
-    get_company_view_link,
+    get_structure_view_link,
 )
 
 
@@ -106,7 +106,7 @@ def has_add_permission(self, request, obj=None):
 
     @admin.display(description="Entreprise")
     def company_siret_link(self, obj):
-        return get_company_view_link(obj.company)
+        return get_structure_view_link(obj.company)
 
 
 class PrescriberMembershipInline(DisabledNotificationsMixin, ItouTabularInline):
@@ -134,7 +134,7 @@ def has_add_permission(self, request, obj=None):
         return True
 
     def organization_id_link(self, obj):
-        return get_admin_view_link(obj.organization)
+        return get_structure_view_link(obj.organization)
 
 
 class InstitutionMembershipInline(ItouTabularInline):
@@ -165,7 +165,7 @@ def has_add_permission(self, request, obj=None):
         return True
 
     def institution_id_link(self, obj):
-        return get_admin_view_link(obj.institution)
+        return get_structure_view_link(obj.institution)
 
 
 class JobApplicationInline(ItouTabularInline):

itou/utils/admin.py

@@ -20,13 +20,15 @@ def get_admin_view_link(obj, *, content=None, view="change"):
     return format_html('<a href="{}">{}</a>', url, content or obj.pk)
 
 
-def get_company_view_link(company):
-    return format_html(
-        "{} — SIRET : {} ({})",
-        get_admin_view_link(company, content=company.display_name),
-        company.siret,
-        company.kind,
-    )
+def get_structure_view_link(structure, display_attr="name"):
+    format_string = "{link}"
+    format_kwargs = {"link": get_admin_view_link(structure, content=getattr(structure, display_attr))}
+    if hasattr(structure, "siret"):
+        format_string += " — SIRET {siret}"
+        format_kwargs["siret"] = structure.siret
+    format_string += " ({kind})"
+    format_kwargs["kind"] = structure.kind
+    return format_html(format_string, **format_kwargs)
 
 
 class AbstractSupportRemarkInline(GenericStackedInline):