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

(-)a/C4/Reserves.pm (-2 / +3 lines)
Lines 854-861 sub ModReserve { Link Here
854
854
855
    my $hold;
855
    my $hold;
856
    unless ( $reserve_id ) {
856
    unless ( $reserve_id ) {
857
        $hold = Koha::Holds->search({ biblionumber => $biblionumber, borrowernumber => $borrowernumber, itemnumber => $itemnumber });
857
        my $holds = Koha::Holds->search({ biblionumber => $biblionumber, borrowernumber => $borrowernumber, itemnumber => $itemnumber });
858
        return unless $hold; # FIXME Should raise an exception
858
        return unless $holds->count; # FIXME Should raise an exception
859
        $hold = $holds->next;
859
        $reserve_id = $hold->reserve_id;
860
        $reserve_id = $hold->reserve_id;
860
    }
861
    }
861
862
(-)a/t/db_dependent/Holds.t (-2 / +10 lines)
Lines 7-13 use t::lib::TestBuilder; Link Here
7
7
8
use C4::Context;
8
use C4::Context;
9
9
10
use Test::More tests => 54;
10
use Test::More tests => 55;
11
use MARC::Record;
11
use MARC::Record;
12
use C4::Biblio;
12
use C4::Biblio;
13
use C4::Items;
13
use C4::Items;
Lines 147-152 ok( $hold->priority eq '4', "Test ModReserve, priority changed correctly" ); Link Here
147
ok( $hold->suspend, "Test ModReserve, suspend hold" );
147
ok( $hold->suspend, "Test ModReserve, suspend hold" );
148
is( $hold->suspend_until, '2013-01-01 00:00:00', "Test ModReserve, suspend until date" );
148
is( $hold->suspend_until, '2013-01-01 00:00:00', "Test ModReserve, suspend until date" );
149
149
150
ModReserve({ # call without reserve_id
151
    rank          => '3',
152
    biblionumber  => $item_bibnum,
153
    itemnumber    => $itemnumber,
154
    borrowernumber => $borrowernumber,
155
});
156
$hold = Koha::Holds->find( $reserve_id );
157
ok( $hold->priority eq '3', "Test ModReserve, priority changed correctly" );
158
150
ToggleSuspend( $reserve_id );
159
ToggleSuspend( $reserve_id );
151
$hold = Koha::Holds->find( $reserve_id );
160
$hold = Koha::Holds->find( $reserve_id );
152
ok( ! $hold->suspend, "Test ToggleSuspend(), no date" );
161
ok( ! $hold->suspend, "Test ToggleSuspend(), no date" );
153
- 

Return to bug 19766