https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28754
David Nind <da...@davidnind.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #123147|0 |1
is obsolete| |
--- Comment #2 from David Nind <da...@davidnind.com> ---
Created attachment 123158
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123158&action=edit
Bug 28754: Only adjust holds on specific biblio and don't go past end of array
Our query for lowest priority holds only needs to adjust holds on the biblio we
are looking at
so I add biblionumber
Additionally we can simply find the end of the array and use that rather than
99998
so I set new_rank to scalar @priority
Lastly, we don't need to fetch the lowest priority holds if we are ignoring
lowest priority
so I move it into the conditional
To test:
1 - Add holds with lowest priorty to 2 records in the catalog
2 - Add a hold on a third record
3 - Note errors in log like:
[2021/07/23 17:47:17] [WARN] splice() offset past end of array at
/kohadevbox/koha/C4/Reserves.pm line 1649
4 - Apply patch and restart all the things
5 - Add a new hold on third record - no warns
6 - Make one of the holds on third record have lowestPriority (click rightmost
arrow with line at bottom)
7 - Note no warns
8 - Adjust other holds on record and note no warns
Signed-off-by: David Nind <da...@davidnind.com>
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
