From 54dee5b17c809d447b1da463ccd3b3ef80ea3219 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 16 May 2019 16:55:16 +0100 Subject: [PATCH] Bug 22917: Fix a typo in an output_pref call Content-Type: text/plain; charset=utf-8 Due to a type in a call to output_pref (datenonly vs dateonly) we were comparing logactions within a minutes accuracy. Counting the logs for the day would be sufficient for the test ;) Signed-off-by: Jonathan Druart Signed-off-by: Marcel de Rooy --- t/db_dependent/Circulation.t | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index 37ab7a57b7..54b3c78eaa 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -3000,7 +3000,7 @@ subtest 'AddRenewal and AddIssuingCharge tests' => sub { # Check the item out AddIssue( $patron->unblessed, $item->barcode ); t::lib::Mocks::mock_preference( 'RenewalLog', 0 ); - my $date = output_pref( { dt => dt_from_string(), datenonly => 1, dateformat => 'iso' } ); + my $date = output_pref( { dt => dt_from_string(), dateonly => 1, dateformat => 'iso' } ); my %params_renewal = ( timestamp => { -like => $date . "%" }, module => "CIRCULATION", @@ -3016,7 +3016,7 @@ subtest 'AddRenewal and AddIssuingCharge tests' => sub { unlike ( $checkouts->next->lastreneweddate, qr/00:00:00/, 'AddRenewal should set the renewal date with the time part'); t::lib::Mocks::mock_preference( 'RenewalLog', 1 ); - $date = output_pref( { dt => dt_from_string(), datenonly => 1, dateformat => 'iso' } ); + $date = output_pref( { dt => dt_from_string(), dateonly => 1, dateformat => 'iso' } ); $old_log_size = Koha::ActionLogs->count( \%params_renewal ); AddRenewal( $patron->id, $item->id, $library->id ); $new_log_size = Koha::ActionLogs->count( \%params_renewal ); -- 2.11.0