[Koha-bugs] [Bug 30860] Add option for CanBookBeReserved to return all item values

[bugzilla-daemon at bugs.koha-community.org]

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

ByWater Sandboxes <bws.sandboxes at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #145579|0                           |1
        is obsolete|                            |

--- Comment #21 from ByWater Sandboxes <bws.sandboxes at gmail.com> ---
Created attachment 145583
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=145583&action=edit
Bug 30860: Return all items from CanBookBeReserved on request.pl

This patch requests all item values from CanBookBeReserved on request.pl

Before this we either:
- Looped every item to find out that book could not be reserved
- Looped until we found an item that could be reserved, then looped all items
to get statuses

In the worst case we avoid double processing a single item, in the best case we
avoid double
processing all items (if only last on record is holdable)

To test:
1 - Find a record in staff client with several items
2 - Set AllowHoldsOnDamagedItems  to 'Dont allow'
3 - Add a damaged item to record
4 - Set a hold rule to only allow holds form homebranch and ensure record has
items from other branches
5 - Setup things to prevent more items from being held
6 - Attempt hold for patron
7 - Note item statuses
8 - Apply patch
9 - Confirm statuses are as they were before

Signed-off-by: Andrew Fuerste-Henry <andrewfh at dubcolib.org>

-- 
You are receiving this mail because:
You are watching all bug changes.