[Koha-bugs] [Bug 6487] No error explanation if patron expiration date is missing

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Thu Jun 9 16:26:56 CEST 2011


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

--- Comment #1 from Owen Leonard <oleonard at 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.


More information about the Koha-bugs mailing list