Attachment 48744 Details for Bug 15804 – [3.22.x] Bug 15804: Use standard dialog style for confirmation of MARC subfield deletion

[patch] [3.22.x] Bug 15804: Use standard dialog style for confirmation of MARC subfield deletion

Bug-15804-Use-standard-dialog-style-for-confirmati.patch (text/plain), 3.78 KB, created by Julian Maurice on 2016-03-07 12:32:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Julian Maurice

Created: 2016-03-07 12:32:07 UTC

Size: 3.78 KB

patch

obsolete

>From 8323259d8401907975446af35f2a018748d35e40 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 11 Feb 2016 12:23:49 -0500
>Subject: [PATCH] Bug 15804: Use standard dialog style for confirmation of MARC
> subfield deletion
>
>When deleting a MARC framework subfield, the confirmation message is not
>styled. This patch modifies the template to style the confirmation
>message like similar ones.
>
>To test, apply the patch and go to Administration -> MARC framworks.
>
> - Click 'MARC structure' for a framework you can edit.
> - Click 'subfields' for a tag you can edit.
> - Click 'Delete' for a subfield you can delete. You should see a
>   confirmation dialog styled like other "alert" type dialogs.
> - Test both canceling and confirming deletion to confirm that both
>   function correctly.
>
>Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
>---
> .../en/modules/admin/marc_subfields_structure.tt   | 42 ++++++++++++----------
> 1 file changed, 23 insertions(+), 19 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tt
>index bd76f43..20a4e1c 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tt
>@@ -346,25 +346,29 @@ function populateHiddenCheckboxes(tab) {
> 
> [% IF ( delete_confirm ) %]
> 
>-<h3>Confirm deletion of subfield [% tagsubfield %]?</h3>
>-    <p>Subfield: [% tagsubfield %]</p>
>-    <p>Description: [% liblibrarian %]</p>
>-
>-            <form action="[% delete_link %]" method="post"><input type="hidden" name="op" value="delete_confirmed" />
>-                <input type="hidden" name="searchfield" value="[% searchfield %]" />
>-                <input type="hidden" name="tagfield" value="[% tagfield  | html %]" />
>-				<input type="hidden" name="tagsubfield" value="[% tagsubfield %]" />
>-				<input type="hidden" name="frameworkcode" value="[% frameworkcode %]" />
>-				<input type="submit" value="Yes, delete this subfield" />
>-            </form>
>-			
>-			<form action="[% script_name %]" method="post">
>-			<input type="hidden" name="searchfield" value="[% searchfield %]" />
>-                        <input type="hidden" name="tagfield" value="[% tagfield | html %]" />
>-			<input type="hidden" name="tagsubfield" value="[% tagsubfield %]" />
>-			<input type="hidden" name="frameworkcode" value="[% frameworkcode %]" />
>-			<input type="submit" value="No, do not delete" />
>-            </form>
>+    <div class="dialog alert">
>+        <h3>Confirm deletion of subfield [% tagsubfield %]?</h3>
>+        <p>Subfield: [% tagsubfield %]</p>
>+        <p>Description: [% liblibrarian %]</p>
>+
>+        <form action="[% delete_link %]" method="post"><input type="hidden" name="op" value="delete_confirmed" />
>+            <input type="hidden" name="searchfield" value="[% searchfield %]" />
>+            <input type="hidden" name="tagfield" value="[% tagfield | html %]" />
>+            <input type="hidden" name="tagsubfield" value="[% tagsubfield %]" />
>+            <input type="hidden" name="frameworkcode" value="[% frameworkcode %]" />
>+            <input type="submit" class="approve" value="Yes, delete this subfield"/>
>+        </form>
>+
>+
>+        <form action="[% script_name %]" method="post">
>+            <input type="hidden" name="searchfield" value="[% searchfield %]" />
>+            <input type="hidden" name="tagfield" value="[% tagfield | html%]" />
>+            <input type="hidden" name="tagsubfield" value="[% tagsubfield %]" />
>+            <input type="hidden" name="frameworkcode" value="[% frameworkcode %]" />
>+            <input type="submit" class="deny" value="No, do not delete"/>
>+        </form>
>+    </div>
>+
> [% END %]
> 
> [% IF ( delete_confirmed ) %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 15804: 47934 | 48183 | 48239 | 48240 | 48453 | 48454 | 48455 | 48744