From faacf18d81db171cf5e7a4dc2fd5e6f0199410fc Mon Sep 17 00:00:00 2001
From: Leon Sandøy <leon.sandoy@conmodo.com>
Date: Tue, 13 Mar 2018 00:22:22 +0100
Subject: Minor bugfix to allow tag.delete to return success as False if the
 tag doesn't exist

---
 pysite/views/api/bot/tags.py | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'pysite/views/api/bot')

diff --git a/pysite/views/api/bot/tags.py b/pysite/views/api/bot/tags.py
index 9db926f4..241f7686 100644
--- a/pysite/views/api/bot/tags.py
+++ b/pysite/views/api/bot/tags.py
@@ -107,10 +107,13 @@ class TagsView(APIView, DBMixin):
 
         json = data[0]
         tag_name = json.get("tag_name")
+        tag_exists = self.db.get(self.table_name, tag_name)
 
-        self.db.delete(
-            self.table_name,
-            tag_name
-        )
+        if tag_exists:
+            self.db.delete(
+                self.table_name,
+                tag_name
+            )
+            return jsonify({"success": True})
 
-        return jsonify({"success": True})
+        return jsonify({"success": False})
-- 
cgit v1.2.3