Ban indentation, inafitiant, try something like this next time, it's way more efficiant
on join:
add 1 to {joins}
set {joins::%Player%} to {joins}
if player has not played before:
set join message to "&2%Player% has joined for the first time &7[%{joins::%player%}%]"
else:
set join message to "&2%Player% has joined"
on quit:
set leave message to "&2%player% has left"