Split views into GET and POST

app/main/views/templates.py

@@ -383,26 +383,10 @@ def delete_service_template(service_id, template_id):
     )
 
 
-@main.route("/services/<service_id>/templates/<template_id>/redact", methods=['GET', 'POST'])
+@main.route("/services/<service_id>/templates/<template_id>/redact", methods=['GET'])
 @login_required
 @user_has_permissions('manage_templates', admin_override=True)
-def redact_template(service_id, template_id):
+def confirm_redact_template(service_id, template_id):
-
-    if request.method == 'POST':
-
-        service_api_client.redact_service_template(service_id, template_id)
-
-        flash(
-            'Personalised content will be hidden for messages sent with this template',
-            'default_with_tick'
-        )
-
-        return redirect(url_for(
-            '.view_template',
-            service_id=service_id,
-            template_id=template_id,
-        ))
-
     return render_template(
         'views/templates/template.html',
         template=get_template(
@@ -421,6 +405,25 @@ def redact_template(service_id, template_id):
     )
 
 
+@main.route("/services/<service_id>/templates/<template_id>/redact", methods=['POST'])
+@login_required
+@user_has_permissions('manage_templates', admin_override=True)
+def redact_template(service_id, template_id):
+
+    service_api_client.redact_service_template(service_id, template_id)
+
+    flash(
+        'Personalised content will be hidden for messages sent with this template',
+        'default_with_tick'
+    )
+
+    return redirect(url_for(
+        '.view_template',
+        service_id=service_id,
+        template_id=template_id,
+    ))
+
+
 @main.route('/services/<service_id>/templates/<template_id>/versions')
 @login_required
 @user_has_permissions(

app/templates/views/templates/template.html

@@ -56,7 +56,7 @@
        
       {% if not template.redact_personalisation %}
         <span class="page-footer-delete-link page-footer-delete-link-without-button">
-          <a href="{{ url_for('.redact_template', service_id=current_service.id, template_id=template.id) }}">Hide personalisation after sending</a>
+          <a href="{{ url_for('.confirm_redact_template', service_id=current_service.id, template_id=template.id) }}">Hide personalisation after sending</a>
         </span>
       {% endif %}
     {% endif %}