View | Details | Raw Unified | Return to bug 33791
Collapse All | Expand All

(-)a/C4/Reserves.pm (-2 / +2 lines)
Lines 2022-2028 sub MoveReserve { Link Here
2022
2022
2023
    if ($res->{borrowernumber} == $borrowernumber) {
2023
    if ($res->{borrowernumber} == $borrowernumber) {
2024
        my $hold = Koha::Holds->find( $res->{reserve_id} );
2024
        my $hold = Koha::Holds->find( $res->{reserve_id} );
2025
        $hold->fill;
2025
        $hold->fill({ itemnumber => $itemnumber });
2026
    }
2026
    }
2027
    else {
2027
    else {
2028
        # warn "Reserved";
2028
        # warn "Reserved";
Lines 2038-2044 sub MoveReserve { Link Here
2038
2038
2039
        if ( $borr_res ) {
2039
        if ( $borr_res ) {
2040
            # The item is reserved by the current patron
2040
            # The item is reserved by the current patron
2041
            $borr_res->fill;
2041
            $borr_res->fill({ itemnumber => $itemnumber });
2042
        }
2042
        }
2043
2043
2044
        if ( $cancelreserve eq 'revert' ) { ## Revert waiting reserve to priority 1
2044
        if ( $cancelreserve eq 'revert' ) { ## Revert waiting reserve to priority 1
(-)a/Koha/Hold.pm (-2 / +3 lines)
Lines 777-789 sub cancel { Link Here
777
=head3 fill
777
=head3 fill
778
778
779
    $hold->fill;
779
    $hold->fill;
780
    $hold->fill({ itemnumber => $i }); # optional itemnumber: see MoveReserves
780
781
781
This method marks the hold as filled. It effectively moves it to old_reserves.
782
This method marks the hold as filled. It effectively moves it to old_reserves.
782
783
783
=cut
784
=cut
784
785
785
sub fill {
786
sub fill {
786
    my ( $self ) = @_;
787
    my ( $self, $params ) = @_;
787
    $self->_result->result_source->schema->txn_do(
788
    $self->_result->result_source->schema->txn_do(
788
        sub {
789
        sub {
789
            my $patron = $self->patron;
790
            my $patron = $self->patron;
Lines 792-797 sub fill { Link Here
792
                {
793
                {
793
                    found    => 'F',
794
                    found    => 'F',
794
                    priority => 0,
795
                    priority => 0,
796
                    $params->{itemnumber} ? ( itemnumber => $params->{itemnumber} ) : (),
795
                }
797
                }
796
            );
798
            );
797
799
798
- 

Return to bug 33791