Jay Blanchard wrote:
[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.
At one point he mentioned changing ValidateString() to return false on
an error.
At least that is what I thought he said.
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
