Merge pull request #3513 from BirkbeckCTP/3502-editor_doi_options

#3502 DOI edit and add buttons will no longer be shown to authors on the article metadata page. Text on the identifiers section has also been updated
openlibhums · May 18, 2023 · bf2cbf6 · bf2cbf6
2 parents a78300d + 0289b4f
commit bf2cbf6
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/templates/admin/elements/metadata.html b/src/templates/admin/elements/metadata.html
@@ -239,7 +239,7 @@ <h2>Projected Issue</h2>
 
 <div class="title-area">
     <h2>Identifiers</h2>
-    <a class="button" href="{% url 'edit_identifiers' article.pk %}" target="_blank"><i class="fa fa-edit">&nbsp;</i>Edit</a>
+    {% if user_is_editor %}<a class="button" href="{% url 'edit_identifiers' article.pk %}" target="_blank"><i class="fa fa-cogs">&nbsp;</i>Manage Identifiers</a>{% endif %}
 </div>
 {% setting_var request.journal 'use_crossref' as use_crossref %}
 <div class="content">
@@ -264,21 +264,24 @@ <h2>Identifiers</h2>
                         {% for identifier in article.identifiers %}
                             {% if identifier.identifier == doi %}
                                 {% if identifier.deposit %}
-                                    <a href="{% url 'poll_doi_output' article.pk identifier.pk %}">
+                                    {% if user_is_editor %}<a href="{% url 'poll_doi_output' article.pk identifier.pk %}">{% endif %}
                                     {% if identifier.deposit.success %}
                                         <i class="fa fa-check">&nbsp;</i>
                                     {% else %}
                                         <i class="fa fa-exclamation-circle">&nbsp;</i>
                                     {% endif %}
-                                    </a>
+                                    {% if user_is_editor %}</a>{% endif %}
+                                {% else %}
+                                    <i class="fa fa-exclamation-circle">&nbsp;</i> This DOI has not been deposited.
                                 {% endif %}
                             {% endif %}
                         {% endfor %}
                     </td>
                 {% else %}
+
                     <td>DOI</td>
                     <td><span class="fa fa-exclamation-triangle"> Article has no DOI</span></td>
-                    <td><a href="{% url 'add_new_identifier' article.pk %}" class="button">Add DOI</a></td>
+                    <td>{% if user_is_editor %}<a href="{% url 'add_new_identifier' article.pk %}" class="button">Add DOI</a>{% else %}---{% endif %}</td>
                 {% endif %}
                 {% endwith %}
             </tr>