diff --git a/acqui/invoice.pl b/acqui/invoice.pl index 2c88908086..f822ee4480 100755 --- a/acqui/invoice.pl +++ b/acqui/invoice.pl @@ -74,7 +74,7 @@ if ( C4::Context->preference('AcqEnableFiles') ) { if ( $op && $op eq 'cud-close' ) { output_and_exit( $input, $cookie, $template, 'insufficient_permission' ) unless $logged_in_patron->has_permission( { acquisition => 'edit_invoices' } ); - my @invoiceid = $input->multi_param('invoiceid'); + my @invoiceid = split( ',', $input->param('invoiceid') ); foreach my $invoiceid ( @invoiceid ) { CloseInvoice($invoiceid); } diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt index 6f9901bf71..d2fb83ce06 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoices.tt @@ -128,7 +128,7 @@ Reopen selected invoices [% ELSE %] Merge selected invoices - Close selected invoices + Close selected invoices [% END # /IF tab == 'closed' %] [% END # /IF CAN_user_acquisition_merge_invoices %] [% END # /BLOCK invoices_table %] @@ -461,16 +461,14 @@ }); $('#open_sel,#close_sel').click(function () { - var referer = $(this).attr("data-referer"); - var op = $(this).attr("data-op"); var table = $(this).data('table'); - var invoice_link = "invoice.pl?op="+op; + var invoice_ids = ""; if ($('#' + table + ' .select-invoice:checked').length) { $('#' + table + ' .select-invoice:checked').each(function () { var row = $(this).parents('tr'); - invoice_link = invoice_link + "&invoiceid="+$(row).attr('data-invoiceid'); + invoice_ids = invoice_ids + ","+$(row).attr('data-invoiceid'); }); - window.location.href =invoice_link +"&"+referer; + $(this).attr('data-invoiceid', invoice_ids); } else { alert ("Please select at least one invoice." ); }