Why is this script that gives you an aura only working sometimes?

I am making a script that gives you an aura but it only works sometimes. Why?

game.Players.PlayerAdded:Connect(function(plr)
	
	wait()
	
	local elementdata = plr:WaitForChild("ElementData")
	local elementtf = elementdata:WaitForChild("HasElement")
	local elementtype = elementdata:WaitForChild("ElementType")
	
	repeat wait() until elementtf.Value == true and (elementtype.Value ~= "NA" and #elementtype.Value ~= 0)
	
	plr.CharacterAdded:Connect(function(char)
		if elementtype.Value == "Ice" then
			local aura = game.ReplicatedStorage.ElementAuras.IceAura:Clone()
			aura.Anchored = false
			aura.CanCollide = false
			aura.Parent = char
			aura.Name = aura.Name.."/"..plr.Name
			local bp = Instance.new("BodyPosition")
			bp.Parent = aura
			bp.Position = char:WaitForChild("HumanoidRootPart").Position + Vector3.new(0,-2, 0)
			wait(1)
			
			while true do
				wait()
				bp.Position = char:WaitForChild("HumanoidRootPart").Position + Vector3.new(0,-2, 0)
			end
			
		elseif elementtype.Value == "Fire" then
			local aura = game.ReplicatedStorage.ElementAuras.FireAura:Clone()
			aura.Anchored = false
			aura.CanCollide = false
			aura.Parent = char
			aura.Name = aura.Name.."/"..plr.Name
			local bp = Instance.new("BodyPosition")
			bp.Parent = aura
			bp.Position = char:WaitForChild("HumanoidRootPart").Position + Vector3.new(0,-2, 0)
			wait(1)
			
			while true do
				wait()
				bp.Position = char:WaitForChild("HumanoidRootPart").Position + Vector3.new(0,-2, 0)
			end
			
			
		elseif elementtype.Value == "Earth" then
			local aura = game.ReplicatedStorage.ElementAuras.EarthAura:Clone()
			aura.Anchored = false
			aura.CanCollide = false
			aura.Parent = char
			aura.Name = aura.Name.."/"..plr.Name
			local bp = Instance.new("BodyPosition")
			bp.Parent = aura
			bp.Position = char:WaitForChild("HumanoidRootPart").Position + Vector3.new(0,-2, 0)
			wait(1)
			
			while true do
				wait()
				bp.Position = char:WaitForChild("HumanoidRootPart").Position + Vector3.new(0,-2, 0)
			end
			
		end
	end) 
end)