Attachment 127073 Details for Bug 29355 – Bug 29355: Make pickup location dropdowns RESTdefaultPageSize agnostic

[patch] Bug 29355: Make pickup location dropdowns RESTdefaultPageSize agnostic

Bug-29355-Make-pickup-location-dropdowns-RESTdefau.patch (text/plain), 3.23 KB, created by David Nind on 2021-10-28 18:28:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2021-10-28 18:28:56 UTC

Size: 3.23 KB

patch

obsolete

>From 3f6ceade38297fa9c27d2c570d4b1aeb67a47b20 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Thu, 28 Oct 2021 15:03:18 -0300
>Subject: [PATCH] Bug 29355: Make pickup location dropdowns RESTdefaultPageSize
> agnostic
>
>This patch makes the select2 dropdowns for pickup locations not be
>limited to the RESTdefaultPageSize syspref limit.
>
>To test:
>1. Have less than 20 libraries in your system as valid pickup locations
>2. Place a hold via the staff client
>=> SUCCESS: See that all your libraries appear in the pickup location dropdowns at the bib and item level
>3. Update RESTdefaultPageSize, set the value to something lower than your count of pickup libraries
>4. place another hold
>=> FAIL: Your pickup location list gets cut off and only shows as many locations as RESTdefaultPageSize allows
>5. Apply this patch
>6. Repeat 4
>=> SUCCESS: All your pickup locations show
>7. Sign off :-D
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt | 6 ++++--
> koha-tmpl/intranet-tmpl/prog/js/holds.js                   | 3 ++-
> 2 files changed, 6 insertions(+), 3 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 a6d269f71f..1d4031babf 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
>@@ -1261,7 +1261,8 @@
>                             var search_term = (params.term === undefined) ? '' : params.term;
>                             var query = {
>                                 "q": JSON.stringify({"name":{"-like":search_term+'%'}}),
>-                                "_order_by": "name"
>+                                "_order_by": "name",
>+                                "_per_page": -1
>                             };
>                             return query;
>                         },
>@@ -1323,7 +1324,8 @@
>                             var query = {
>                                 "q": JSON.stringify({"name":{"-like":search_term+'%'}}),
>                                 "_order_by": "name",
>-                                "patron_id": patron_id
>+                                "patron_id": patron_id,
>+                                "_per_page": -1
>                             };
>                             return query;
>                         },
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/holds.js b/koha-tmpl/intranet-tmpl/prog/js/holds.js
>index 13309533a9..1f32a00f9f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/holds.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/holds.js
>@@ -250,7 +250,8 @@ $(document).ready(function() {
>                                 var search_term = (params.term === undefined) ? '' : params.term;
>                                 var query = {
>                                     "q": JSON.stringify({"name":{"-like":search_term+'%'}}),
>-                                    "_order_by": "name"
>+                                    "_order_by": "name",
>+                                    "_per_page": -1
>                                 };
>                                 return query;
>                             },
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 29355: 127071 | 127073 | 127074 | 127187