From 05b964ae20dd5edcc08b33bc02bcdcf78020dd69 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Thu, 23 Feb 2012 09:46:32 +0100 Subject: [PATCH] [SIGNED-OFF] 7453 Followup for borrower without expiry date While it should be impossible now to have a patron without expiry date, I still had one. It does not hurt to add a simple check on expiry date defined before calling delta days on null values and breaking the opac login. Trivial change. Signed-off-by: Marcel de Rooy Signed-off-by: Katrin Fischer Tested by deleting the expirydate from my own patron record. Missing expiry date now no longer breaks the summary page in the patron account on OPAC. --- opac/opac-user.pl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/opac/opac-user.pl b/opac/opac-user.pl index ff1fecf..a88bffb 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -124,7 +124,7 @@ my @bordat; $bordat[0] = $borr; # Warningdate is the date that the warning starts appearing -if ( C4::Context->preference('NotifyBorrowerDeparture') && +if ( $borr->{dateexpiry} && C4::Context->preference('NotifyBorrowerDeparture') && Date_to_Days(Add_Delta_Days($warning_year,$warning_month,$warning_day,- C4::Context->preference('NotifyBorrowerDeparture'))) < Date_to_Days( $today_year, $today_month, $today_day ) ) { -- 1.7.5.4