http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6487

--- Comment #1 from Owen Leonard <oleon...@myacpl.org> 2011-06-09 14:26:56 UTC 
---
Created attachment 4437
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4437
Proposed fix

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.

To test, modify a patron record in mysql to give it a null expiration date.
When you try to check out to that patron you should see an error message:

"Cannot check out!

    * Expiration: Patron's card has expired. Renew or Edit Details"

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to