From 2653de6a8541078dc5dc8c4dbac920651c296439 Mon Sep 17 00:00:00 2001
From: Silvan Verhoeven <silvan.verhoeven@student.hpi.de>
Date: Fri, 9 Aug 2024 21:45:40 +0200
Subject: [PATCH] refactor: use attachments.exists

---
 myhpi/core/templates/core/information_page.html |  2 +-
 myhpi/core/templates/core/minutes.html          |  2 +-
 myhpi/core/templates/core/sidebar.html          | 12 +++++++-----
 3 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/myhpi/core/templates/core/information_page.html b/myhpi/core/templates/core/information_page.html
index f50d3dd3..398b34c1 100644
--- a/myhpi/core/templates/core/information_page.html
+++ b/myhpi/core/templates/core/information_page.html
@@ -5,7 +5,7 @@
 
 {% block content %}
     {% with page.body|markdown as parsed_md %}
-        {% include "core/toc_button.html" with toc=parsed_md.1 attachments=page.attachments.all %}
+        {% include "core/toc_button.html" with toc=parsed_md.1 attachments=page.attachments.exists %}
         <div class="row flex-column flex-lg-row">
             <div class="col-lg-9">
                 <h1 class="page-title">
diff --git a/myhpi/core/templates/core/minutes.html b/myhpi/core/templates/core/minutes.html
index c4c6a33b..25d64e2b 100644
--- a/myhpi/core/templates/core/minutes.html
+++ b/myhpi/core/templates/core/minutes.html
@@ -7,7 +7,7 @@
 
 {% block content %}
     {% with page.body|markdown as parsed_md %}
-        {% include "core/toc_button.html" with toc=parsed_md.1 attachments=page.attachments.all %}
+        {% include "core/toc_button.html" with toc=parsed_md.1 attachments=page.attachments.exists %}
         <div class="row flex-column flex-lg-row">
             <h1 class="page-title">
                 <span class="underline">
diff --git a/myhpi/core/templates/core/sidebar.html b/myhpi/core/templates/core/sidebar.html
index 491284b8..eb83ffd8 100644
--- a/myhpi/core/templates/core/sidebar.html
+++ b/myhpi/core/templates/core/sidebar.html
@@ -26,7 +26,7 @@ <h1 class="side-panel-title">{% translate "Table of contents" %}</h1>
         {{ parsed_md.1 }}
     </aside>
     {% endif %}
-    {% if page.attachments.all %}
+    {% if page.attachments.exists %}
         <aside class="side-panel border-accent sidebar-container d-print-none">
             <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
             <ul>
@@ -39,9 +39,11 @@ <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
         </aside>
     {% endif %}
 </div>
-
-{%if parsed_md.1|hasTocContent or page.attachments.all %}
-    <div class="offcanvas offcanvas-bottom" tabindex="-1" id="sidebar-offcanvas" aria-labelledby="sidebar-offcanvas-label">
+{% if parsed_md.1|hasTocContent or page.attachments.exists %}
+    <div class="offcanvas offcanvas-bottom"
+         tabindex="-1"
+         id="sidebar-offcanvas"
+         aria-labelledby="sidebar-offcanvas-label">
         <div class="offcanvas-header">
             <h5 class="offcanvas-title" id="sidebar-offcanvas-label">{% translate "Table of contents" %}</h5>
             <button type="button" class="btn-close" data-bs-dismiss="offcanvas"
@@ -52,7 +54,7 @@ <h5 class="offcanvas-title" id="sidebar-offcanvas-label">{% translate "Table of
                 <h1 class="side-panel-title">{% translate "Table of contents" %}</h1>
                 {{ parsed_md.1 }}
             </aside>
-            {% if page.attachments.all %}
+            {% if page.attachments.exists %}
                 <aside class="side-panel border-accent sidebar-container d-print-none">
                     <h1 class="side-panel-title">{% translate "Attachments" %}</h1>
                     <ul>