in reply to Re: Re: Code Review Needed!
in thread Code Review Needed!

Either that or use a default value

# Load the keywords used my $terms = ''; if ($s->{terms}) { $terms = join ' ', @{$s->{terms}}; }
or:
# Load the keywords used my $terms = 'some default value'; if ($s->{terms}) { $terms = join ' ', @{$s->{terms}}; }

Or, even better might be to allow Soapbox.pm to handle undef values


HTH,
Charles K. Clarkson