Bug 28622: (bug 15758 follow-up) Remove incorrect branch selection in adv search
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 23 Jun 2021 14:19:15 +0000 (16:19 +0200)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 30 Jul 2021 13:48:54 +0000 (09:48 -0400)
commit3b552ac937b8c2b1402c42d46088b6562c54e345
tree4b224c114fb0605f5e3031243de7d42863f9535a
parentd1c7333fbfca7cdae943f2e82efb08f429370866
Bug 28622: (bug 15758 follow-up) Remove incorrect branch selection in adv search

From
  commit df97814f3034a3d5015546c4eca21f08ae951e37
  Bug 15758: Koha::Libraries - Remove GetBranches

+    selected_branchcode => ( C4::Context->IsSuperLibrarian ? C4::Context->userenv : '' ),

And

-        [% PROCESS options_for_libraries libraries => Branches.all() %]
+        [%# FIXME Should not we filter the libraries displayed? %]
+        [% PROCESS options_for_libraries libraries => Branches.all( selected => selected_branchcode, unfiltered => 1 ) %]

The change in the pl is wrong as it should pass C4::Context->userenv->{branch},
but prior to this commit it seems that we didn't select the logged in
library anyway.

Looks like we should remove this code but preserve the current behavior and select "All libraries".

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
catalogue/search.pl
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt