From f245d427c5c65ec8a6e9d6e7383f865b7bd967f7 Mon Sep 17 00:00:00 2001
From: Lari Taskula <lari.taskula@jns.fi>
Date: Wed, 8 Feb 2017 16:45:23 +0200
Subject: [PATCH] Bug 7534: Display pickup locations instead of all libraries
 in opac-reserve.pl

This patch sets opac-reserve.tt to use the list of pickup libraries instead
of every library in the system.

To test:
1. Go to cgi-bin/koha/admin/branches.pl
2. Configure one of your libraries to "Pickup location" => "No"
3. Go place a hold in intranet
4. See the provided "Pickup at" list
5. Observe the library that you configured is not present in the list
6. Place the hold
7. See list of existing holds on that biblio
8. See "Pickup library" column and observe your configured library is
   not included in the list
---
 koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
index 12b4e1b..a1c368c 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -374,7 +374,7 @@ function checkMultiHold() {
         <li>
             <label for="pickup">Pickup at:</label>
             <select name="pickup" size="1" id="pickup">
-                [% PROCESS options_for_libraries libraries => Branches.all() %]
+                [% PROCESS options_for_libraries libraries => Branches.pickup_locations() %]
             </select>
         </li>
 
@@ -805,7 +805,7 @@ function checkMultiHold() {
          [% END %]
     [% ELSE %]
         <select name="pickup">
-            [% PROCESS options_for_libraries libraries => Branches.all( selected => reserveloo.branchcode ) %]
+            [% PROCESS options_for_libraries libraries => Branches.pickup_locations( selected => reserveloo.branchcode ) %]
         </select>
     [% END %]
         </td>
-- 
1.9.1