Re: One last try at this!

[Jim Lucas <lists@xxxxxxxxx>]

Jay Blanchard wrote:
> [snip]
> if(empty($orgname)) { $formerror['orgname'] = "Optional"; }
> elseif($result = ValidateString($orgname, "2")) { $formerror['orgname']
> =
> $result; }
> 	
> if(empty($website)) { $formerror['website'] = "Optional"; }
> if($result = ValidateString($website, "2")) { $formerror['website'] =
> $result; }
>
> if(empty($event)) { $formerror['event'] = "Optional"; }
> if($result = ValidateString($event, "2")) { $formerror['event'] =
> $result; }
> [/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;
}

Jim Lucas
> [snip]
> Again, sorry to be a pain, but I just don't get it.
> [/snip]
>
> No problem Grasshopper
>
>   

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