From 73b12dd96b245e210bf500ed89122d2ec9e09d6c Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Mon, 22 Sep 2025 11:29:19 +0100
Subject: [PATCH] Add management links to case details in collection_detail
 template and update question_link_header to include case details

---
 atlas/templates/atlas/collection_detail.html  | 54 ++++++++++---------
 .../templates/atlas/question_link_header.html |  5 +-
 atlas/views.py                                |  3 ++
 3 files changed, 36 insertions(+), 26 deletions(-)

diff --git a/atlas/templates/atlas/collection_detail.html b/atlas/templates/atlas/collection_detail.html
index 37d3483f..6275710f 100644
--- a/atlas/templates/atlas/collection_detail.html
+++ b/atlas/templates/atlas/collection_detail.html
@@ -2,6 +2,34 @@
 
 {% load partials %}
 
+{% partialdef casedetails-management-links %}
+
+                (<a href="{% url 'atlas:collection_case_displaysetup' casedetail.collection.pk casedetail.case.pk %}"><i class="bi bi-display" title="Setup default display"></i></a>
+
+                {% if casedetail.default_viewerstate %}
+                    <i class="bi bi-check text-success" title="This case has a default viewerstate defined"></i>
+                {% endif %}
+                )
+
+                (<a href="{% url 'atlas:collection_case_details' casedetail.collection.pk casedetail.case.pk %}"><i class="bi bi-info-square" title="Case details"></i></a>)
+
+                {% if collection.collection_type == "QUE" %}
+                    (<a href='{% url "atlas:collection_case_questions" casedetail.collection.pk casedetail.case.pk %}'>
+                        {% if casedetail.question_schema %}
+                            <i class="bi bi-question-square text-success" title="This case has questions defined."></i>
+                        {% else %}
+                            <i class="bi bi-question-square text-danger" title="This case has no questions defined."></i>
+                        {% endif %}
+                    </a>
+                    )
+                {% endif %}
+
+                {% if casedetail.case.previous_case %}
+                    (<a href='{% url "atlas:collection_case_priors" casedetail.collection.pk casedetail.case.pk %}'>
+                        <i class="bi bi-link-45deg" title="Manage priors"></i></a>)
+                {% endif %}
+{% endpartialdef %}
+
 {% block content %}
     <h2>{{collection.name}}</h2>
 
@@ -43,31 +71,7 @@
                 {% endif %}
                 {{casedetail.case.title}}
 
-
-                (<a href="{% url 'atlas:collection_case_displaysetup' casedetail.collection.pk casedetail.case.pk %}"><i class="bi bi-display" title="Setup default display"></i></a>
-
-                {% if casedetail.default_viewerstate %}
-                    <i class="bi bi-check text-success" title="This case has a default viewerstate defined"></i>
-                {% endif %}
-                )
-
-                (<a href="{% url 'atlas:collection_case_details' casedetail.collection.pk casedetail.case.pk %}"><i class="bi bi-info-square" title="Case details"></i></a>)
-
-                {% if collection.collection_type == "QUE" %}
-                    (<a href='{% url "atlas:collection_case_questions" casedetail.collection.pk casedetail.case.pk %}'>
-                        {% if casedetail.question_schema %}
-                            <i class="bi bi-question-square text-success" title="This case has questions defined."></i>
-                        {% else %}
-                            <i class="bi bi-question-square text-danger" title="This case has no questions defined."></i>
-                        {% endif %}
-                    </a>
-                    )
-                {% endif %}
-
-                {% if casedetail.case.previous_case %}
-                    (<a href='{% url "atlas:collection_case_priors" casedetail.collection.pk casedetail.case.pk %}'>
-                        <i class="bi bi-link-45deg" title="Manage priors"></i></a>)
-                {% endif %}
+                {% partial casedetails-management-links %}
 
             </li>
 
diff --git a/atlas/templates/atlas/question_link_header.html b/atlas/templates/atlas/question_link_header.html
index 7c84b961..32a4b320 100644
--- a/atlas/templates/atlas/question_link_header.html
+++ b/atlas/templates/atlas/question_link_header.html
@@ -15,8 +15,11 @@
         {% endif %}
 
         {% if collection %}
+            <div style="float: right;">
+                Collection:Case details
+                        {% include "atlas/collection_detail.html#casedetails-management-links" %}
+            </div>
             <div>
-
                 {% if previous %}
                     <a href="{% url 'atlas:collection_case_view' collection.id case_number|add:-1 %}">Previous question</a>
                 {% endif %}
diff --git a/atlas/views.py b/atlas/views.py
index 31cb3cd6..0fb7a76d 100755
--- a/atlas/views.py
+++ b/atlas/views.py
@@ -2853,6 +2853,8 @@ def collection_case_view(request, pk, case_number):
 
     case, case_count = collection.get_case_by_index(case_number, case_count=True)
 
+    casedetail = CaseDetail.objects.get(case=case, collection=collection)
+
     series_list = case.series.all().prefetch_related("images", "examination", "plane")
 
     previous = case_number > 0
@@ -2866,6 +2868,7 @@ def collection_case_view(request, pk, case_number):
             "collection": collection,
             "collection_length": case_count,
             "case": case,
+            "casedetail": casedetail,
             "series_list": series_list,
             "case_number": case_number,
             "previous": previous,