From cd22ca6c4486e21a2dac77f8e3c7422f8b13c4e9 Mon Sep 17 00:00:00 2001
From: patrick <patrick@fet.at>
Date: Wed, 26 May 2021 02:51:15 +0000
Subject: [PATCH] update admin action 'make fetmeeting'

---
 fet2020/posts/admin.py | 45 ++++++++++++++++++------------------------
 1 file changed, 19 insertions(+), 26 deletions(-)

diff --git a/fet2020/posts/admin.py b/fet2020/posts/admin.py
index d9431dc7..c295480d 100644
--- a/fet2020/posts/admin.py
+++ b/fet2020/posts/admin.py
@@ -15,44 +15,37 @@ admin.site.unregister(taggit.models.Tag)
 def make_fetmeeting(self, request, queryset):
     qs = self.get_queryset(request).filter(id=request.POST["_selected_action"]).first()
 
-    failed = False
-    agenda_key = None
-    protocol_key = None
-
-    try:
-        agenda_key = createPadifNotExists(qs.slug + "-agenda")
-    except Exception as e:
+    agenda_key = createPadifNotExists(qs.slug + "-agenda")
+    if not agenda_key:
         self.message_user(
             request,
             _("Das Agenda konnte nicht erstellt werden. Error: %s") % str(e),
             messages.ERROR,
         )
-        failed = True
+        return
 
-    try:
-        protocol_key = createPadifNotExists(qs.slug + "-protocol")
-    except Exception as e:
+    protocol_key = createPadifNotExists(qs.slug + "-protocol")
+    if not protocol_key:
         self.message_user(
             request,
             _("Das Protokoll konnte nicht erstellt werden. Error: %s") % str(e),
             messages.ERROR,
         )
-        failed = True
+        return
 
-    if not failed:
-        queryset.update(
-            post_type="F",
-            has_agenda=True,
-            has_protocol=True,
-            agenda_key=agenda_key,
-            protocol_key=protocol_key,
-        )
-        self.message_user(
-            request,
-            _("Das Event %s wurde erfolgreich in eine FET Sitzung konvertiert.")
-            % (qs.title),
-            messages.SUCCESS,
-        )
+    queryset.update(
+        post_type="F",
+        has_agenda=True,
+        has_protocol=True,
+        agenda_key=agenda_key,
+        protocol_key=protocol_key,
+    )
+    self.message_user(
+        request,
+        _("Das Event %s wurde erfolgreich in eine FET Sitzung konvertiert.")
+        % (qs.title),
+        messages.SUCCESS,
+    )
 
 
 make_fetmeeting.short_description = "In eine Fachschaftssitzung konvertieren"