From bf519a280712bd1b41ed08f664b46fc0db7163a7 Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Wed, 10 Apr 2024 22:43:57 +0100
Subject: [PATCH] lots of further updates

---
 atlas/admin.py                                |  4 ++
 atlas/api.py                                  |  3 +
 atlas/forms.py                                |  5 +-
 atlas/models.py                               | 10 +++
 atlas/templates/atlas/case_form.html          |  2 +-
 .../templates/atlas/casecollection_form.html  |  2 +-
 atlas/templates/atlas/selfreview_form.html    |  2 +-
 atlas/templates/atlas/series.html             | 11 ++--
 atlas/templates/atlas/series_form.html        | 65 +++++++++++--------
 atlas/templates/atlas/series_headers.html     |  8 +++
 atlas/urls.py                                 |  1 +
 atlas/views.py                                | 11 ++++
 rad/static/css/forms.css                      |  8 +--
 templates/author_form.html                    |  2 +-
 14 files changed, 92 insertions(+), 42 deletions(-)
 create mode 100644 atlas/templates/atlas/series_headers.html

diff --git a/atlas/admin.py b/atlas/admin.py
index a43e60e4..a35f5611 100755
--- a/atlas/admin.py
+++ b/atlas/admin.py
@@ -15,6 +15,8 @@ from .models import (
     PathologicalProcess,
     Presentation,
     UncategorisedDicom,
+    SeriesDetail,
+    Resource
 )
 
 from django.forms import ModelForm
@@ -42,6 +44,8 @@ admin.site.register(PathologicalProcess)
 admin.site.register(Presentation)
 admin.site.register(CaseDetail)
 admin.site.register(UncategorisedDicom)
+admin.site.register(SeriesDetail)
+admin.site.register(Resource)
 
 
 class DifferentialInline(admin.TabularInline):
diff --git a/atlas/api.py b/atlas/api.py
index c2678305..a0d29556 100644
--- a/atlas/api.py
+++ b/atlas/api.py
@@ -23,6 +23,8 @@ from generic.models import Examination, Modality
 
 from .models import Case, DuplicateDicom, Series, SeriesImage, UncategorisedDicom
 
+from loguru import logger
+
 
 router = Router()
 
@@ -52,6 +54,7 @@ class CaseSchema(ModelSchema):
         model_fields = ["id", "title"]
 
 
+@logger.catch()
 @router.post("/upload_dicom", auth=django_auth)
 def upload_dicom(request, files: List[UploadedFile] = File(...)):
     uploaded = []
diff --git a/atlas/forms.py b/atlas/forms.py
index 021695ef..7500f73a 100755
--- a/atlas/forms.py
+++ b/atlas/forms.py
@@ -695,4 +695,7 @@ class CaseCollectionAuthorForm(ExamAuthorFormMixin):
 
 class CaseAuthorForm(ExamAuthorFormMixin):
     class Meta(ExamAuthorFormMixin.Meta):
-        model = Case
\ No newline at end of file
+        model = Case
+class SeriesAuthorForm(ExamAuthorFormMixin):
+    class Meta(ExamAuthorFormMixin.Meta):
+        model = Series
\ No newline at end of file
diff --git a/atlas/models.py b/atlas/models.py
index 9cb3a26b..21abc328 100644
--- a/atlas/models.py
+++ b/atlas/models.py
@@ -69,6 +69,8 @@ from django.contrib.contenttypes.fields import GenericRelation
 
 from django.core.validators import MaxValueValidator, MinValueValidator
 
