From 4e82b8db94ee24ebd49541c25a6dac55cb8d543c Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 4 Mar 2016 11:52:25 -0500
Subject: [PATCH]  Bug 15981 - Serials frequencies can be deleted without
 warning

A JavaScript alert should ask the user to confirm deletion of a serial
frequency. This patch adds this to the template.

To test, apply the patch and go to Serials -> Manage frequencies

- In the list of frequencies, click any "Delete" link.
- Verify that a JavaScript alert is triggered.
- Test that cancelling the alert cancels the deletion operation.
- Test that confirming the alert allows the deletion to complete.

Signed-off-by: Nicole C Engard <nengard@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
---
 .../prog/en/modules/serials/subscription-frequencies.tt      | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-frequencies.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-frequencies.tt
index 8442ed23e1..5ffbbce36f 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-frequencies.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-frequencies.tt
@@ -61,6 +61,9 @@ $(document).ready(function() {
             $("#issuesperunit").val(1);
         }
     });
+    $(".delete_frequency").on("click",function(){
+        return confirmDelete();
+    });
 });
 //]]>
 </script>
@@ -208,7 +211,7 @@ $(document).ready(function() {
                   <td>[% frequency.displayorder %]</td>
                   <td>
                     <a href="/cgi-bin/koha/serials/subscription-frequencies.pl?op=modify&frequencyid=[% frequency.id %]">Modify</a> |
-                    <a href="/cgi-bin/koha/serials/subscription-frequencies.pl?op=del&frequencyid=[% frequency.id %]">Delete</a>
+                    <a class="delete_frequency" href="/cgi-bin/koha/serials/subscription-frequencies.pl?op=del&frequencyid=[% frequency.id %]">Delete</a>
                   </td>
                 </tr>
               [% END %]
-- 
2.39.5