[Koha-bugs] [Bug 30847] Cleanup opac-reserve.pl
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Wed Jul 13 08:40:54 CEST 2022
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30847
Jonathan Druart <jonathan.druart+koha at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #135340|0 |1
is obsolete| |
Attachment #135341|0 |1
is obsolete| |
Attachment #135342|0 |1
is obsolete| |
Attachment #135343|0 |1
is obsolete| |
Attachment #135344|0 |1
is obsolete| |
Attachment #135345|0 |1
is obsolete| |
Attachment #135346|0 |1
is obsolete| |
Attachment #135347|0 |1
is obsolete| |
Attachment #135348|0 |1
is obsolete| |
--- Comment #14 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
Created attachment 137668
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=137668&action=edit
Bug 30847: Cleanup calculation of holds and rank
Currently we loop over all the holds on the record to get a count of the
reserves not
waiting and a rank for the upcoming hold
We can simply count the number of nont waiting holds and add 1
Note: Before this patch the count of holds included the new hold we were going
to place
which meant rank and count were the same. I believe countaing existing holds
and our
upcoming rank is the correct behavior
To test:
1 - Set OPACShowHoldQueueDetails system preference to 'Show holds and their
priorty level'
2 - Place a few holds on a bib on the staff side
3 - Mark 1 waiting
4 - Attempt to place hold on the bib on the opac
5 - Note 'Your priority' and 'Number of holds' are the same number
6 - Place another hold, confirm both increase
7 - Set another hold waiting, confirm both decrease
8 - Apply patch
9 - Confirm 'Your priority' is now 1 greater than Number of holds
10 - Place a hold, set a hold waiting, confirm numbers adjust as appropriate
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list