Whats wrong with this?

 function doTeam()
local playerDir = game.Players:GetPlayers();
local chosen = table.remove(playerDir, math.random(1, #playerDir)); 
chosen.TeamColor = game.Teams["Bright red Team"].TeamColor;

for index,value in next, playerDir do
value.TeamColor = game.Teams["Bright blue Team"].TeamColor;
end

end

while wait(7) do
doTeam()
end