From 5d9b8f32139ef9196fd28e97ca9c835e5612aa74 Mon Sep 17 00:00:00 2001
From: moanos <julian-samuel@gebuehr.net>
Date: Sun, 13 Jul 2025 09:12:24 +0200
Subject: [PATCH] feat: Re-add functionality to set AN as checked

---
 .../details/detail-adoption-notice.html       | 12 ++++--
 src/fellchensammlung/views.py                 | 37 +++++++++++++------
 2 files changed, 34 insertions(+), 15 deletions(-)

diff --git a/src/fellchensammlung/templates/fellchensammlung/details/detail-adoption-notice.html b/src/fellchensammlung/templates/fellchensammlung/details/detail-adoption-notice.html
index 0bdfae0..88123db 100644
--- a/src/fellchensammlung/templates/fellchensammlung/details/detail-adoption-notice.html
+++ b/src/fellchensammlung/templates/fellchensammlung/details/detail-adoption-notice.html
@@ -76,10 +76,14 @@
 
                                     {% if has_edit_permission %}
 
-                                        <a class="dropdown-item">
-                                            <i class="fas fa-check fa-fw"
-                                               aria-hidden="true"></i> {% trans 'Alsalhost:8000/ aktiv bestätigen' %}
-                                        </a>
+                                        <form class="dropdown-item" method="POST">
+                                            {% csrf_token %}
+                                            <input type="hidden" name="action" value="checked_active">
+                                            <button type="submit" id="submit">
+                                                <i class="fas fa-check fa-fw"
+                                                   aria-hidden="true"></i> {% trans 'Als aktiv bestätigen' %}
+                                            </button>
+                                        </form>
                                         <a class="dropdown-item"
                                            href="{% url 'adoption-notice-edit' adoption_notice_id=adoption_notice.pk %}">
                                             <i class="fas fa-pencil fa-fw"
diff --git a/src/fellchensammlung/views.py b/src/fellchensammlung/views.py
index 93dbfbc..b05ab9d 100644
--- a/src/fellchensammlung/views.py
+++ b/src/fellchensammlung/views.py
@@ -85,6 +85,24 @@ def change_language(request):
         return render(request, "fellchensammlung/errors/403.html", status=403)
 
 
+def handle_an_check_actions(request, action, adoption_notice=None):
+    if adoption_notice is None:
+        adoption_notice = AdoptionNotice.objects.get(id=request.POST.get("adoption_notice_id"))
+    edit_permission = request.user == adoption_notice.owner or user_is_trust_level_or_above(request.user,
+                                                                                            TrustLevel.MODERATOR)
+
+    # Check if the user is permitted to perform the actions
+    if action in ("checked_inactive", "checked_active") and not request.user.is_authenticated or not edit_permission:
+        return render(request, "fellchensammlung/errors/403.html", status=403)
+
+    if action == "checked_inactive":
+        adoption_notice.set_closed()
+    elif action == "checked_active":
+        print("dads")
+        adoption_notice.set_active()
+    return None
+
+
 def adoption_notice_detail(request, adoption_notice_id):
     adoption_notice = AdoptionNotice.objects.get(id=adoption_notice_id)
     if request.user.is_authenticated:
@@ -98,6 +116,7 @@ def adoption_notice_detail(request, adoption_notice_id):
     has_edit_permission = user_is_owner_or_trust_level(request.user, adoption_notice)
     if request.method == 'POST':
         action = request.POST.get("action")
+        handle_an_check_actions(request, action, adoption_notice)
         if request.user.is_authenticated:
             if action == "comment":
                 comment_form = CommentForm(request.POST)
@@ -601,9 +620,10 @@ def my_notifications(request):
         action = request.POST.get("action")
         process_notification_actions(request, action)
 
-    context = {"notifications_unread": Notification.objects.filter(user_to_notify=request.user, read=False).order_by("-created_at"),
+    context = {"notifications_unread": Notification.objects.filter(user_to_notify=request.user, read=False).order_by(
+        "-created_at"),
                "notifications_read_last": Notification.objects.filter(user_to_notify=request.user,
-                                                                      read=True).order_by("-read_at") }
+                                                                      read=True).order_by("-read_at")}
     return render(request, 'fellchensammlung/notifications.html', context=context)
 
 
@@ -617,16 +637,11 @@ def modqueue(request):
 @login_required
 def updatequeue(request):
     if request.method == "POST":
-        adoption_notice = AdoptionNotice.objects.get(id=request.POST.get("adoption_notice_id"))
-        edit_permission = request.user == adoption_notice.owner or user_is_trust_level_or_above(request.user,
-                                                                                                TrustLevel.MODERATOR)
-        if not edit_permission:
-            return render(request, "fellchensammlung/errors/403.html", status=403)
         action = request.POST.get("action")
-        if action == "checked_inactive":
-            adoption_notice.set_closed()
-        if action == "checked_active":
-            adoption_notice.set_active()
+
+        # This function handles the activation and deactivation of ANs
+        # Separate function because it's used in multiple places
+        handle_an_check_actions(request, action)
 
     if user_is_trust_level_or_above(request.user, TrustLevel.MODERATOR):
         last_checked_adoption_list = AdoptionNotice.objects.order_by("last_checked")