Attachment 17337 Details for Bug 8215 – Bug 8215: Followup Fix confirmation links for delete course and item

[patch] Bug 8215: Followup Fix confirmation links for delete course and item

Bug-8215-Followup-Fix-confirmation-links-for-delet.patch (text/plain), 2.96 KB, created by Kyle M Hall (khall) on 2013-04-10 11:31:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-04-10 11:31:04 UTC

Size: 2.96 KB

patch

obsolete

>From 4c6f88166f69207e08b779e9487e764b1e64577b 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
>
>---
> .../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

Actions: View | Diff | Splinter Review

Attachments on bug 8215: 9990 | 10181 | 10224 | 11131 | 11612 | 11657 | 11677 | 11678 | 11679 | 11680 | 11681 | 11682 | 11683 | 11860 | 11861 | 11862 | 11865 | 11875 | 11881 | 11882 | 11980 | 12039 | 12120 | 12128 | 12188 | 12189 | 12192 | 12246 | 12361 | 12400 | 12413 | 12414 | 12593 | 12627 | 13496 | 13497 | 13498 | 13499 | 15195 | 15196 | 15197 | 15198 | 15199 | 16228 | 16229 | 16251 | 16252 | 16532 | 16533 | 16534 | 16536 | 16537 | 16545 | 16567 | 16568 | 16587 | 16588 | 16599 | 16621 | 16623 | 16634 | 17333 | 17334 | 17335 | 17336 | 17337 | 17377 | 17378 | 17379 | 17894 | 17895 | 17896 | 17897 | 17898 | 17899 | 18274 | 18275 | 18276 | 18277 | 18278 | 18279 | 18282 | 18283 | 18284 | 18285 | 18286 | 18287 | 18288 | 18289 | 18290 | 18291 | 18292 | 18293 | 21885 | 23264