@@ -, +, @@ 0 - Apply only first patch (unit tests) 1 - Place two holds for a patron on a bib 2 - Checkout that item via sip_cli_emulator 3 - Note all holds are cancelled/filled 4 - prove -v t/db_dependent/SIP/Transaction.t 5 - it fails 6 - Apply second patch 7 - prove tests, all green 8 - Place two holds on a single bib for one patron again 9 - Checkout the item via sip_cli_emulator --- C4/SIP/ILS/Transaction/Checkout.pm | 9 --------- 1 file changed, 9 deletions(-) --- a/C4/SIP/ILS/Transaction/Checkout.pm +++ a/C4/SIP/ILS/Transaction/Checkout.pm @@ -129,15 +129,6 @@ sub do_checkout { $self->ok(0); return $self; } - # Fill any reserves the patron had on the item. - # TODO: this logic should be pulled internal to AddIssue for all Koha. - $debug and warn "pending_queue: " . (@$pending) ? Dumper($pending) : '[]'; - foreach (grep {$_->{borrowernumber} eq $self->{patron}->{borrowernumber}} @$pending) { - $debug and warn "Filling reserve (borrowernumber,biblionumber,reservedate): " - . sprintf("(%s,%s,%s)\n",$_->{borrowernumber},$_->{biblionumber},$_->{reservedate}); - ModReserveFill($_); - # TODO: adjust representation in $self->item - } # can issue $debug and warn "do_checkout: calling AddIssue(\$borrower,$barcode, $overridden_duedate, 0)\n" # . "w/ \$borrower: " . Dumper($borrower) --