[Koha-patches] [PATCH] Bug 3420: Follow-up patch for 'patrons with no checkouts'

Galen Charlton gmcharlt at gmail.com
Mon Aug 17 14:14:57 CEST 2009


Hi,

On Sat, Aug 15, 2009 at 4:24 PM, Garry Collum<gcollum at gmail.com> wrote:
>             foreach my $cell (@$x) {
> -                print $cell->{value}.$sep;
> +                print $cell->{value}.$sep if $cell->{value};
>             }

This doesn't seem quite right - what if $cell->{value} is legitimately
blank or zero?  Even if it is undef, I assume that an empty string
should be included in the report output.

Regards,

Galen
-- 
Galen Charlton
gmcharlt at gmail.com



More information about the Koha-patches mailing list