From da678ed2e8accc6656adcdeff23d5157ee7f1ddf Mon Sep 17 00:00:00 2001
From: Agustin Moyano <agustinmoyano@theke.io>
Date: Wed, 12 Jun 2019 00:06:27 -0300
Subject: [PATCH] Bug 22284: (follow-up) Filter pickup on specific item click

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
---
 .../prog/en/modules/reserve/request.tt             |  5 +++--
 koha-tmpl/intranet-tmpl/prog/js/request.js         | 23 ++++++++++++++++++++++
 reserve/request.pl                                 |  2 ++
 3 files changed, 28 insertions(+), 2 deletions(-)
 create mode 100644 koha-tmpl/intranet-tmpl/prog/js/request.js

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 f319eec271..9d85da339b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -333,9 +333,9 @@
                         Hold must be record level
                     </span>
                 [% ELSIF ( itemloo.available ) %]
-                    <input type="radio" name="checkitem" value="[% itemloo.itemnumber | html %]" />
+                    <input type="radio" name="checkitem" data-pickup-locations="[% itemloo.pickup_locations_code | html %]" value="[% itemloo.itemnumber | html %]" />
                 [% ELSIF ( itemloo.override ) %]
-                    <input type="radio" name="checkitem" class="needsoverride" value="[% itemloo.itemnumber | html %]" />
+                    <input type="radio" name="checkitem" class="needsoverride" value="[% itemloo.itemnumber | html %]" data-pickup-locations="[% itemloo.pickup_locations_code | html %]" />
                     <i class="fa fa-exclamation-triangle fa-lg" style="color:gold" title="Requires override of hold policy"/></i>
                 [% ELSE %]
                     <span class="error">
@@ -699,6 +699,7 @@
     [% INCLUDE 'calendar.inc' %]
     [% INCLUDE 'columns_settings.inc' %]
     [% Asset.js("js/circ-patron-search-results.js") | $raw %]
+    [% Asset.js("js/request.js") | $raw %]
     <script>
         var biblionumber = "[% biblionumber | $raw %]";
         var borrowernumber = "[% patron.borrowernumber | $raw %]";
diff --git a/koha-tmpl/intranet-tmpl/prog/js/request.js b/koha-tmpl/intranet-tmpl/prog/js/request.js
new file mode 100644
index 0000000000..9c45e7ec50
--- /dev/null
+++ b/koha-tmpl/intranet-tmpl/prog/js/request.js
@@ -0,0 +1,23 @@
+$(document).ready(function() {
+    $('input[name="checkitem"]').click(function() {
+        var pl = $(this).data('pickup-locations');
+        var select = $('select[name="pickup"]');
+        var options = $('option', select);
+        if(pl == 'all') {
+            options.removeClass('hide');
+        } else {
+            pl = pl.split(',');
+            options.each(function() {
+                var value = $(this).val();
+                if($.inArray(value, pl) == -1) {
+                    $(this).addClass('hide');
+                    if(select.val() == value) {
+                        select.val(pl[0]);
+                    }
+                } else {
+                    $(this).removeClass('hide');
+                }
+            });
+        }
+    });
+});
diff --git a/reserve/request.pl b/reserve/request.pl
index 068c7adfe4..8641ecc9e4 100755
--- a/reserve/request.pl
+++ b/reserve/request.pl
@@ -505,8 +505,10 @@ foreach my $biblionumber (@biblionumbers) {
                     $num_available++;
                     if($branchitemrule->{'hold_fulfillment_policy'} eq 'any' ) {
                         $item->{pickup_locations} = 'Any library';
+                        $item->{pickup_locations_code} = 'all';
                     } else {
                         $item->{pickup_locations} = join (', ', map { $_->{branchname} } Koha::Items->find($itemnumber)->pickup_locations({ patron => $patron }));
+                        $item->{pickup_locations_code} = join (',', map { $_->{branchcode} } Koha::Items->find($itemnumber)->pickup_locations({ patron => $patron }));
                     }
 
                     push( @available_itemtypes, $item->{itype} );
-- 
2.11.0