https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25193
--- Comment #15 from Jonathan Druart <[email protected]> --- (In reply to Aleisha Amohia from comment #14) > (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? My concern was about the other holds a patron could have on the same bibliographic record. I don't know if it makes sense but, reading the code: 2826 if ( C4::Context->preference('AllowRenewalIfOtherItemsAvailableLocation') eq 'holdbranch' ){ 2827 my $res = Koha::Holds->search({ biblionumber => $item->biblionumber, borrowernumber => $patron->borrowernumber })->next; 2828 next unless $item->holdingbranch eq $res->branchcode; 2829 } We skip the patron if they have their first hold on this biblio in the holding library of the item. Should not we do this test on all the holds this patron has on the bibliographic record? Katrin, 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/
