if (!$s_user = $REMOTE_USER) $s_user="error";
is a correct statement. It assignes $s_user the value $REMOTE_USER. If $REMOTE_USER is null then statement is false. It's a C thing and sometimes useful and at other times really confusing when you don't mean to do that.