@@ -, +, @@ reimplementing it 1) Make that a warning is being shown with and without this patch when placing a hold to an expired patron account --- .../intranet-tmpl/prog/en/modules/reserve/request.tt | 6 +++--- reserve/request.pl | 9 --------- 2 files changed, 3 insertions(+), 12 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -385,10 +385,10 @@ [% END # /IF ( exceeded_maxreserves || ... %] - [% IF ( expiry || diffbranch || patron.is_debarred || ( amount_outstanding && Koha.Preference('maxoutstanding') && amount_outstanding > Koha.Preference('maxoutstanding') ) ) %] + [% IF ( patron.is_expired || diffbranch || patron.is_debarred || ( amount_outstanding && Koha.Preference('maxoutstanding') && amount_outstanding > Koha.Preference('maxoutstanding') ) ) %]
- [% END # /IF expiry || diffbranch ... %] + [% END # /IF patron.is_expired || diffbranch ... %] [% IF ( messageborrower ) %]
--- a/reserve/request.pl +++ a/reserve/request.pl @@ -204,14 +204,6 @@ if ($borrowernumber_hold && !$action) { ); } - # we check the date expiry of the borrower (only if there is an expiry date, otherwise, set to 1 (warn) - my $expiry_date = $patron->dateexpiry; - my $expiry = 0; # flag set if patron account has expired - if ($expiry_date and - Date_to_Days(split /-/,$date) > Date_to_Days(split /-/,$expiry_date)) { - $expiry = 1; - } - # check if the borrower make the reserv in a different branch if ( $patron->branchcode ne C4::Context->userenv->{'branch'} ) { $diffbranch = 1; @@ -220,7 +212,6 @@ if ($borrowernumber_hold && !$action) { my $amount_outstanding = $patron->account->balance; $template->param( patron => $patron, - expiry => $expiry, diffbranch => $diffbranch, messages => $messages, warnings => $warnings, --