Why this Is not working?

--Irvin
--This is a test version of the script

--You can edit these variables.

waitTeam = "Bright green" -- The team for the waiting room.

waitTime = 30 -- The time spent in the waiting room before battle.

battleTeam = "Bright yellow" -- The team for the battle arena.

battleTime = 180 -- The amount of time the battle will last.

winBonus = 5 --The KO bonus given to the last man standing.

tieBonus = 1 --If more than 1 player is left standing, then this will be added to the standing player's KOs.

--Only I should edit things beyond this point.

battle = false

function onDiedInBattle(humanoid, player)
	local stats = player:findFirstChild("leaderstats")
	if stats ~= nil and battle==true then
		local deaths = stats:findFirstChild("Wipeouts")
		deaths.Value = deaths.Value + 1

		player.TeamColor = BrickColor.new(waitTeam)
		print("Player died")

		-- do short dance to try and find the killer

		local killer = getKillerOfHumanoidIfStillInGame(humanoid)

		handleKillCount(humanoid, player)
		local playersStillInBattle = {}
		for i,child in pairs(game.Players:getChildren()) do
			if child.TeamColor == BrickColor.new(battleTeam) then
				table.insert(playersStillInBattle, child)
			end
		end
		if #playersStillInBattle <= 1 then
			battle = false
			message("All players have been eliminated!", 2)
			nameWinner()
			changePlayersToTeamColor(waitTeam)
			killAllPlayers()
		end
	end
end

function getKillerOfHumanoidIfStillInGame(humanoid)
	-- returns the player object that killed this humanoid
	-- returns nil if the killer is no longer in the game

	-- check for kill tag on humanoid - may be more than one - todo: deal with this
	local tag = humanoid:findFirstChild("creator")

	-- find player with name on tag
	if tag ~= nil then
		
		local killer = tag.Value
		if killer.Parent ~= nil then -- killer still in game
			return killer
		end
	end

	return nil
end

function handleKillCount(humanoid, player)
	local killer = getKillerOfHumanoidIfStillInGame(humanoid)
	if killer ~= nil then
		local stats = killer:findFirstChild("leaderstats")
		if stats ~= nil then
			local kills = stats:findFirstChild("KOs")
			if killer ~= player then
				kills.Value = kills.Value + 1
				
			else
				kills.Value = kills.Value - 1
				
			end
		end
	end
end


-----------------------------------------------


function onPlayerEntered(newPlayer)
		wait()

		local kills = Instance.new("IntValue")
		kills.Name = "KOs"
		kills.Value = 0

		local deaths = Instance.new("IntValue")
		deaths.Name = "Wipeouts"
		deaths.Value = 0

		-- VERY UGLY HACK
		-- Will this leak threads?
		-- Is the problem even what I think it is (player arrived before character)?
		while true do
			if newPlayer.Character ~= nil then break end
			wait(5)
		end

		newPlayer.Changed:connect(
			function(property) 
				if property == "Character" and newPlayer.TeamColor ~= BrickColor.new(battleTeam) then
					print("Player spawned, removing tools")
					for i,tool in pairs(newPlayer.Backpack:getChildren()) do
						if tool.className == "Tool" or tool.className == "HopperBin" then
							tool:remove()
						end
					end
				end
			end
		)
			
		if newPlayer:findFirstChild("leaderstats") == nil then
			local stats = Instance.new("IntValue")
			stats.Name = "leaderstats"
			kills.Parent = stats
			deaths.Parent = stats
			stats.Parent = newPlayer
		else
			kills.Parent = newPlayer.leaderstats
			deaths.Parent = newPlayer.leaderstats
		end

		newPlayer.TeamColor = BrickColor.new(waitTeam)

		for i,tool in pairs(newPlayer.Backpack:getChildren()) do
			if tool.className == "Tool" or tool.className == "HopperBin" then
				tool:remove()
			end
		end
end

game.Players.ChildAdded:connect(onPlayerEntered)


function changePlayersToTeamColor(color)
	for i, player in pairs(game.Players:getChildren()) do
		if player.Neutral == true then
			player.Neutral = false
		end
		player.TeamColor = BrickColor.new(color)
	end
end

function killAllPlayers()
	for i, player in pairs(game.Players:getChildren()) do
		player.Character.Humanoid.Health = 0
	end
end

function beginBattle()
	for i, player in pairs(game.Players:getChildren()) do
		if player.Character.Humanoid.Health <= 0 then
			while true do
				if player.Character ~= nil then
					if player.Character.Humanoid.Health >= 0 then break end
				end
				wait()
			end
		end
		print(player.Name.." is now ready for battle.")
		player.Character.Humanoid.Died:connect(function() onDiedInBattle(player.Character.Humanoid, player) end)
	end
end

function nameWinner()
	local winners = {}
	function getWinners()
		for i,player in pairs(game.Players:getChildren()) do
			if player.TeamColor == BrickColor.new(battleTeam) then
				table.insert(winners, player)
			end
		end
	end
	getWinners()
	if #winners == 1 then
		winners[1].leaderstats.KOs.Value = winners[1].leaderstats.KOs.Value + winBonus
	elseif #winners > 1 then
		for i=1, #winners do
			winners[i].leaderstats.KOs.Value = winners[i].leaderstats.KOs.Value + tieBonus
		end
	end
	if #winners == 1 then
		message("The winner of this game was "..winners[1].Name..".", 3)
	elseif #winners == 0 then
		message("Everyone died in this game.", 3)
	elseif #winners == 2 then
		message("The game was tied between "..winners[1].Name.." and "..winners[2].Name..".", 3)
	elseif #winners > 2 then
		string = "This game was tied between "
		for i=1, #winners - 1 do
			string = string..winners[i].Name..", "
		end
		string = string.."and "..winners[#winners].Name.."."
		message(string, 3)
	end
end

while true do
	message("Game will begin in "..waitTime.." seconds!" , 3)
	wait(waitTime)
	changePlayersToTeamColor(battleTeam)
	killAllPlayers()
	message("Let the battle begin!", 3)
	message("The battle will last "..(battleTime/60).." minutes!", 2)
	wait(0.25)
	battle = true
	beginBattle()
	local btime = 0
	while battle == true do
		btime = btime + 1
		if btime >= battleTime or not battle then break end
		wait(1)
	end
	print(btime)
	if battle == true then
		battle = false
		message("The battle is now over.", 2)
		nameWinner()
		changePlayersToTeamColor(waitTeam)
		killAllPlayers()
	else
		wait(2)
	end
	wait(5.1)
end

Both Colors are set to AutoAssignable when a player joins the green color is in the lobby and yellow is spread across the map but for some reason player 1 goes to the lobby and player 2 spawns in the map already I have tested this in Roblox Studio but it wont work I tried to set yellow team AutoAssignable to false and player 1 and 2 will spawn at the lobby but they wont change their positions to the game after 30 seconds they both stay in the lobby I dont know what else to try please help ??