RE: [PHP] One last try at this!
| From: | Jay Blanchard | 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.