@@ -, +, @@
holds on circulation.pl
for that patron
"waiting until" part with the waiting date plus 5 days
string )
"waiting until" line.
---
Koha/Hold.pm | 25 ++++++++++
.../prog/en/modules/circ/circulation.tt | 2 +-
t/db_dependent/Hold.t | 53 ++++++++++++++++++++++
3 files changed, 79 insertions(+), 1 deletion(-)
create mode 100755 t/db_dependent/Hold.t
--- a/Koha/Hold.pm
+++ a/Koha/Hold.pm
@@ -21,9 +21,11 @@ use Modern::Perl;
use Carp;
+use C4::Context qw(preference);
use Koha::Branches;
use Koha::Biblios;
use Koha::Items;
+use Koha::DateUtils qw(dt_from_string);
use base qw(Koha::Object);
@@ -37,6 +39,29 @@ Koha::Hold - Koha Hold object class
=cut
+=head3 waiting_expires_on
+
+Returns a DateTime for the date a waiting holds expires on.
+Returns undef if the system peference ReservesMaxPickUpDelay is not set.
+Returns undef if the hold is not waiting ( found = 'W' ).
+
+=cut
+
+sub waiting_expires_on {
+ my ($self) = @_;
+
+ return unless $self->found() eq 'W';
+
+ my $ReservesMaxPickUpDelay = C4::Context->preference('ReservesMaxPickUpDelay');
+ return unless $ReservesMaxPickUpDelay;
+
+ my $dt = dt_from_string( $self->waitingdate() );
+
+ $dt->add( days => $ReservesMaxPickUpDelay );
+
+ return $dt;
+}
+
=head3 biblio
Returns the related Koha::Biblio object for this hold
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
+++ a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
@@ -775,7 +775,7 @@ No patron matched [% message %]
[% IF ( w.branch.branchcode == Branches.GetLoggedInBranchcode() ) %][% ELSE %][% END %]
- Waiting at [% w.branch.branchname | html %]
+ Waiting at [% w.branch.branchname | html %] [% IF w.waiting_expires_on %] until [% w.waiting_expires_on | $KohaDates %] [% END %]
--- a/t/db_dependent/Hold.t
+++ a/t/db_dependent/Hold.t
@@ -0,0 +1,53 @@
+#!/usr/bin/perl
+
+# This file is part of Koha.
+#
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see .
+
+use Modern::Perl;
+
+use C4::Context;
+use Koha::Database;
+
+use Test::More tests => 5;
+
+use_ok('Koha::Hold');
+
+my $schema = Koha::Database->new()->schema();
+$schema->storage->txn_begin();
+
+my $dbh = C4::Context->dbh;
+$dbh->{RaiseError} = 1;
+
+my $hold = Koha::Hold->new({ found => 'W', waitingdate => '2000-01-01'});
+
+C4::Context->set_preference( 'ReservesMaxPickUpDelay', '' );
+my $dt = $hold->waiting_expires_on();
+is( $dt, undef, "Koha::Hold->waiting_expires_on returns undef if ReservesMaxPickUpDelay is not set");
+
+C4::Context->set_preference( 'ReservesMaxPickUpDelay', '5' );
+$dt = $hold->waiting_expires_on();
+is( $dt->ymd, "2000-01-06", "Koha::Hold->waiting_expires_on returns DateTime of waitingdate + ReservesMaxPickUpDelay if set");
+
+$hold->found('T');
+$dt = $hold->waiting_expires_on();
+is( $dt, undef, "Koha::Hold->waiting_expires_on returns undef if found is not 'W' ( Set to 'T' )");
+
+$hold->found(q{});
+$dt = $hold->waiting_expires_on();
+is( $dt, undef, "Koha::Hold->waiting_expires_on returns undef if found is not 'W' ( Set to empty string )");
+
+$schema->storage->txn_rollback();
+
+1;
--