Add confirmation before deletion (#289)

AIPscan/Aggregator/templates/confirm.html

@@ -0,0 +1,8 @@
+{% extends "base.html" %}
+
+{% block content %}
+<form method="GET" action="{{ request.url }}">
+  <input type="hidden" name="confirm" value="1" />
+  <input type="submit" value="{{ action }}" class='btn btn-danger' /> <a href="{{ url_for(cancel_route) }}" class='btn btn-default'>Cancel</a>
+</form>
+{% endblock %}

AIPscan/Aggregator/views.py

@@ -14,6 +14,7 @@
     request,
     url_for,
 )
+from functools import wraps
 
 from AIPscan import db, typesense_helpers
 from AIPscan.Aggregator import database_helpers, tasks
@@ -69,6 +70,21 @@ def _test_storage_service_connection(storage_service):
         raise ConnectionError(str(err))
 
 
+def confirm_required(action, cancel_route):
+    def decorator(f):
+        @wraps(f)
+        def decorated_function(*args, **kwargs):
+            if request.args.get("confirm"):
+                return f(*args, **kwargs)
+
+            return render_template(
+                "confirm.html",
+                action=action,
+                cancel_route=cancel_route,
+            )
+        return decorated_function
+    return decorator
+
 @aggregator.route("/", methods=["GET"])
 def ss_default():
     # load the default storage service
@@ -171,6 +187,7 @@ def new_storage_service():
 
 
 @aggregator.route("/delete_storage_service/<storage_service_id>", methods=["GET"])
+@confirm_required("Delete storage service", "aggregator.ss_default")
 def delete_storage_service(storage_service_id):
     storage_service = StorageService.query.get(storage_service_id)
 
@@ -241,6 +258,7 @@ def new_fetch_job(fetch_job_id):
 
 
 @aggregator.route("/delete_fetch_job/<fetch_job_id>", methods=["GET"])
+@confirm_required("Delete fetch job", "aggregator.ss_default")
 def delete_fetch_job(fetch_job_id):
     fetch_job = FetchJob.query.get(fetch_job_id)