https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11999

Arthur Suzuki <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #99927|0                           |1
        is obsolete|                            |
 Attachment #106481|0                           |1
        is obsolete|                            |
 Attachment #106494|0                           |1
        is obsolete|                            |

--- Comment #110 from Arthur Suzuki <[email protected]> ---
Created attachment 111687
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111687&action=edit
Bug 11999: Add maxreserves check in CanBookBeReserved and CanItemBeReserved

- 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

Apply the patch
3/ Place some holds through ILS-DI, you shouldn't be able to place more
   than 1 hold for a borrower
4/ 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 <[email protected]>
Signed-off-by: Alex Arnaud <[email protected]>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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/

Reply via email to