Bugzilla – Attachment 20524 Details for
Bug 3134
Ability to select multiple reports to delete at once
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 3134 - Ability to selelct multiple reports to delete at once
Bug-3134---Ability-to-selelct-multiple-reports-to-.patch (text/plain), 5.27 KB, created by
Jonathan Druart
on 2013-08-21 08:47:02 UTC
(
hide
)
Description:
Bug 3134 - Ability to selelct multiple reports to delete at once
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2013-08-21 08:47:02 UTC
Size:
5.27 KB
patch
obsolete
>From 26d6788a9cc378ddb82b13da5adc0aa42134867f Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Wed, 3 Jul 2013 10:05:55 -0400 >Subject: [PATCH] Bug 3134 - Ability to selelct multiple reports to delete at > once > >This patch adds the option to select multiple saved reports for >deletion. > >To test you must have two or more saved reports to delete. Deletion >should work properly when: > >- Selecting one report for deletion by checking the box. >- Selecting more than one report for deletion by checking boxes. >- Clicking the old "Delete" link > >Clicking the delete button should prompt you to confirm. Clicking cancel >should cancel. > >Clicking the delete button when no boxes are checked should trigger an >alert asking you to select reports for deletion. > >Signed-off-by: Liz Rea <wizzyrea@gmail.com> >Functional tests pass, template tests pass. > >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >--- > C4/Reports/Guided.pm | 14 +++++++------- > .../prog/en/modules/reports/guided_reports_start.tt | 19 +++++++++++++++++-- > reports/guided_reports.pl | 11 +++++++++-- > 3 files changed, 33 insertions(+), 11 deletions(-) > >diff --git a/C4/Reports/Guided.pm b/C4/Reports/Guided.pm >index f0db7a3..9447c71 100644 >--- a/C4/Reports/Guided.pm >+++ b/C4/Reports/Guided.pm >@@ -609,13 +609,13 @@ sub format_results { > } > > sub delete_report { >- my ($id) = @_; >- my $dbh = C4::Context->dbh(); >- my $query = "DELETE FROM saved_sql WHERE id = ?"; >- my $sth = $dbh->prepare($query); >- $sth->execute($id); >-} >- >+ my (@ids) = @_; >+ my $dbh = C4::Context->dbh; >+ my $query = 'DELETE FROM saved_sql WHERE id IN (' . join( ',', ('?') x @ids ) . ')'; >+ my $sth = $dbh->prepare($query); >+ $sth->execute(@ids); >+ $sth->finish; >+} > > my $SAVED_REPORTS_BASE_QRY = <<EOQ; > SELECT s.*, r.report, r.date_run, $AREA_NAME_SQL_SNIPPET, av_g.lib AS groupname, av_sg.lib AS subgroupname, >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt >index 0138086..9f35e08 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt >@@ -122,6 +122,15 @@ $(document).ready(function(){ > rtable.fnSetColumnVis(4, true); > } > }); >+ >+ $("#reports_form").submit(function(){ >+ var checkedItems = $("input[name=ids]:checked"); >+ if ($(checkedItems).size() == 0) { >+ alert(_("You must select one or more reports to delete")); >+ return false; >+ } >+ return confirm(_("Are you sure you want to delete the selected reports?")); >+ }); > [% END %] > > [% IF ( showsql ) %] >@@ -132,7 +141,7 @@ $(document).ready(function(){ > [% IF ( saved1 ) %] > $(".confirmdelete").click(function(){ > $(this).parents('tr').attr("class","warn"); >- if(confirm("Are you sure you want to "+$(this).attr("title")+"?")){ >+ if(confirm(_("Are you sure you want to delete this saved report?"))){ > return true; > } else { > $(this).parents('tr').attr("class",""); >@@ -273,6 +282,8 @@ canned reports and writing custom SQL reports.</p> > <option value="">All</option> > </select> > </div> >+<form action="/cgi-bin/koha/reports/guided_reports.pl" id="reports_form" method="post"> >+<input type="hidden" name="phase" value="Delete Multiple" /> > <table id="table_reports"> > <thead> > <tr> >@@ -293,7 +304,7 @@ canned reports and writing custom SQL reports.</p> > <tbody> > [% FOREACH savedreport IN savedreports %] > [% UNLESS ( loop.odd ) %]<tr class="odd">[% ELSE %]<tr>[% END %] >- <td>[% savedreport.id %]</td> >+ <td><label>[% savedreport.id %] <input type="checkbox" name="ids" value="[% savedreport.id %]" /></label></td> > <td> > [% IF ( savedreport.report_name ) %] > [% savedreport.report_name %] >@@ -338,6 +349,10 @@ canned reports and writing custom SQL reports.</p> > [% END %] > </tbody> > </table> >+ <fieldset class="action"> >+ <input type="submit" value="Delete selected" /> >+ </fieldset> >+ </form> > </div> > </div> > [% ELSE %]<div class="dialog message"> >diff --git a/reports/guided_reports.pl b/reports/guided_reports.pl >index 929feea..b9c49ad 100755 >--- a/reports/guided_reports.pl >+++ b/reports/guided_reports.pl >@@ -108,11 +108,18 @@ elsif ( $phase eq 'Build new' ) { > ); > } > >+elsif ( $phase eq 'Delete Multiple') { >+ my @ids = $input->param('ids'); >+ delete_report( @ids ); >+ print $input->redirect("/cgi-bin/koha/reports/guided_reports.pl?phase=Use%20saved"); >+ exit; >+} >+ > elsif ( $phase eq 'Delete Saved') { > > # delete a report from the saved reports list >- my $id = $input->param('reports'); >- delete_report($id); >+ my $ids = $input->param('reports'); >+ delete_report($ids); > print $input->redirect("/cgi-bin/koha/reports/guided_reports.pl?phase=Use%20saved"); > exit; > } >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 3134
:
14604
|
19376
|
19806
|
19843
|
20461
|
20465
|
20516
|
20517
| 20524 |
20525
|
20526
|
20527
|
20528