<div dir="ltr"><div><div><div><div>Hi devs,<br><br></div>I have just caught that:<br><br></div>Koha::Patron->is_expired returns  0 if $self->dateexpiry eq '0000-00-00'<br></div>And there is a test I wrote for it on bug 17578 (commit 5a0a2ce584dec74808f471b0f92b2114c3d6029e , GetMemberDetails - Remove is_expired)<br></div><div><br></div><div>But then we also have, in C4::Circulation::CanBookBeIssued:<br> 752     if ( !defined $patron->dateexpiry || $patron->dateexpiry eq '0000-00-00') {<br> 753         $issuingimpossible{EXPIRED} = 1;<br> 754     } else {<br></div><div>With a test:<br>is( $issuingimpossible->{EXPIRED}, 1, 'The patron should be considered as expired if dateexpiry is 0000-00-00' );<br><br></div><div>So, my questions, do you have a very old Koha installations to search if there is borrowers.is_expired == "0000-00-00" in it?<br></div><div>Do you remember if it meant expired or not at some point?<br><br></div><div>I am trying to adapt our code to sql strict modes, and this value is no longer valid, we will need to update existing values.<br><br></div><div>Cheers,<br></div><div>Jonathan<br></div><div><br></div><div>(yes, the same code should be used)<br></div></div>