@@ -, +, @@ screens - Apply the patch - Make sure you have at least one saved report - Make sure you have the create_reports permission - Go to the Show, Edit and Run screens and verify the Delete link is present and functional, including the JavaScript confirmation dialogue - Change to a user without the create_reports permission and check that the Delete link is not displayed on the Show, Edit and Run screens - Sign ye off merrily! --- koha-tmpl/intranet-tmpl/prog/en/includes/reports-toolbar.inc | 8 ++++++++ .../intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) --- a/koha-tmpl/intranet-tmpl/prog/en/includes/reports-toolbar.inc +++ a/koha-tmpl/intranet-tmpl/prog/en/includes/reports-toolbar.inc @@ -20,6 +20,14 @@ [% END %] [% END %] + [% IF ( CAN_user_reports_create_reports ) %] +
+ + Delete + +
+ [% END %] + [% UNLESS ( errors ) # Unless there are errors saving a report %]
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt @@ -157,7 +157,7 @@ $(document).ready(function(){ $(this).select(); }); [% END %] -[% IF ( saved1 ) %] +[% IF ( saved1 || showsql || execute || editsql || save_successful ) %] $(".confirmdelete").click(function(){ $(this).parents('tr').attr("class","warn"); if(confirm(_("Are you sure you want to delete this saved report?"))){ --