Patrons get incorrectly debarred because of the use of Datetime->delta_days
wich happens to always return a positive number.
Also changed the use of Datetime->truncate function so it follows the docs,
using 'day' instead of 'days' (which is used in DateTime::Duration).
This patch has been tested against master.
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
my $dateend_temp = $end_dt->clone();
# start and end should not be closed days
- my $duration = $dateend_temp->delta_days($datestart_temp);
- $datestart_temp->truncate( to => 'days' );
- $dateend_temp->truncate( to => 'days' );
+ $datestart_temp->truncate( to => 'day' );
+ $dateend_temp->truncate( to => 'day' );
+ my $duration = $dateend_temp - $datestart_temp;
while ( DateTime->compare( $datestart_temp, $dateend_temp ) == -1 ) {
$datestart_temp->add( days => 1 );
if ( $self->is_holiday($datestart_temp) ) {