From e66e4dcd4691d492594eb169d9f816c302682081 Mon Sep 17 00:00:00 2001
From: FSMPI Admin-Team <admin@fsmpi.rwth-aachen.de>
Date: Sat, 4 Mar 2017 16:13:43 +0100
Subject: [PATCH] Do not require global admin rights for deleting default
 metadata and reminders

---
 server.py       | 1 -
 views/tables.py | 8 +++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/server.py b/server.py
index 0448af7..7f8edb6 100755
--- a/server.py
+++ b/server.py
@@ -281,7 +281,6 @@ def edit_reminder(type_id, reminder_id):
 
 @app.route("/type/reminder/delete/<int:type_id>/<int:reminder_id>")
 @login_required
-@group_required(config.ADMIN_GROUP)
 def delete_reminder(type_id, reminder_id):
     protocoltype = ProtocolType.query.filter_by(id=type_id).first()
     if protocoltype is None:
diff --git a/views/tables.py b/views/tables.py
index c64aa40..9ac653d 100644
--- a/views/tables.py
+++ b/views/tables.py
@@ -209,9 +209,8 @@ class MeetingRemindersTable(Table):
         ]
         action_links = [
             Table.link(url_for("edit_reminder", type_id=self.protocoltype.id, reminder_id=reminder.id), "Ändern"),
+            Table.link(url_for("delete_reminder", type_id=self.protocoltype.id, reminder_id=reminder.id), "Löschen", confirm="Bist du dir sicher, dass du die Einladungsmail {} Tage vor der Sitzung löschen willst?".format(reminder.days_before))
         ]
-        if self.protocoltype.has_admin_right(user):
-            action_links.append(Table.link(url_for("delete_reminder", type_id=self.protocoltype.id, reminder_id=reminder.id), "Löschen", confirm="Bist du dir sicher, dass du die Einladungsmail {} Tage vor der Sitzung löschen willst?".format(reminder.days_before)))
         action_part = [Table.concat(action_links)]
         return general_part + action_part
 
@@ -387,10 +386,9 @@ class DefaultMetasTable(Table):
             meta.key,
         ]
         links = [
-            Table.link(url_for("edit_defaultmeta", meta_id=meta.id), "Ändern")
+            Table.link(url_for("edit_defaultmeta", meta_id=meta.id), "Ändern"),
+            Table.link(url_for("delete_defaultmeta", meta_id=meta.id, confirm="Bist du dir sicher, dass du das Metadatenfeld {} löschen willst?".format(meta.name)), "Löschen")
         ]
-        if meta.protocoltype.has_admin_right(user):
-            links.append(Table.link(url_for("delete_defaultmeta", meta_id=meta.id, confirm="Bist du dir sicher, dass du das Metadatenfeld {} löschen willst?".format(meta.name)), "Löschen"))
         link_part = [Table.concat(links)]
         return general_part + link_part
 
-- 
GitLab