> > You should replace:
> >
> > $expire  = $dbh->selectrow_array($sqlquery) or dbError();
> >
> > with 2 lines:
> >
> > $expire = $dbh->selectrow_array($sqlquery);
> > dbError() unless defined($expire);
> 
> 
> What happens if there is something returned, but the first 
> column is NULL,
> which will set $expire to undef?  This will not produce the 
> wanted results,

Mike's actual SQL was a select count(*) statement, which must return
results (under non-error circumstances), so I believe my particular
example remains justified. Should your SQL statement actually request
row data, you are absolutely correct...

Steve

Reply via email to