From d65932a3d9d2008dca03ffc708c28f6230c1603c Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@biblibre.com>
Date: Thu, 21 Mar 2013 14:52:52 +0100
Subject: [PATCH] Bug 8215: Followup Fix confirmation links for delete course and item

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
---
 .../en/modules/course_reserves/course-details.tt   |   27 ++++++-------------
 1 files changed, 9 insertions(+), 18 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
index 6b2f787..d19386e 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
@@ -7,10 +7,15 @@
 
 <script type="text/javascript">
 //<![CDATA[
+    $(document).ready(function(){
+        $("a.delete_item").click(function(){
+            return confirm( _("Are you sure you want to delete this item?"));
+        });
 
-    function confirmItemDelete(){
-        return confirm( _("Are you sure you want to delete this item?"));
-    }
+        $("#delete_course").click(function(){
+            return confirm( _("Are you sure you want to delete this course?") );
+        });
+    });
 
 //]]>
 </script>
@@ -28,20 +33,6 @@
     <div id="bd">
         <div id="yui-main">
             <div id="toolbar">
-                <script type="text/javascript">
-                //<![CDATA[
-                    $(document).ready(function(){
-                        deleteCourseButton.on("click", confirmDelete );
-                    });
-
-                    function confirmDelete(p_oEvent){
-                        if ( ! confirm( _("Are you sure you want to delete this course?") ) ) {
-                            YAHOO.util.Event.stopEvent( p_oEvent );
-                        }
-                    }
-
-                //]]>
-                </script>
                 <ul class="toolbar">
                     [% IF CAN_user_coursereserves_add_reserves %]<li><a class="btn" id="add_items" href="/cgi-bin/koha/course_reserves/add_items.pl?course_id=[% course.course_id %]">Add reserves</a></li>[% END %]
                     [% IF ( CAN_user_coursereserves_manage_courses ) %]<li><a class="btn" id="edit_course" href="/cgi-bin/koha/course_reserves/course.pl?course_id=[% course.course_id %]">Edit course</a></li>[% END %]
@@ -153,7 +144,7 @@
                                     [% ELSIF cr.item.itemlost %]
                                         Item Lost
                                     [% ELSE %]
-                                        <a href="course-details.pl?course_id=[% course.course_id %]&action=del_reserve&cr_id=[% cr.cr_id %]" onclick="return confirmItemDelete()" >Remove</a>
+                                        <a href="course-details.pl?course_id=[% course.course_id %]&action=del_reserve&cr_id=[% cr.cr_id %]" class="delete_item" >Remove</a>
                                     [% END %]
 
                                 </td>
-- 
1.7.2.5