From 9fe4546dc15918ba20e96686886d94c416fb1cc7 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Wed, 3 Feb 2016 08:40:20 -0500
Subject: [PATCH] Bug 15784: Library deletion warning is incorrectly styled
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

When deleting a library in Administration ->  Libraries and groups, the
deletion confirmation message is not styled with the standard "dialog
alert" <div>. This patch modifies the confirmation markup to match the
standard.

To test, apply the patch and go to Administration -> Libraries and
groups.

- Try deleting a library. The confirmation box should have the standard
  "dialog alert" class.
- Test the 'confirm' and 'cancel' actions and make sure both complete
  correctly.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
(cherry picked from commit cfb46e45cb74fdc4a42bb48df78fe820d31fdc86)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
---
 .../prog/en/modules/admin/branches.tt             | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
index 941514c2bf..d2cdd273fb 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
@@ -164,12 +164,19 @@ tinyMCE.init({
 [% END %]
 
 [% IF ( delete_confirm ) %]
-    <form action="[% action %]" method="post">
-<fieldset><legend>Confirm deletion of [% branchname %] ([% branchcode %])?</legend><input type="hidden" name="op" value="delete_confirmed" />
+    <div class="dialog alert">
+      <form action="[% action %]" method="post">
+        <h3>Are you sure you want to delete [% branchname %] ([% branchcode %])?</h3>
+
+        <input type="hidden" name="op" value="delete_confirmed" />
         <input type="hidden" name="branchcode" value="[% branchcode |html %]" />
         <input type="hidden" name="branchname" value="[% branchname |html %]">
-        <fieldset class="action"><input type="submit" value="Delete Library" /> <a class="cancel" href="/cgi-bin/koha/admin/branches.pl">Cancel</a></fieldset>
-</fieldset></form>
+        <input type="submit" class="approve" value="Yes, delete" />
+      </form>
+      <form action="/cgi-bin/koha/admin/branches.pl" method="get">
+        <input type="submit" class="deny" value="No, do not delete"/>
+      </form>
+    </div>
 [% END %]
 
 [% IF ( else ) %]
-- 
2.39.5