From 28b73a293efd0a445aee2b6c879e417f7779b327 Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Thu, 14 Aug 2008 11:13:50 -0500 Subject: [PATCH] bug 2502: correctly check patron account expiration When placing a hold request from the staff interface, now correctly indicate if the patron account has expired. Prior to this patch, would warn that the account had expired even when the expiry date is in the future, and vice versa. Signed-off-by: Galen Charlton --- reserve/request.pl | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/reserve/request.pl b/reserve/request.pl index eb41244959..a2abe4aad3 100755 --- a/reserve/request.pl +++ b/reserve/request.pl @@ -110,9 +110,10 @@ if ($cardnumber) { } # we check the date expiry of the borrower (only if there is an expiry date, otherwise, set to 1 (warn) - my $expiry = $borrowerinfo->{dateexpiry}; - unless ($expiry and $expiry ne '0000-00-00' and - Date_to_Days(split /-/,$date) > Date_to_Days(split /-/,$expiry)) { + my $expiry_date = $borrowerinfo->{dateexpiry}; + my $expiry = 0; # flag set if patron account has expired + if ($expiry_date and $expiry_date ne '0000-00-00' and + Date_to_Days(split /-/,$date) > Date_to_Days(split /-/,$expiry_date)) { $messages = $expiry = 1; } -- 2.39.5