Fix for Bug 6487 - No error explanation if patron expiration date is missing
authorOwen Leonard <oleonard@myacpl.org>
Wed, 8 Jun 2011 14:07:32 +0000 (10:07 -0400)
committerChris Cormack <chrisc@catalyst.net.nz>
Thu, 9 Jun 2011 22:49:43 +0000 (10:49 +1200)
commit84a8c9a52222355dc298d00164f363f37e9a400c
tree8b5f6dc7b720522aad73a8428f81afd7a95d10c2
parent716442708b199f07a85df75434b1fa99d9ccaf0f
Fix for Bug 6487 - No error explanation if patron expiration date is missing

The script tries to parse the patron expiration date without
checking that it is a valid date. If expiration date is empty
it doesn't get caught along with past expiration dates.

I wonder if this line was supposed to catch this problem:

Line 233: if ( $warning_year*$warning_month*$warning_day==0

This patch adds "!$borrower->{'dateexpiry'} ||" to that line.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
circ/circulation.pl
koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt