@@ -, +, @@ --- Koha/Hold.pm | 1 + t/db_dependent/Koha/Hold.t | 6 ++++-- 2 files changed, 5 insertions(+), 2 deletions(-) --- a/Koha/Hold.pm +++ a/Koha/Hold.pm @@ -848,6 +848,7 @@ sub fill { { found => 'F', priority => 0, + timestamp => dt_from_string->strftime( '%Y-%m-%d %H:%M:%S' ), $params->{item_id} ? ( itemnumber => $params->{item_id} ) : (), } ); --- a/t/db_dependent/Koha/Hold.t +++ a/t/db_dependent/Koha/Hold.t @@ -106,7 +106,7 @@ subtest 'pickup_library/branch tests' => sub { subtest 'fill() tests' => sub { - plan tests => 14; + plan tests => 15; $schema->storage->txn_begin; @@ -136,6 +136,7 @@ subtest 'fill() tests' => sub { biblionumber => $biblio->id, borrowernumber => $patron->id, itemnumber => $item->id, + timestamp => dt_from_string('2021-06-25 14:05:35'), priority => 10, } } @@ -148,7 +149,7 @@ subtest 'fill() tests' => sub { my $interface = 'api'; C4::Context->interface($interface); - + my $hold_timestamp = $hold->timestamp; my $ret = $hold->fill; is( ref($ret), 'Koha::Hold', '->fill returns the object type' ); @@ -159,6 +160,7 @@ subtest 'fill() tests' => sub { is( $old_hold->id, $hold->id, 'reserve_id retained' ); is( $old_hold->priority, 0, 'priority set to 0' ); + isnt( $old_hold->timestamp, $hold_timestamp, 'timestamp updated' ); is( $old_hold->found, 'F', 'found set to F' ); subtest 'item_id parameter' => sub { --