+from loguru import logger
+
 
 image_storage = FileSystemStorage(
     # Physical file location ROOT
@@ -685,6 +687,13 @@ class Series(SeriesBase):
 
         self.save()
 
+    def get_base_template(self):
+        return "atlas/base.html"
+
+    def get_link_headers(self):
+        return "atlas/series_headers.html"
+
+
 
 class CaseCollection(ExamOrCollectionGenericBase):
     app_name = "atlas"
@@ -1099,6 +1108,7 @@ class UncategorisedDicom(models.Model):
 
         return duplicate
 
+    @logger.catch()
     def save(self, *args, **kwargs):
         """Override save method to add image hash"""
         if self.image:
diff --git a/atlas/templates/atlas/case_form.html b/atlas/templates/atlas/case_form.html
index a1fbeb0a..2d7cd002 100755
--- a/atlas/templates/atlas/case_form.html
+++ b/atlas/templates/atlas/case_form.html
@@ -71,7 +71,7 @@
     <div class="alert alert-info" role="alert">Creating a case in the collection: <a href="{% url 'atlas:collection_detail' pk=form.collection.pk %}">{{form.collection.name}}</a></div>
     {% endif %}
 
-    <form action="" method="post" enctype="multipart/form-data" id="atlas-form">
+    <form class="highlight" action="" method="post" enctype="multipart/form-data" id="atlas-form">
         {% csrf_token %}
 
         {{ form }}
diff --git a/atlas/templates/atlas/casecollection_form.html b/atlas/templates/atlas/casecollection_form.html
index 4b926578..38aecb7f 100755
--- a/atlas/templates/atlas/casecollection_form.html
+++ b/atlas/templates/atlas/casecollection_form.html
@@ -33,7 +33,7 @@
         
     {% endif %}
         
-    <form action="" method="post" enctype="multipart/form-data" id="atlas-form">
+    <form class="highlight" action="" method="post" enctype="multipart/form-data" id="atlas-form">
         {% csrf_token %}
 
         {{ form }}
diff --git a/atlas/templates/atlas/selfreview_form.html b/atlas/templates/atlas/selfreview_form.html
index a5a2d093..1e9aaf2c 100644
--- a/atlas/templates/atlas/selfreview_form.html
+++ b/atlas/templates/atlas/selfreview_form.html
@@ -8,7 +8,7 @@
         Adding self feedback for {{user_exam.exam}}/{{case}}
         
     </div>
-    <form action="" method="post">
+    <form class="highlight" action="" method="post">
         {% csrf_token %}
         {{ form.non_field_errors }}
 
diff --git a/atlas/templates/atlas/series.html b/atlas/templates/atlas/series.html
index 3588a2aa..f91f579e 100755
--- a/atlas/templates/atlas/series.html
+++ b/atlas/templates/atlas/series.html
@@ -1,12 +1,13 @@
 
 {% extends "atlas/base.html" %}
 
+{% block navigation%}
+
+    {{ block.super }}
+    {% include 'atlas/series_headers.html' %}
+{% endblock %}
+
 {% block content %}
-    <a href="{% url 'atlas:series_update' pk=series.pk %}" title="Edit the Series">Edit</a>
-    <a href="{% url 'atlas:series_delete' pk=series.pk %}" title="Delete the Series">Delete</a>
-    {% if request.user.is_superuser %}
-        <a href="{% url 'admin:atlas_series_change' series.id %}" title="Edit the Series using the admin interface">Admin Edit</a>
-    {% endif %}
 
     {% include 'atlas/series_viewer.html' %}
 {% endblock %}
diff --git a/atlas/templates/atlas/series_form.html b/atlas/templates/atlas/series_form.html
index bb008de8..468de11f 100755
--- a/atlas/templates/atlas/series_form.html
+++ b/atlas/templates/atlas/series_form.html
@@ -6,8 +6,8 @@
     <!--<script type="text/javascript" src="/admin/jsi18n/"></script>-->
 
     <script type="text/javascript">
-    let csrf_token = "{{csrf_token}}";
-    let hash_url = false;
+        let csrf_token = "{{csrf_token}}";
+        let hash_url = false;
         document.addEventListener('drop', function (e) { e.preventDefault(); }, false);
 
         $(document).ready(function () {
@@ -332,7 +332,7 @@
                 }
 
                 let select_object = document.getElementsByName("examination")[0]
-                
+
                 if (select_object.value == "") {
                     $.get(`${select_object.dataset.autocompleteLightUrl}?q=${study_description}`, (data) => {
                         console.log(data)
@@ -341,9 +341,9 @@
                             $(select_object).select2("trigger","select", {
                                 data: data.results[0]
                             })
-                            
+
                         }
-                    }) 
+                    })
 
                 }
 
@@ -353,31 +353,31 @@
 
                 for (let i = 0; i < site_codes.length; i++) {
                     if (
-                    accession_number.startsWith(site_codes[i])
-                    || patient_id.startsWith(site_codes[i])) {
-                        not_anon = true;
-                    }
+                        accession_number.startsWith(site_codes[i])
+                        || patient_id.startsWith(site_codes[i])) {
+                            not_anon = true;
+                        }
                 }
 
                 if (not_anon) {
-                toastr.warning(`File does not appear to be annonymised<br/>Accession: ${accession_number} <br/>Patient ID: ${patient_id}<br/>`, 'Anonymisation warning', {
-                            "closeButton": false,
-                            "debug": false,
-                            "newestOnTop": false,
-                            "progressBar": false,
-                            "positionClass": "toast-top-full-width",
-                            "preventDuplicates": false,
-                            "onclick": null,
-                            "showDuration": "0",
-                            "hideDuration": "0",
-                            "timeOut": 0,
-                            "extendedTimeOut": 0,
-                            "showEasing": "swing",
-                            "hideEasing": "linear",
-                            "showMethod": "fadeIn",
-                            "hideMethod": "fadeOut",
-                });
-            }
+                    toastr.warning(`File does not appear to be annonymised<br/>Accession: ${accession_number} <br/>Patient ID: ${patient_id}<br/>`, 'Anonymisation warning', {
+                        "closeButton": false,
+                        "debug": false,
+                        "newestOnTop": false,
+                        "progressBar": false,
+                        "positionClass": "toast-top-full-width",
+                        "preventDuplicates": false,
+                        "onclick": null,
+                        "showDuration": "0",
+                        "hideDuration": "0",
+                        "timeOut": 0,
+                        "extendedTimeOut": 0,
+                        "showEasing": "swing",
+                        "hideEasing": "linear",
+                        "showMethod": "fadeIn",
+                        "hideMethod": "fadeOut",
+                    });
+                }
 
                 // TODO: fix
                 //$.ajax({
