Attachment 107835 Details for Bug 16112 – Bug 16112: Add the ability to define due date for batch renew

[patch] Bug 16112: Add the ability to define due date for batch renew

Bug-16112-Add-the-ability-to-define-due-date-for-b.patch (text/plain), 3.38 KB, created by PTFS Europe Sandboxes on 2020-08-05 15:24:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: PTFS Europe Sandboxes

Created: 2020-08-05 15:24:43 UTC

Size: 3.38 KB

patch

obsolete

>From 1889bfbd67e8a314752251acbbcbd74f12a4a6a3 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 4 Aug 2020 17:16:56 +0200
>Subject: [PATCH] Bug 16112: Add the ability to define due date for batch renew
>
>Same as bug 16748, for batch renew.
>
>Test plan:
>Check an item in
>Renew it using the circ/renew.pl page
>=> You can pick a due date!
>=> The date is kept from one renew to another
>Renew it until you reach the maximum number of renewals
>=> You get a warning with "continue without renewing"
>=> The date is still kept
>
>Signed-off-by: Henry Bolshaw <bolshawh@parliament.uk>
>---
> circ/renew.pl                                         | 5 +++++
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt | 2 ++
> 2 files changed, 7 insertions(+)
>
>diff --git a/circ/renew.pl b/circ/renew.pl
>index c3f1d2a4b3..fb513806b4 100755
>--- a/circ/renew.pl
>+++ b/circ/renew.pl
>@@ -48,6 +48,7 @@ $barcode =~ s/^\s*|\s*$//g; # remove leading/trailing whitespae
> $barcode = barcodedecode($barcode) if( $barcode && C4::Context->preference('itemBarcodeInputFilter'));
> my $override_limit = $cgi->param('override_limit');
> my $override_holds = $cgi->param('override_holds');
>+my $hard_due_date  = $cgi->param('hard_due_date');
> 
> my ( $item, $issue, $borrower );
> my $error = q{};
>@@ -99,6 +100,9 @@ if ($barcode) {
>                     if ( $cgi->param('renewonholdduedate') ) {
>                         $date_due = dt_from_string( scalar $cgi->param('renewonholdduedate'));
>                     }
>+                    if ( C4::Context->preference('SpecifyDueDate') && $hard_due_date ) {
>+                        $date_due = dt_from_string( $hard_due_date );
>+                    }
>                     $date_due = AddRenewal( undef, $item->itemnumber(), $branchcode, $date_due );
>                     $template->param( date_due => $date_due );
>                 }
>@@ -125,6 +129,7 @@ if ($barcode) {
>     );
> }
> 
>+$template->param( hard_due_date => ($hard_due_date ? output_pref({ str => $hard_due_date, dateformat => 'iso' }) : undef) );
> # Checking if there is a Fast Cataloging Framework
> $template->param( fast_cataloging => 1 ) if Koha::BiblioFrameworks->find( 'FA' );
> 
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt
>index b45c0394db..d9d335e828 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt
>@@ -144,6 +144,7 @@
>                             [% END %]
> 
>                                 <form method="get" action="/cgi-bin/koha/circ/renew.pl">
>+                                    <input type="hidden" name="hard_due_date" value="[% hard_due_date | html %]" />
>                                     <button type="submit" class="deny"><i class="fa fa-times"></i> Continue without renewing</button>
>                                 </form>
>                     </div>
>@@ -171,6 +172,7 @@
>                         [% IF Koha.Preference('SpecifyDueDate') %]
>                             <label for="hard_due_date">Hard due date [% INCLUDE 'date-format.inc' %]:</label>
>                             <input type="text" size="20" id="hard_due_date" name="hard_due_date" value="[% hard_due_date | $KohaDates with_hours => 1 %]" />
>+                            <br/>
>                         [% END %]
> 
>                         <label for="barcode">Enter item barcode: </label>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 16112: 107786 | 107835 | 107865 | 107890 | 107891 | 108639 | 108640 | 108641 | 108658 | 108659 | 108661 | 108662 | 108663 | 108665 | 108776 | 108871 | 108872 | 108873