http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=1459 jmf@liblime.com changed: What |Removed |Added ---------------------------------------------------------------------------- Platform|Macintosh |All ------- Comment #2 from jmf@liblime.com 2007-10-05 17:46 ------- The solution proposed is the following: + if ( $borrower->{'dateexpiry'} eq '0000-00-00') { $issuingimpossible{EXPIRED} = 1; + } else { + if ( Date_to_Days(Today) > + Date_to_Days( split "-", $borrower->{'dateexpiry'} ) ) + { + $issuingimpossible{EXPIRED} = 1; + } This works when the patron's expiry date is set to 0000-00-00, so I've accepte the patch, but that's a nonsensical date, and in most cases, since the expirydate defaults to NULL, the date will be null. We need to handle the case of a null date, perhaps on the fly assigning a date based on the date the patron's record was created? ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.