Teleporter teleports all players at once, even if it's supposed to teleport only one player!

[Corruptux]

So I made portals that teleport you to a specific location on the map, and for some reason it teleports all of the players in the server. It also teleports everyone when a different object that has a humanoid in it touches the portal.

All of the scripts are local scripts in starterGui, and the portals are in a model inside of the workspace.

Here are the scripts:

Portal 1:

game.Workspace.Zone4.SansCatPortal.Portal1.Main.Touched:Connect(function()
	game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.SansCatPortal.Portal2.Main.CFrame - Vector3.new(0,0,8)
end)

Portal 2:

game.Workspace.Zone4.SansCatPortal.Portal2.Main.Touched:Connect(function()
	game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.SansCatPortal.Portal1.Main.CFrame + Vector3.new(0,0,8)
end)

It also seems to happen on a portal that only teleports players with a gamepass:

Portal 1:

local M = game:GetService("MarketplaceService")
local GamepassID = 17768668
game.Workspace.Zone4.VIPPortal.Portal1.Main.Touched:Connect(function()
	if M:UserOwnsGamePassAsync(game.Players.LocalPlayer.UserId,GamepassID) then
		game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.VIPPortal.Portal2.Main.CFrame + Vector3.new(0,0,8)
	else
		
	end
end)

Portal 2:

game.Workspace.Zone4.VIPPortal.Portal2.Main.Touched:Connect(function()
	game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.VIPPortal.Portal1.Main.CFrame + Vector3.new(0,0,8)
end)

If anyone can help, I’d appreciate that!

[dthecoolest]

See this tutorial which describes the issue very well with touched events on the client.

@colbert2677

Wait - other clients see my actions!

Which is the point of this thread! Understand this: the connection you establish using the above code is still local . With the way Touched works however, that means that the client is actively monitoring any part touches. It’s not that there’s any replication going on here: the client sees that a part touched something and so the signal is fired.

In order to prevent other clients from seeing your actions, you need to verify within the function that the part that touched the object is part of the player’s character. This is only if you’re making local interactions, such as showing shop interfaces like the example a ways above.

To check a part’s association with the character, we simply need to run IsDescendantOf. If you consider the parent-child hierarchy for Roblox instances, we can use this to check if the part is part of the character’s ancestry.

local LocalPlayer = game:GetService("Players").LocalPlayer

shopPart.Touched:Connect(function (part)
    -- Let's check that a Character is valid as well
    if LocalPlayer.Character and part:IsDescendantOf(LocalPlayer.Character) then
        print("Our character touched the shop part!")
        shopGui.Enabled = true
    end
end)

And just like that, you’re set! It’s all about verifying that the touched part is part of the LocalPlayer’s character, if one exists,

[cyswip]

@Corruptux

game.Workspace.Zone4.VIPPortal.Portal2.Main.Touched:Connect(function()
	game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.VIPPortal.Portal1.Main.CFrame + Vector3.new(0,0,8)
end)

Anything touched the part will fire this and because of all the clients have this localscript it teleports them

[cyswip]

do something like this

game.Workspace.Zone4.VIPPortal.Portal2.Main.Touched:Connect(function(hit)
if hit.Parent == game.Players.LocalPlayer.Character then
	game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.VIPPortal.Portal1.Main.CFrame + Vector3.new(0,0,8)
end
end)

[EmbatTheHybrid (Embat)]

Everyone had mention the correct answer that you need to verify touches when they’re client sided or in general, any client sided event unrelated to Guis, such as a ProximityPrompt’s Triggered event which will also need verification, but I want to address something with this script

local M = game:GetService("MarketplaceService")
local GamepassID = 17768668
game.Workspace.Zone4.VIPPortal.Portal1.Main.Touched:Connect(function()
	if M:UserOwnsGamePassAsync(game.Players.LocalPlayer.UserId,GamepassID) then
		game.Players.LocalPlayer.Character.HumanoidRootPart.CFrame = game.Workspace.Zone4.VIPPortal.Portal2.Main.CFrame + Vector3.new(0,0,8)
	else
		
	end
end)

There’s really no need to do it like this if it’s client sided, because even players who don’t have the gamepass will have the event made for them, check if they have the gamepass first before making the touched event

local M = game:GetService("MarketplaceService")
local Players = game:GetService("Players")
local GamepassID = 17768668

if M:UserOwnsGamePassAsync(Players.LocalPlayer.UserId,GamepassID) then
	workspace.Zone4.VIPPortal.Portal1.Main.Touched:Connect(function()
		Players.LocalPlayer.Character.HumanoidRootPart.CFrame = workspace.Zone4.VIPPortal.Portal2.Main.CFrame + Vector3.new(0,0,8)
	end)
end

And I recommend moving the scripts somewhere where the events will only be made once, such as StarterPlayerScripts, as anything in StarterGui, including Localscripts, will be replicated on respawn, so it’ll recreate the events again which could add up over time as they wont overwrite each other I believe

[Corruptux]

I’m gonna try that! Hope it works!

[Corruptux]

It worked, thank you!