My teleport script doesn't work?

So, I made this.. All of it works in studio testing, but when I try ingame heres what happens: The Gui's text changing works (The button saying stuff like: "Teleporting to base in: 3" that for example), the force field on the teleport to base works, anchoring the player and unanchoring the player works... The problem is the teleport part... It doesn't work, it doesn't bring the player where it's ment to go, it doesn't do anything..... Just leave the player where it is, umm but it still works in studio testing..? :/ Nothing in output.. Help me?...??.. :c

PlayerInBattle = script.Parent.PlayerInBattle

function clicked()
	local battletarget1 = CFrame.new(42.983, 14.6, -49.147)
	local battletarget2 = CFrame.new(-40.468, 14.6, -49.133)
	local battletarget3 = CFrame.new(-42.749, 14.6, 41.207)
	local battletarget4 = CFrame.new(40.703, 14.6, 41.193)
	local spawntarget = CFrame.new(189.87, 69.6, -216.3)
	local player = script.Parent.Parent.Parent
	local Torso = player.Parent.Character:WaitForChild ("Torso")
	if PlayerInBattle.Value == false then
		script.Parent.Text = "Entering battle in: 3"
		Torso.Anchored = true
		wait(1)
		script.Parent.Text = "Entering battle in: 2"
		wait(1)
		script.Parent.Text = "Entering battle in: 1"
		wait(1)
		local randombattlespawn = math.random(1,4)
		if randombattlespawn == 1 then
			local Torso1 = player.Parent.Character:WaitForChild ("Torso")
			Torso1.CFrame = battletarget1
			PlayerInBattle.Value = true
		end
		if randombattlespawn == 2 then
			local Torso1 = player.Parent.Character:WaitForChild ("Torso")
			Torso1.CFrame = battletarget2
			PlayerInBattle.Value = true
		end
		if randombattlespawn == 3 then
			local Torso1 = player.Parent.Character:WaitForChild ("Torso")
			Torso1.CFrame = battletarget3
			PlayerInBattle.Value = true
		end
		if randombattlespawn == 4 then
			local Torso1 = player.Parent.Character:WaitForChild ("Torso")
			Torso1.CFrame = battletarget4
			PlayerInBattle.Value = true
		end
		Torso.Anchored = false
	script.Parent.Text = "Click here to teleport back to base!"
	elseif PlayerInBattle.Value == true then
		script.Parent.Text = "Teleporting to base in: 3"
		Torso.Anchored = true
		wait(1)
		script.Parent.Text = "Teleporting to base in: 2"
		local ff = Instance.new ("ForceField")
		ff.Parent = Torso.Parent
		wait(1)
		script.Parent.Text = "Teleporting to base in: 1"
		wait(1)
		PlayerInBattle.Value = false
		local Torso1 = player.Parent.Character:WaitForChild ("Torso")
		Torso1.CFrame = spawntarget
		Torso.Anchored = false
		script.Parent.Text = "Click here to teleport into battle!"
		ff:Remove()
	end
end

script.Parent.MouseButton1Click:connect(clicked)

I don't really know why your code is working.. but all your positions to teleport to are very close to eachother.. so maybe you are moving but you don't notice?

Anyways.. I'm going to suggest an alternative for selecting the teleport. If you make a table for all the locations, then index the table randomely.. it will effectively select a random location.

What i mean by indexing a table randomely;

local a = {3,2,1}
--I have a table 'a' with 3 indexes. I want to get the first index
print(a[1]) --> '3'
--Now, if we put the math.random function inside of the brackets then it will index it randomely
print(a[math.random(1,2)])
--But there are 3 indexes in the table.. so we should use '3' as the max number.
print(a[math.random(1,3)])
--[[And to just make things easier.. in case you wanna add more indexes without having to change that number then just use the '#' operator on the table. It will return how many indexes are in the table]]
print(a[math.random(1,#a)])

Now, to use this to your purposes;

PlayerInBattle = script.Parent.PlayerInBattle

function clicked()
	--Table of locations
	local battleTargets = {
		CFrame.new(42.983, 14.6, -49.147),
		CFrame.new(-40.468, 14.6, -49.133),
		CFrame.new(-42.749, 14.6, 41.207),
		CFrame.new(40.703, 14.6, 41.193)
	}
	local spawntarget = CFrame.new(189.87, 69.6, -216.3)
	local player = script.Parent.Parent.Parent
	local Torso = player.Parent.Character:WaitForChild ("Torso")
	if not PlayerInBattle.Value then
		Torso.Anchored = true
		--Count down with a for loop, eh?
		for i = 3,1,-1 do
			script.Parent.Text = "Entering battle in: "..i
			wait(1)
		end
		--Index table randomely
		local Spawn = battleTargets[math.random(1,#battleTargets)]
		--'Torso' is already defined.. no need to redefine it
		Torso.CFrame = Spawn
		Torso.Anchored = false
		script.Parent.Text = "Click here to teleport back to base!"
	--No need for elseif, just use 'else'
	else
		Torso.Anchored = true
		local ff = Instance.new('ForceField',Torso.Parent)
		--Yay for for loops c:
		for i = 3,1,-1 do
			script.Parent.Text = "Teleporting to base in: "..i
			 wait(1)
		end
		PlayerInBattle.Value = false
		Torso.CFrame = spawntarget
		Torso.Anchored = false
		script.Parent.Text = "Click here to teleport into battle!"
		wait(2)
		ff:Destroy()
	end
end

script.Parent.MouseButton1Click:connect(clicked)