]> git.koha-community.org Git - koha.git/commit
Bug 34666: Allow item_group to be null and still match
authorNick Clemens <nick@bywatersolutions.com>
Wed, 30 Aug 2023 18:38:56 +0000 (18:38 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 8 Sep 2023 14:54:52 +0000 (11:54 -0300)
commitfe3872f62885b4b0ee483e446c0b2b85e74970f7
treeb6aa64b511b93ae53945e562e9653548064d53dd
parent99d7e55be67332fea1ced61ddf8105b4261923d8
Bug 34666: Allow item_group to be null and still match

The current logic requires that the grop ids match, but this eliminates null matches
from the group.

The fallout essentially is that the queue won't be checked to fill holds in cases of title level matches
where holds don't have an item group id. The queue checks the transport cost matrix while the check reserves
check does not, so this may have an impact on holds costs and delivery times

To test:
0 - Apply unit test patch
1 - prove -v t/db_dependent/Reserves.t
2 - It fails
3 - Apply this patch
4 - prove -v t/db_dependent/Reserves.t
5 - It passes!

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
C4/Reserves.pm