From b63693abe3bf27a7d5505763303bbe53090dd9e7 Mon Sep 17 00:00:00 2001
From: Fridolin Somers <fridolin.somers@biblibre.com>
Date: Fri, 6 Sep 2019 10:08:19 +0200
Subject: [PATCH] Revert "Bug 23273: Fix CSV export for overdues"

This reverts commit 1e63658b308507bb5e23bff4250b0425f2b614e3.
---
 circ/overdue.pl                               | 30 +++++++------------
 .../prog/en/modules/circ/overdue.tt           | 18 +++++------
 2 files changed, 18 insertions(+), 30 deletions(-)

diff --git a/circ/overdue.pl b/circ/overdue.pl
index f5683bda22..0b6f0f802a 100755
--- a/circ/overdue.pl
+++ b/circ/overdue.pl
@@ -39,30 +39,16 @@ my $borflagsfilter  = $input->param('borflag') || '';
 my $branchfilter    = $input->param('branch') || '';
 my $homebranchfilter    = $input->param('homebranch') || '';
 my $holdingbranchfilter = $input->param('holdingbranch') || '';
-my $dateduefrom     = $input->param('dateduefrom');
-my $datedueto       = $input->param('datedueto');
 my $op              = $input->param('op') || '';
 
-if ( $dateduefrom ) {
+my ($dateduefrom, $datedueto);
+if ( $dateduefrom = $input->param('dateduefrom') ) {
     $dateduefrom = dt_from_string( $dateduefrom );
 }
-if ( $datedueto ) {
+if ( $datedueto = $input->param('datedueto') ) {
     $datedueto = dt_from_string( $datedueto )->set_hour(23)->set_minute(59);
 }
 
-my $filters = {
-    itemtype      => $itemtypefilter,
-    borname       => $bornamefilter,
-    borcat        => $borcatfilter,
-    itemtype      => $itemtypefilter,
-    borflag       => $borflagsfilter,
-    branch        => $branchfilter,
-    homebranch    => $homebranchfilter,
-    holdingbranch => $holdingbranchfilter,
-    dateduefrom   => $dateduefrom,
-    datedueto     => $datedueto,
-};
-
 my $isfiltered      = $op =~ /apply/i && $op =~ /filter/i;
 my $noreport        = C4::Context->preference('FilterBeforeOverdueReport') && ! $isfiltered && $op ne "csv";
 
@@ -193,14 +179,18 @@ if (@patron_attr_filter_loop) {
 }
 
 
-use Data::Printer colored => 1; warn p $filters;
 $template->param(
     patron_attr_header_loop => [ map { { header => $_->{description} } } grep { ! $_->{isclone} } @patron_attr_filter_loop ],
-    filters => $filters,
+    branchfilter => $branchfilter,
+    homebranchfilter => $homebranchfilter,
+    holdingbranchfilter => $holdingbranchfilter,
     borcatloop=> \@borcatloop,
     itemtypeloop => \@itemtypeloop,
     patron_attr_filter_loop => \@patron_attr_filter_loop,
+    borname => $bornamefilter,
     showall => $showall,
+    dateduefrom => $dateduefrom,
+    datedueto   => $datedueto,
 );
 
 if ($noreport) {
@@ -356,8 +346,10 @@ if ($noreport) {
     # generate parameter list for CSV download link
     my $new_cgi = CGI->new($input);
     $new_cgi->delete('op');
+    my $csv_param_string = $new_cgi->query_string();
 
     $template->param(
+        csv_param_string        => $csv_param_string,
         todaysdate              => output_pref($today_dt),
         overdueloop             => \@overduedata,
         nnoverdue               => scalar(@overduedata),
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
index 0649d944a7..ad0d0594b8 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
@@ -34,11 +34,7 @@
 
   <p>
     [% IF ( isfiltered ) %]
-        [% SET url_params = '' %]
-        [% FOR var IN filters.keys %]
-            [% url_params = BLOCK %][% url_params %]&amp;[% var | uri %]=[% filters.$var | uri %][% END %]
-        [% END %]
-      <a href="overdue.pl?op=csv[% url_params | $raw %]">Download file of displayed overdues</a>
+      <a href="overdue.pl?op=csv&amp;[% csv_param_string | uri %]">Download file of displayed overdues</a>
     [% ELSE %]
       <a href="overdue.pl?op=csv">Download file of all overdues</a>
     [% END %]
@@ -106,15 +102,15 @@
 	<fieldset><legend>Date due:</legend>
 	<ol>
     <li><label for="from">From:</label>
-    <input type="text" id="from" name="dateduefrom" size="10" value="[% filters.dateduefrom | $KohaDates %]" class="datepickerfrom" />
+    <input type="text" id="from" name="dateduefrom" size="10" value="[% dateduefrom | $KohaDates %]" class="datepickerfrom" />
 	</li>
 	<li>
     <label for="to">To:</label>
-    <input type="text" id="to" name="datedueto" size="10" value="[% filters.datedueto | $KohaDates %]" class="datepickerto" />
+    <input type="text" id="to" name="datedueto" size="10" value="[% datedueto | $KohaDates %]" class="datepickerto" />
     </li>
     </ol></fieldset>
     <ol>
-    <li><label>Name or cardnumber:</label><input type="text" name="borname" value="[% filters.borname | html %]" /></li>
+    <li><label>Name or cardnumber:</label><input type="text" name="borname" value="[% borname | html %]" /></li>
     <li><label>Patron category:</label><select name="borcat" id="borcat"><option value="">Any</option>
       [% FOREACH borcatloo IN borcatloop %]
         [% IF ( borcatloo.selected ) %]<option value="[% borcatloo.value | html %]" selected="selected">[% borcatloo.catname | html %]</option>[% ELSE %]<option value="[% borcatloo.value | html %]">[% borcatloo.catname | html %]</option>[% END %]
@@ -171,7 +167,7 @@
         <label>Item home library:</label>
         <select name="homebranch" id="homebranch">
             <option value="">Any</option>
-            [% PROCESS options_for_libraries libraries => Branches.all( selected => filters.homebranch ) %]
+            [% PROCESS options_for_libraries libraries => Branches.all( selected => homebranchfilter ) %]
         </select>
     </li>
 
@@ -179,7 +175,7 @@
         <label>Item holding library:</label>
         <select name="holdingbranch" id="holdingbranch">
             <option value="">Any</option>
-            [% PROCESS options_for_libraries libraries => Branches.all( selected => filters.holdingbranch ) %]
+            [% PROCESS options_for_libraries libraries => Branches.all( selected => holdingbranchfilter ) %]
         </select>
     </li>
 
@@ -187,7 +183,7 @@
         <label>Library of the patron:</label>
         <select name="branch" id="branch">
             <option value="">Any</option>
-            [% PROCESS options_for_libraries libraries => Branches.all( selected => filters.branch, only_from_group => 1 ) %]
+            [% PROCESS options_for_libraries libraries => Branches.all( selected => branchfilter, only_from_group => 1 ) %]
         </select>
     </li>
 
-- 
2.39.5