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

(-)a/Koha/Recalls.pm (-2 / +7 lines)
Lines 157-164 sub add_recall { Link Here
157
                second => $checkout_due_date->second
157
                second => $checkout_due_date->second
158
            }
158
            }
159
        )->add( days => $due_interval );
159
        )->add( days => $due_interval );
160
        $checkout->update( { date_due => $recall_due_date } )
160
161
            if DateTime->compare( $recall_due_date, $checkout_due_date ) == -1;
161
        # update checkout due date if recall due date is sooner
162
        if ( DateTime->compare( $recall_due_date, $checkout_due_date ) == -1 ) {
163
            $checkout->update( { date_due => $recall_due_date } );
164
            $checkout->item->onloan( $recall_due_date->ymd() );
165
            $checkout->item->store( { log_action => 0, skip_holds_queue => 1 } );
166
        }
162
167
163
        # get itemnumber of most relevant checkout if a biblio-level recall
168
        # get itemnumber of most relevant checkout if a biblio-level recall
164
        unless ( $recall->item_level ) { $itemnumber = $checkout->itemnumber; }
169
        unless ( $recall->item_level ) { $itemnumber = $checkout->itemnumber; }
(-)a/t/db_dependent/Koha/Recalls.t (-2 / +7 lines)
Lines 18-24 Link Here
18
use Modern::Perl;
18
use Modern::Perl;
19
19
20
use Test::NoWarnings;
20
use Test::NoWarnings;
21
use Test::More tests => 24;
21
use Test::More tests => 25;
22
use t::lib::Dates;
22
use t::lib::Dates;
23
use t::lib::TestBuilder;
23
use t::lib::TestBuilder;
24
use t::lib::Mocks;
24
use t::lib::Mocks;
Lines 141-146 ok( Link Here
141
    $checkout_due_date_after_recall lt $initial_checkout_due_date,
141
    $checkout_due_date_after_recall lt $initial_checkout_due_date,
142
    "Checkout due date has been moved backwards"
142
    "Checkout due date has been moved backwards"
143
);
143
);
144
$item2                          = Koha::Items->find( $item2->itemnumber );
145
$checkout_due_date_after_recall = dt_from_string($checkout_due_date_after_recall);
146
is(
147
    t::lib::Dates::compare( $checkout_due_date_after_recall->ymd(), $item2->onloan ), 0,
148
    "items.onloan is updated with adjusted due date"
149
);
144
150
145
Koha::CirculationRules->set_rule(
151
Koha::CirculationRules->set_rule(
146
    {
152
    {
147
- 

Return to bug 39985