[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")) [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