[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. 

--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to