Bug 21837: Don't set holdingbranch filter using selected homebranch filter
authorNick Clemens <nick@bywatersolutions.com>
Wed, 14 Nov 2018 19:08:09 +0000 (19:08 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Wed, 21 Nov 2018 11:24:02 +0000 (11:24 +0000)
To recreate:
1 - Open the overdues report (don't worry if you don't have results)
2 - Choose to filter by homebranch
3 - Note the holdingbranch filter is set to the selected homebranch
4 - Apply patch
5 - Clear filters
6 - repeat
7 - only homebranch is set

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

circ/overdue.pl

index 5adc8d8..0b6f0f8 100755 (executable)
@@ -183,7 +183,7 @@ $template->param(
     patron_attr_header_loop => [ map { { header => $_->{description} } } grep { ! $_->{isclone} } @patron_attr_filter_loop ],
     branchfilter => $branchfilter,
     homebranchfilter => $homebranchfilter,
-    holdingbranchfilter => $homebranchfilter,
+    holdingbranchfilter => $holdingbranchfilter,
     borcatloop=> \@borcatloop,
     itemtypeloop => \@itemtypeloop,
     patron_attr_filter_loop => \@patron_attr_filter_loop,