[Koha-bugs] [Bug 11999] Add two checks in CanBookBeReserved and CanItemBeReserved
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Tue Jan 10 13:21:58 CET 2017
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11999
Julian Maurice <julian.maurice at biblibre.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #50224|0 |1
is obsolete| |
--- Comment #44 from Julian Maurice <julian.maurice at biblibre.com> ---
Created attachment 58741
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=58741&action=edit
Bug 11999: Add two checks in CanBookBeReserved and CanItemBeReserved
- Check if borrower already has reserved the same biblio (or item).
- Check if borrower has reached the maximum number of holds allowed
(syspref maxreserves)
The goal of this patch is to do these checks also when using ILS-DI
services HoldTitle and HoldItem
Test plan:
----------
Before patch:
1/ Set syspref maxreserves to 1
2/ Place some holds through ILS-DI and note that you can place more than
1 hold for a borrower
3/ Try to hold the same biblio/item with the same borrower through
ILS-DI and note that you can
Apply the patch
4/ Place some holds through ILS-DI, you shouldn't be able to place more
than 1 hold for a borrower
5/ Try to hold the same biblio/item with the same borrower through
ILS-DI. You shouldn't be able to do that.
6/ Try to place holds on staff interface and OPAC. The behaviour must be
identical than before the patch.
Maxreserves and alreadyreserved works on ILSDI
Maxreserves works also on staff and opac
Already reserves works also on staff and opac
tests on t/db_dependent/Reserves.t and t/db_dependent/Holds.t passe (using
koha_ut db)
Signed-off-by: Alex Sassmannshausen <alex.sassmannshausen at ptfs-europe.com>
Signed-off-by: Alex Arnaud <alex.arnaud at biblibre.com>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list