RE: [PHP] One last try at this!

From: Date: Wed, 17 Jan 2007 19:51:04 +0000
Subject: RE: [PHP] One last try at this!
Groups: php.general 
Request: Send a blank email to php-general+get-247258@lists.php.net to get a copy of this message
[snip] > The second condition of each if statement does not contain equality > checking, it sets the $result to ValidateString($event, "2"). That > should be > if($result == ValidateString($event, "2")) or > if($result === ValidateString($event, "2")) > > What if the intension was to fail if the result of ValidateString() was false?? Then it should be writen as such. if ( ( $result = ValidateString($event, "2") ) !== FALSE ) { $formerror['event'] = $result; } [/snip] Hmmm.....did you test that? $result = ValidateString($event, "2") is always TRUE, $result has had something assigned to it unless the function ValidateString returns a FALSE. The OP never provided the code for the function so we just don't know. Also, the OP forgot to run subsequent conditions in an elseif check, instead he used 3 if statements that will be executed each time.

« previous php.general (#247258) next »