From 7b7d73dee2bf334365e7aedb7572686dadfaa444 Mon Sep 17 00:00:00 2001
From: Philipp Schafft <lion@lion.leolix.org>
Date: Fri, 9 Oct 2020 14:43:06 +0000
Subject: [PATCH] Feature: Ask for confirmation before killing a source

---
 admin/includes/confirm.xsl     | 36 ++++++++++++++++++++++++++++++++++
 admin/includes/mountnav.xsl    |  2 +-
 admin/ui/confirmkillsource.xsl | 22 +++++++++++++++++++++
 web/assets/css/style.css       |  4 ++--
 4 files changed, 61 insertions(+), 3 deletions(-)
 create mode 100644 admin/includes/confirm.xsl
 create mode 100644 admin/ui/confirmkillsource.xsl

diff --git a/admin/includes/confirm.xsl b/admin/includes/confirm.xsl
new file mode 100644
index 00000000..8d976654
--- /dev/null
+++ b/admin/includes/confirm.xsl
@@ -0,0 +1,36 @@
+<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/page.xsl"/>
+
+    <xsl:template name="confirm">
+        <xsl:param name="text" />
+        <xsl:param name="action-cancel" />
+        <xsl:param name="action-confirm" />
+        <xsl:param name="params-cancel" />
+        <xsl:param name="params-confirm" />
+
+        <h2><xsl:value-of select="$title" /></h2>
+        <section class="box">
+            <h3 class="box_title">Please confirm</h3>
+            <p><xsl:copy-of select="$text" /></p>
+            <ul class="boxnav">
+                <li>
+                    <form method="get" action="{$action-cancel}">
+                        <xsl:for-each select="xt:node-set($params-cancel)/*">
+                            <input type="hidden" name="{@member}" value="{@value}" />
+                        </xsl:for-each>
+                        <input type="submit" value="Cancel" />
+                    </form>
+                </li>
+                <li>
+                    <form method="post" action="{$action-confirm}">
+                        <xsl:for-each select="xt:node-set($params-confirm)/*">
+                            <input type="hidden" name="{@member}" value="{@value}" />
+                        </xsl:for-each>
+                        <input class="critical" type="submit" value="Confirm" />
+                    </form>
+                </li>
+            </ul>
+        </section>
+    </xsl:template>
+</xsl:stylesheet>
diff --git a/admin/includes/mountnav.xsl b/admin/includes/mountnav.xsl
index 9fb5c127..44814cb9 100644
--- a/admin/includes/mountnav.xsl
+++ b/admin/includes/mountnav.xsl
@@ -8,7 +8,7 @@
                 <li><a href="/admin/moveclients.xsl?mount={$mount}">Move listeners</a></li>
                 <li><a href="/admin/updatemetadata.xsl?mount={$mount}">Metadata</a></li>
                 <li><a href="/admin/fallbacks.xsl?mount={$mount}&amp;omode=strict">Set fallback</a></li>
-                <li class="critical"><a href="/admin/killsource.xsl?mount={$mount}">Kill source</a></li>
+                <li class="critical"><a href="/admin/ui/confirmkillsource.xsl?mount={$mount}">Kill source</a></li>
             </ul>
         </div>
     </xsl:template>
diff --git a/admin/ui/confirmkillsource.xsl b/admin/ui/confirmkillsource.xsl
new file mode 100644
index 00000000..a02bdb3d
--- /dev/null
+++ b/admin/ui/confirmkillsource.xsl
@@ -0,0 +1,22 @@
+<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 source connection <code><xsl:value-of select="$mount/@value" /></code>. This will disconnect all listeners.</xsl:with-param>
+                <xsl:with-param name="action-cancel">/admin/listmounts.xsl</xsl:with-param>
+                <xsl:with-param name="action-confirm">/admin/killsource.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:with-param>
+            </xsl:call-template>
+        </xsl:for-each>
+    </xsl:template>
+</xsl:stylesheet>
diff --git a/web/assets/css/style.css b/web/assets/css/style.css
index 37a5bbad..00e63468 100644
--- a/web/assets/css/style.css
+++ b/web/assets/css/style.css
@@ -141,7 +141,7 @@ ul.boxnav > li {
     margin-left: 1em;
 }
 
-ul.boxnav > li > a {
+ul.boxnav > li > a, ul.boxnav > li input[type='submit'] {
     background-color: #4f8cb0;
     border: none;
     color: white;
@@ -152,7 +152,7 @@ ul.boxnav > li > a {
     display: inline-block;
 }
 
-ul.boxnav > li.critical > a, a.critical {
+ul.boxnav > li.critical > a, a.critical, input[type='submit'].critical {
     background-color: #ff704d !important;
 }