Hi,

On Fri, Mar 6, 2009 at 5:38 AM,  <[email protected]> wrote:

This patch has no functional effect.  Adding one more line of context shows why:

>    my $expiry = 0; # flag set if patron account has expired
>     if ($expiry_date and $expiry_date ne '0000-00-00' and
>             Date_to_Days(split /-/,$date) > Date_to_Days(split 
> /-/,$expiry_date)) {
>                $messages = $expiry = 1;
> +    }else{
> +        $expiry = 0;
>     }
> -
> +

If $expiry is already initialized to 0, the else block has no effect.
Furthermore, I'm not able to reproduce the buggy behavior in HEAD.

Regards,

Galen
-- 
Galen Charlton
VP, Research & Development, LibLime
[email protected]
p: 1-888-564-2457 x709
skype: gmcharlt
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to