Package: nm.debian.org Severity: minor Some data belonging to legacy processes is no longer shown on the site
- Mailbox view (https://nm.debian.org/legacy/display-mail-archive/$PNR) doesn't show any mails - On the process page (https://nm.debian.org/legacy/process/$PNR) -- "Personal information" advocates are not shown and neither the former graphic for Mailbox stats. -- "Process log" "Changed by" does not show uids - On pages of Persons and AMs (https://nm.debian.org/person/$uid) Old style process tables are also missing advocates added a patch for the advocates and "Changed by" parts.
>From 9bfbcd607d8e233302d9f0fe20c468117a825afd Mon Sep 17 00:00:00 2001
From: Judit Foglszinger <ur...@debian.org>
Date: Wed, 11 Mar 2020 05:39:13 +0600
Subject: [PATCH] fix some references to uid (now person.ldap_fields.uid)
---
legacy/templates/legacy/process.html | 4 ++--
public/templates/public/person_process_table.html | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/legacy/templates/legacy/process.html b/legacy/templates/legacy/process.html
index 61c63f9..63c171f 100644
--- a/legacy/templates/legacy/process.html
+++ b/legacy/templates/legacy/process.html
@@ -17,7 +17,7 @@
<th>{% trans "Advocate" %}{{process.advocates.count|pluralize}}</th>
<td>
{% for a in process.advocates.all %}
- <a href="{{ a.get_absolute_url }}">{{a.uid}}</a>{% if not forloop.last %},{% endif %}
+ <a href="{{ a.get_absolute_url }}">{{a.person.ldap_fields.uid}}</a>{% if not forloop.last %},{% endif %}
{% endfor %}
</td>
</tr>
@@ -103,7 +103,7 @@
{% endif %}
<td>{{l.logdate|date:"Y-m-d"}}</td>
<td>
- <a href="{{l.changed_by.get_absolute_url}}">{{l.changed_by.uid}}</a>
+ <a href="{{l.changed_by.get_absolute_url}}">{{l.changed_by.person.ldap_fields.uid}}</a>
</td>
<td>{{l.progress|desc_progress}}</td>
{% if visitor.am %}
diff --git a/public/templates/public/person_process_table.html b/public/templates/public/person_process_table.html
index 91a965c..9aa9f30 100644
--- a/public/templates/public/person_process_table.html
+++ b/public/templates/public/person_process_table.html
@@ -30,7 +30,7 @@
</td>
<td>
{% for a in p.advocates.all %}
- <a href="{{ a.get_absolute_url }}" title="{{a.fullname}}">{{a.uid}}</a>{% if not forloop.last %},{% endif %}
+ <a href="{{ a.get_absolute_url }}" title="{{a.person.fullname}}">{{a.person.ldap_fields.uid}}</a>{% if not forloop.last %},{% endif %}
{% endfor %}
</td>
</tr>
--
2.25.1
signature.asc
Description: This is a digitally signed message part.