@@ -410,14 +410,23 @@
 
     {{ form.media }}
 {% endblock %}
+
+{% block navigation%}
+    {{ block.super }}
+    {% if object %}
+        {% include "atlas/series_headers.html" %}
+    {% endif %}
+{% endblock %}
 {% block content %}
+
+
     <h2>Series Form</h2>
     This form will create a image set that can be associated with a atlas question. If the question has already been created
     it can be linked below.
     <form action="" method="post" enctype="multipart/form-data" id="atlas-form">
         {% csrf_token %}
         <a href="/generic/examination/create" id="add_examination" class="add-popup"
-            onclick="return showAddPopup(this);"><img src="{% static '/img/icon-addlink.svg' %}"></a>
+           onclick="return showAddPopup(this);"><img src="{% static '/img/icon-addlink.svg' %}"></a>
         <table>
             {{ form.as_table }}
         </table>
diff --git a/atlas/templates/atlas/series_headers.html b/atlas/templates/atlas/series_headers.html
new file mode 100644
index 00000000..7bdab267
--- /dev/null
+++ b/atlas/templates/atlas/series_headers.html
@@ -0,0 +1,8 @@
+<br/>
+<a href="{% url 'atlas:series_detail' pk=series.pk %}" title="View the Series">View</a>
+<a href="{% url 'atlas:series_update' pk=series.pk %}" title="Edit the Series">Edit</a>
+<a href="{% url 'atlas:series_delete' pk=series.pk %}" title="Delete the Series">Delete</a>
+<a href="{% url 'atlas:series_authors' pk=series.pk %}" title="Update the authors">Authors</a>
+{% if request.user.is_superuser %}
+    <a href="{% url 'admin:atlas_series_change' series.id %}" title="Edit the Series using the admin interface">Admin Edit</a>
+{% endif %}
\ No newline at end of file
diff --git a/atlas/urls.py b/atlas/urls.py
index bc856648..6fd077d5 100755
--- a/atlas/urls.py
+++ b/atlas/urls.py
@@ -242,6 +242,7 @@ urlpatterns = [
     ),
     path("structure/create", views.StructureCreate.as_view(), name="structure_create"),
     path("series/<int:pk>", views.series_detail, name="series_detail"),
+    path("series/<int:pk>/authors", views.SeriesAuthorUpdate.as_view(), name="series_authors"),
     path("series/", views.SeriesView.as_view(), name="series_view"),
     path("series_image/<int:pk>/dicom", views.series_image_dicom, name="series_image_dicom"),
     path(
diff --git a/atlas/views.py b/atlas/views.py
index 519b5e2d..a9228ba4 100755
--- a/atlas/views.py
+++ b/atlas/views.py
@@ -48,6 +48,7 @@ from .forms import (
     FindingForm,
     ResourceForm,
     SelfReviewForm,
+    SeriesAuthorForm,
     SeriesForm,
     SeriesImageFormSet,
     SeriesFormSet,
@@ -2387,3 +2388,13 @@ class CaseAuthorUpdate(RevisionMixin, AuthorRequiredMixin, UpdateView):
         context = super(CaseAuthorUpdate, self).get_context_data(**kwargs)
         context["collection"] = context["object"]
         return context
+
+class SeriesAuthorUpdate(RevisionMixin, AuthorRequiredMixin, UpdateView):
+    model = Series
+    form_class = SeriesAuthorForm
+    template_name = "author_form.html"
+
+    def get_context_data(self, **kwargs):
+        context = super(SeriesAuthorUpdate, self).get_context_data(**kwargs)
+        context["collection"] = context["object"]
+        return context
diff --git a/rad/static/css/forms.css b/rad/static/css/forms.css
index 59e95467..8bf95876 100644
--- a/rad/static/css/forms.css
+++ b/rad/static/css/forms.css
@@ -8,14 +8,14 @@ form>div:hover .helptext {
   color: #666;
   opacity: 1;
 }
-
 form > div {
   margin: 1px;
 }
 
-form > div:hover {
-  border: 1px solid #666;
-  margin: -1px;
+form.highlight > div:hover {
+  /* border: 1px solid #666;
+  margin: -1px; */
+  background-color: #002635;
 }
 
 
diff --git a/templates/author_form.html b/templates/author_form.html
index 9a20eb76..7b92080a 100644
--- a/templates/author_form.html
+++ b/templates/author_form.html
@@ -7,7 +7,7 @@
 
 
 {% block navigation %}
-{{ block.super }}
+    {{ block.super }}
     {% include object.get_link_headers %}
 {% endblock navigation %}