On Tue, 2005-02-22 at 11:31, Martin Norland wrote: > Bret Hughes wrote: > > On Tue, 2005-02-22 at 10:49, Matt M. wrote: > > > >>On Tue, 22 Feb 2005 10:47:43 -0600, Matt M. <h.dudeness@xxxxxxxxx> wrote: > [snip] > >>>try: > >>> > >>>if (isset($_POST['op']) && $_POST['op'] != 'ds') > >>> > >> > >>sorry for 2 emails, looks like you would want > >>if (!isset($_POST['op']) || $_POST['op'] != 'ds') > >> > > > > I vote for the first one. > > > > Bret > > Your vote is misplaced - the first one isn't what he wanted... with that > statement, his form will never be printed and it can never be set. > You are right of course, In looking at the original post I misread it to be if (! $_POST['op'] != 'ds ). I guess Since this is the sort of construct I have cleaned up the most lately. Actually most of what I have been finding is stuff like if (!$varname ){ That is, doing something if a variable is not set to a true value and also using that for a test for existence. turn on notices and get a lesson on how to make a log file grow quickly :) Bret -- PHP Database Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php