Your not getting any errors because $s->{terms} always tests true. What if one day it tests false? That is why your testing it.

    # Assign the inserted values
    $s->{insert} =  {
        all         => [ "%All%", $all ],
        any         => [ "%Any%", $any ],
        terms       => [ "%Terms%", $terms ],
        . . .
[download]

What is the best way to keep these 'uninitialized' errors from appearing? When I declare my $terms, should I set it to empty quotes or is there a better way?

Thanks for the help,

Dave Hoover
"Twice blessed is help unlooked for." --Tolkien
http://www.redsquirreldesign.com/dave

Either that or use a default value

    # Load the keywords used
    my $terms = '';
    if ($s->{terms}) {
        $terms = join ' ', @{$s->{terms}};
    }
[download]

or:

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

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


HTH,
Charles K. Clarkson