Another quick question... what's wrong with this?

Code:
 on *:TEXT:`SScrim:#WireScrim,#test:{
  if ($nick == %Player1 || $nick == %Player2 || $nick == %Player3 || $nick == %Player4 || $nick == %Player5) { /notice $nick You're already on the group for this scrim | halt }
  if (%Player1 == $null) { set $nick %Player1 | notice $nick You'll be in the roster of this scrim, stand by for ip/pw... | halt }
}