https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25193
--- Comment #14 from Aleisha Amohia <[email protected]> --- (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. _______________________________________________ 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/
