Bug 30126 - Hold pickup locations show incorrectly when RESTdefaultPageSize value is less than branch count
Summary: Hold pickup locations show incorrectly when RESTdefaultPageSize value is less...
Status: RESOLVED DUPLICATE of bug 30133
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Tomás Cohen Arazi (tcohen)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-02-17 19:07 UTC by Andrew Fuerste-Henry
Modified: 2022-02-18 15:27 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Fuerste-Henry 2022-02-17 19:07:12 UTC 
To recreate:
- have X branches
- set RESTdefaultPageSize to X-1
- set AllowHoldPolicyOverride to allow
- attempt to place a hold
- confirm pickup location list contains only the last 1 branch, alphabetically by branchcode
Comment 1 Tomás Cohen Arazi (tcohen) 2022-02-17 19:16:07 UTC 
Sorry for that XD
Comment 2 Tomás Cohen Arazi (tcohen) 2022-02-18 12:12:20 UTC 
I don't reproudce it, this morning. I found some weirdness in the code, though. But it only affects the 'needs override' icon (i.e. more libraries display the icon than they should).
Comment 3 Nick Clemens (kidclamp) 2022-02-18 15:27:26 UTC 

*** This bug has been marked as a duplicate of bug 30133 ***