https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25193 --- Comment #14 from Aleisha Amohia <aleisha@catalyst.net.nz> --- (In reply to Jonathan Druart from comment #13)
my $res = Koha::Holds->search({ biblionumber => $item->biblionumber, borrowernumber => $patron->borrowernumber })->next; next unless $item->holdingbranch eq $res->branchcode;
What if a given patron has several holds on the same record? Should not we loop on the different holds instead of testing only the first one?
I guess we could put another condition in the search params like: my $res = Koha::Holds->search({ biblionumber => $item->biblionumber, borrowernumber => $patron->borrowernumber, branchcode => $item->holdingbranch }, { order_by => { -asc => 'priority' } })->next; What do you think? -- You are receiving this mail because: You are watching all bug changes.