prefer contrib.creator.display_name over contrib.raw_name

These will be getting updates from ORCID and are usually more complete
and more correct for display, attribution, and search purposes.

Might need to tweak fuzzycat code to handle multiple names at the
verification stage.

1 files changed, 8 insertions, 3 deletions

diff --git a/python/fatcat_web/templates/release_view_contribs.html b/python/fatcat_web/templates/release_view_contribs.html
index 55cdf133..f4969e32 100644
--- a/python/fatcat_web/templates/release_view_contribs.html
+++ b/python/fatcat_web/templates/release_view_contribs.html
@@ -12,12 +12,17 @@
 {% for contrib in release.contribs %}
   <tr><td class="collapsing">{% if contrib.index or contrib.index == 0 %} {{ contrib.index + 1 }}{% endif %}
       <td><span itemprop="author">
-      {% if contrib.creator_id %}
-        <a href="/creator/{{contrib.creator_id}}">{{ contrib.raw_name or 'unknown' }}</a> 
+      {% if contrib.creator_id and contrib.creator and contrib.creator.display_name %}
+        <a href="/creator/{{contrib.creator_id}}">{{ contrib.creator.display_name }}</a>
       {% else %}
         {{ contrib.raw_name or '' }}
       {% endif %}
-      {% if contrib.surname %}
+
+      {% if contrib.creator and contrib.creator.surname %}
+        <code>&nbsp;
+          ({{ contrib.creator.surname }}{% if contrib.creator.given_name %}, {{ contrib.creator.given_name }}{% endif %})
+        </code>
+      {% elif contrib.surname %}
         <code>&nbsp;
           ({{ contrib.surname }}{% if contrib.given_name %}, {{ contrib.given_name }}{% endif %})
         </code>