Bug 30126

Summary:	Hold pickup locations show incorrectly when RESTdefaultPageSize value is less than branch count
Product:	Koha	Reporter:	Andrew Fuerste-Henry <andrew>
Component:	Hold requests	Assignee:	Tomás Cohen Arazi (tcohen) <tomascohen>
Status:	RESOLVED DUPLICATE	QA Contact:	Testopia <testopia>
Severity:	normal
Priority:	P5 - low	CC:	gmcharlt, nick, tomascohen
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:

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 ***