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

(-)a/C4/Reserves.pm (-2 / +2 lines)
Lines 210-216 sub AddReserve { Link Here
210
            waitingdate    => $waitingdate,
210
            waitingdate    => $waitingdate,
211
            expirationdate => $expdate,
211
            expirationdate => $expdate,
212
            itemtype       => $itemtype,
212
            itemtype       => $itemtype,
213
            item_level_hold => $checkitem ? 1 : 0,
213
            item_level     => $checkitem ? 1 : 0,
214
        }
214
        }
215
    )->store();
215
    )->store();
216
    $hold->set_waiting() if $found eq 'W';
216
    $hold->set_waiting() if $found eq 'W';
Lines 1924-1930 sub RevertWaitingStatus { Link Here
1924
    $sth = $dbh->prepare( $query );
1924
    $sth = $dbh->prepare( $query );
1925
    $sth->execute( $reserve->{'reserve_id'} );
1925
    $sth->execute( $reserve->{'reserve_id'} );
1926
1926
1927
    unless ( $hold->item_level_hold ) {
1927
    unless ( $hold->item_level ) {
1928
        $hold->itemnumber(undef)->store;
1928
        $hold->itemnumber(undef)->store;
1929
    }
1929
    }
1930
1930
(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc (-1 / +1 lines)
Lines 142-148 Link Here
142
                        [% END %]
142
                        [% END %]
143
                    </a>
143
                    </a>
144
                [% ELSE %]
144
                [% ELSE %]
145
                    [% IF ( hold.item_level_hold ) %]
145
                    [% IF ( hold.item_level ) %]
146
                        <i>
146
                        <i>
147
                            Only item
147
                            Only item
148
                            <a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% hold.biblionumber | uri %]&amp;itemnumber=[% hold.itemnumber | uri %]#item[% hold.itemnumber | uri %]">
148
                            <a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% hold.biblionumber | uri %]&amp;itemnumber=[% hold.itemnumber | uri %]#item[% hold.itemnumber | uri %]">
(-)a/t/db_dependent/Reserves.t (-4 / +3 lines)
Lines 739-745 subtest 'ChargeReserveFee tests' => sub { Link Here
739
    is( $line->branchcode, $library->id , "Library id is picked from userenv and stored correctly" );
739
    is( $line->branchcode, $library->id , "Library id is picked from userenv and stored correctly" );
740
};
740
};
741
741
742
subtest 'reserves.item_level_hold' => sub {
742
subtest 'reserves.item_level' => sub {
743
    plan tests => 2;
743
    plan tests => 2;
744
744
745
    my $item   = $builder->build_sample_item;
745
    my $item   = $builder->build_sample_item;
Lines 758-764 subtest 'reserves.item_level_hold' => sub { Link Here
758
            $item->itemnumber );
758
            $item->itemnumber );
759
759
760
        my $hold = Koha::Holds->find($reserve_id);
760
        my $hold = Koha::Holds->find($reserve_id);
761
        is( $hold->item_level_hold, 1, 'item_level_hold should be set when AddReserve is called with a specific item' );
761
        is( $hold->item_level, 1, 'item_level_hold should be set when AddReserve is called with a specific item' );
762
762
763
        # Mark it waiting
763
        # Mark it waiting
764
        ModReserveAffect( $item->itemnumber, $patron->borrowernumber, 1 );
764
        ModReserveAffect( $item->itemnumber, $patron->borrowernumber, 1 );
Lines 780-786 subtest 'reserves.item_level_hold' => sub { Link Here
780
            $item->biblionumber, undef, 1 );
780
            $item->biblionumber, undef, 1 );
781
781
782
        my $hold = Koha::Holds->find($reserve_id);
782
        my $hold = Koha::Holds->find($reserve_id);
783
        is( $hold->item_level_hold, 0, 'item_level_hold should not be set when AddReserve is called without a specific item' );
783
        is( $hold->item_level, 0, 'item_level_hold should not be set when AddReserve is called without a specific item' );
784
784
785
        # Mark it waiting
785
        # Mark it waiting
786
        ModReserveAffect( $item->itemnumber, $patron->borrowernumber, 1 );
786
        ModReserveAffect( $item->itemnumber, $patron->borrowernumber, 1 );
787
- 

Return to bug 23133