> > You get this warning because $action is undefined. Comparing undefined value > with something is usually unwanted, so it produces a warning. > > You could rewrite it to: > if (defined($action) && ($action eq 'add')) { > > or assign something meaningful to $action before you use it. > > why is $action undefined? i use my $action; earlier in the script and i thought that the statement $action = $formdata{action}; would take care of assigning a value to $action so that it would not be undefined. thanks for your time and input. -charles