From 7d389f9dcf1c43e3cf6f83ec8ce785f726d6de46 Mon Sep 17 00:00:00 2001
From: "J.-S. Caux" <J.S.Caux@uva.nl>
Date: Sun, 27 Jun 2021 06:58:36 +0200
Subject: [PATCH] Polish up internal plagiarism check

---
 .../submissions/admin/plagiarism_internal_check.html     | 9 +++++----
 .../submissions/admin/submission_prescreening.html       | 3 ++-
 scipost_django/submissions/views.py                      | 1 -
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/scipost_django/submissions/templates/submissions/admin/plagiarism_internal_check.html b/scipost_django/submissions/templates/submissions/admin/plagiarism_internal_check.html
index 699c5bb2c..b73c9ef81 100644
--- a/scipost_django/submissions/templates/submissions/admin/plagiarism_internal_check.html
+++ b/scipost_django/submissions/templates/submissions/admin/plagiarism_internal_check.html
@@ -1,16 +1,17 @@
 {% extends 'submissions/admin/base.html' %}
 
-{% block pagetitle %}: plagiarism internal check ({{ submission.preprint.identifier_w_vn_nr }}){% endblock pagetitle %}
+{% block pagetitle %}: plagiarism (internal check) ({{ submission.preprint.identifier_w_vn_nr }}){% endblock pagetitle %}
 
 {% block breadcrumb_items %}
   {{ block.super }}
   <span class="breadcrumb-item"><a href="{% url 'submissions:do_prescreening' submission.preprint.identifier_w_vn_nr %}">Pre-screening {{ submission.preprint.identifier_w_vn_nr }}</a></span>
-  <span class="breadcrumb-item">Plagiarism internal check ({{ submission.preprint.identifier_w_vn_nr }})</span>
+  <span class="breadcrumb-item">Plagiarism (internal check) {{ submission.preprint.identifier_w_vn_nr }}</span>
 {% endblock %}
 
 {% block content %}
-  <h1>Plagiarism internal check for <a href="{{ submission.get_absolute_url }}">{{ submission.preprint.identifier_w_vn_nr }}</a></h1>
-  <h2>{{  submission.title }}</h2>
+  <h1>Plagiarism (internal check)</h1>
+  <h2><a href="{{ submission.get_absolute_url }}">{{ submission.preprint.identifier_w_vn_nr }}</a></h2>
+  <h3>{{  submission.title }}</h3>
   <h3 class="mb-4">by {{ submission.author_list }}</h3>
 
   <h1 class="highlight">Submission matches</h1>
diff --git a/scipost_django/submissions/templates/submissions/admin/submission_prescreening.html b/scipost_django/submissions/templates/submissions/admin/submission_prescreening.html
index 1eb5a2398..1d178f958 100644
--- a/scipost_django/submissions/templates/submissions/admin/submission_prescreening.html
+++ b/scipost_django/submissions/templates/submissions/admin/submission_prescreening.html
@@ -30,6 +30,7 @@
       <a href="{% url 'colleges:submission' submission.preprint.identifier_w_vn_nr %}">Manage this submission's Fellowship ({{ submission.fellows.count }} fellows)</a><br>
     </li>
     <li>
+      <span class="text-success">{% include 'bi/arrow-right.html' %}</span>
       <a href="{% url 'submissions:plagiarism_internal' submission.preprint.identifier_w_vn_nr %}">Check internal plagiarism</a>
     </li>
     <li>
@@ -60,7 +61,7 @@
         </table>
       {% else %}
         {% include 'bi/square.html' %}
-        <a href="{% url 'submissions:plagiarism' submission.preprint.identifier_w_vn_nr %}">Run plagiarism check</a>
+        <a href="{% url 'submissions:plagiarism' submission.preprint.identifier_w_vn_nr %}">Run iThenticate plagiarism check</a>
       {% endif %}
     </li>
     <li>
diff --git a/scipost_django/submissions/views.py b/scipost_django/submissions/views.py
index 358e3ce15..7a9a65729 100644
--- a/scipost_django/submissions/views.py
+++ b/scipost_django/submissions/views.py
@@ -2344,7 +2344,6 @@ class PlagiarismInternalView(
     permission_required = 'scipost.can_run_pre_screening'
     template_name = 'submissions/admin/plagiarism_internal_check.html'
     editorial_page = True
-    success_url = reverse_lazy('submissions:pool')
 
     def get_context_data(self, *args, **kwargs):
         context = super().get_context_data(*args, **kwargs)
-- 
GitLab