From 9de77f596c43d62f722194cc269bd95f218ebd81 Mon Sep 17 00:00:00 2001
From: Philipp Schafft <lion@lion.leolix.org>
Date: Fri, 9 Oct 2020 14:54:23 +0000
Subject: [PATCH] Feature: Ask for confirmation before killing a client or
 deleting a user

---
 admin/listclients.xsl          |  2 +-
 admin/manageauth.xsl           |  2 +-
 admin/ui/confirmdeleteuser.xsl | 24 ++++++++++++++++++++++++
 admin/ui/confirmkillclient.xsl | 23 +++++++++++++++++++++++
 4 files changed, 49 insertions(+), 2 deletions(-)
 create mode 100644 admin/ui/confirmdeleteuser.xsl
 create mode 100644 admin/ui/confirmkillclient.xsl

diff --git a/admin/listclients.xsl b/admin/listclients.xsl
index 22a22a0f..9bdd0d68 100644
--- a/admin/listclients.xsl
+++ b/admin/listclients.xsl
@@ -37,7 +37,7 @@
                                             <td><xsl:value-of select="connected" /></td>
                                             <td><xsl:value-of select="useragent" /></td>
                                             <td class="actions">
-                                                <a class="critical" href="/admin/killclient.xsl?mount={../@mount}&amp;id={id}">Kick</a>
+                                                <a class="critical" href="/admin/ui/confirmkillclient.xsl?mount={../@mount}&amp;id={id}">Kick</a>
                                                 <a href="/admin/moveclients.xsl?mount={../@mount}&amp;id={id}">Move</a>
                                             </td>
                                         </tr>
diff --git a/admin/manageauth.xsl b/admin/manageauth.xsl
index 137da23e..90bdfdc7 100644
--- a/admin/manageauth.xsl
+++ b/admin/manageauth.xsl
@@ -37,7 +37,7 @@
                                             <td><xsl:value-of select="username" /></td>
                                             <xsl:if test="../../@can-deleteuser = 'true'">
                                                 <td class="actions">
-                                                    <a class="critical" href="/admin/manageauth.xsl?id={../../@id}&amp;username={username}&amp;action=delete">Delete</a>
+                                                    <a class="critical" href="/admin/ui/confirmdeleteuser.xsl?id={../../@id}&amp;username={username}">Delete</a>
                                                 </td>
                                             </xsl:if>
                                         </tr>
diff --git a/admin/ui/confirmdeleteuser.xsl b/admin/ui/confirmdeleteuser.xsl
new file mode 100644
index 00000000..d8a59ae0
--- /dev/null
+++ b/admin/ui/confirmdeleteuser.xsl
@@ -0,0 +1,24 @@
+<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0" xmlns:xt="http://www.jclark.com/xt" extension-element-prefixes="xt" exclude-result-prefixes="xt">
+    <!-- Import include files -->
+    <xsl:include href="includes/confirm.xsl"/>
+    <xsl:variable name="title">Confirm killing source</xsl:variable>
+    <xsl:template name="content">
+        <xsl:for-each select="/report/incident">
+            <xsl:variable name="get-parameters" select="resource[@type='parameter']/value[@member='get-parameters']" />
+            <xsl:variable name="id" select="$get-parameters/value[@member='id']" />
+            <xsl:call-template name="confirm">
+                <xsl:with-param name="text">Please confirm deleting user <code><xsl:value-of select="$get-parameters/value[@member='username']/@value" /></code>.</xsl:with-param>
+                <xsl:with-param name="action-cancel">/admin/manageauth.xsl</xsl:with-param>
+                <xsl:with-param name="action-confirm">/admin/manageauth.xsl</xsl:with-param>
+                <xsl:with-param name="params-cancel">
+                    <xsl:copy-of select="$id" />
+                </xsl:with-param>
+                <xsl:with-param name="params-confirm">
+                    <xsl:copy-of select="$get-parameters/value[@member='username']" />
+                    <xsl:copy-of select="$id" />
+                    <value member="action" value="delete" />
+                </xsl:with-param>
+            </xsl:call-template>
+        </xsl:for-each>
+    </xsl:template>
+</xsl:stylesheet>
diff --git a/admin/ui/confirmkillclient.xsl b/admin/ui/confirmkillclient.xsl
new file mode 100644
index 00000000..b4f19959
--- /dev/null
+++ b/admin/ui/confirmkillclient.xsl
@@ -0,0 +1,23 @@
+<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0" xmlns:xt="http://www.jclark.com/xt" extension-element-prefixes="xt" exclude-result-prefixes="xt">
+    <!-- Import include files -->
+    <xsl:include href="includes/confirm.xsl"/>
+    <xsl:variable name="title">Confirm killing source</xsl:variable>
+    <xsl:template name="content">
+        <xsl:for-each select="/report/incident">
+            <xsl:variable name="get-parameters" select="resource[@type='parameter']/value[@member='get-parameters']" />
+            <xsl:variable name="mount" select="$get-parameters/value[@member='mount']" />
+            <xsl:call-template name="confirm">
+                <xsl:with-param name="text">Please confirm killing the client <code><xsl:value-of select="$get-parameters/value[@member='id']/@value" /></code> on <code><xsl:value-of select="$mount/@value" /></code>.</xsl:with-param>
+                <xsl:with-param name="action-cancel">/admin/listclients.xsl</xsl:with-param>
+                <xsl:with-param name="action-confirm">/admin/killclient.xsl</xsl:with-param>
+                <xsl:with-param name="params-cancel">
+                    <xsl:copy-of select="$mount" />
+                </xsl:with-param>
+                <xsl:with-param name="params-confirm">
+                    <xsl:copy-of select="$mount" />
+                    <xsl:copy-of select="$get-parameters/value[@member='id']" />
+                </xsl:with-param>
+            </xsl:call-template>
+        </xsl:for-each>
+    </xsl:template>
+</xsl:stylesheet>