Attachment 100614 Details for Bug 24159 – Bug 24159: Use days_mode circ rule to calculate the dropbox date

[patch] Bug 24159: Use days_mode circ rule to calculate the dropbox date

Bug-24159-Use-daysmode-circ-rule-to-calculate-the-.patch (text/plain), 1.39 KB, created by Jonathan Druart on 2020-03-12 08:10:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2020-03-12 08:10:15 UTC

Size: 1.39 KB

patch

obsolete

>From 17c0e838d8e5b9f003d632dffdc73532a901582e Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Thu, 12 Mar 2020 08:46:59 +0100
>Subject: [PATCH] Bug 24159: Use days_mode circ rule to calculate the dropbox
> date
>
>Calculate the drop box date depending on the circ rule. Note that only
>the branchcode is taken into account here. Is that correct?
>
>Test plan:
>Use the drop box date feature and confirm that it takes into account the
>calendar, depending on the circ rule.
>---
> Koha/Checkouts.pm | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
>diff --git a/Koha/Checkouts.pm b/Koha/Checkouts.pm
>index ccfeb36e10..cad363d6e9 100644
>--- a/Koha/Checkouts.pm
>+++ b/Koha/Checkouts.pm
>@@ -47,7 +47,14 @@ sub calculate_dropbox_date {
>     my $userenv    = C4::Context->userenv;
>     my $branchcode = $userenv->{branch} // q{};
> 
>-    my $calendar = Koha::Calendar->new( branchcode => $branchcode );
>+    my $useDaysMode_value = Koha::CirculationRules->get_useDaysMode_effective_value(
>+        {
>+            categorycode => undef,
>+            itemtype     => undef,
>+            branchcode   => $branchcode,
>+        }
>+    );
>+    my $calendar     = Koha::Calendar->new( branchcode => $branchcode, days_mode => $useDaysMode_value );
>     my $today        = DateTime->now( time_zone => C4::Context->tz() );
>     my $dropbox_date = $calendar->addDate( $today, -1 );
> 
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 24159: 99758 | 99759 | 99760 | 99761 | 99762 | 100608 | 100609 | 100610 | 100611 | 100612 | 100613 | 100614 | 100615 | 101053 | 101054 | 101556 | 101649 | 101698 | 102009 | 102148 | 102303 | 102554 | 102555 | 102556 | 102557 | 102558 | 102559 | 102560 | 102561 | 102562 | 102563 | 104197 | 104198 | 104199 | 104200 | 104201 | 104202 | 104203 | 104204 | 106007 | 106008 | 106013 | 106014 | 106015 | 106016 | 106017 | 106018 | 106019 | 106020 | 106021 | 106022