Bug - Team Changer ServerSide

Help and Feedback Scripting Support
1 
game.Players.PlayerAdded:Connect(function(p)

	local function Clicked()
		if p:GetRankInGroup(9536284) >= 3 then
			p.TeamColor = BrickColor.new("Sea green")
			p:LoadCharacter()
		
		end
	end
	event.OnServerEvent:Connect(Clicked)
end)

When I click the team changer in StarterUI, everyone above that rank gets their team changed. How can I make it one person only?

2

Can you be more specific?
charsssssss

1 Like
3

When I click it, everyone in the server that is above that rank stated will get their team changed, I only want the person who clicked it to be on that team.

4

You should connect the function outside the PlayerAdded event. You’re connecting it everytime a new Player is added, Meaning all Players are gonna be affected no matter who fires it.

RemoteEvent.OnServerEvent:Connect(function(Player)
    -- Function runs here
end)
2 Likes
5

Don’t wrap that in a PlayerAdded event, instead make a function that has a player parameter (onserverevent first parameter), then check the player’s rank value

6

Trying this currently. Will use it

7

Nothing happens when I click the button.

8

May you please show us the code that you used on the Server-side? I tested it and it worked perfectly for me.

1 Like
9 

local folder = game:GetService("ReplicatedStorage").TeamChangeEvents
local event = folder.CA
local sofcom = folder.CANSOFCOM
local cfmp = folder.CFMP
local civ = folder.CIV
local core = folder.CORE
local hq = folder.HQ
local cadtc = folder.CFDTC



	

	event.OnServerEvent:Connect(function(p)

	
	local function Clicked()
		
		if p:GetRankInGroup(9536284) >= 3 then
			
			p.TeamColor = BrickColor.new("Sea green")
			
			p:LoadCharacter()
			
			event.OnServerEvent:Connect(Clicked) 
			

			
		
	end

	end

end)```
10

Connect OnServerEvent with the Clicked function, not a anonymous function

1 Like
11 
		Clicked.OnServerEvent:Connect()
12

Not like that, put the function in the parentheses

1 Like
13

Declare the Clicked function, And then Connect it. Do not connect the Event inside the Clicked function.

-- Declare the function
-- DO NOT CONNECT IT TWICE!
event.OnServerEvent:Connect(Clicked)
2 Likes
14

Should it be something like this?

local p = game.Players.LocalPlayer		

		
			

				local function Clicked()
if p:GetRankInGroup(9536284) >= 3 then
			
			
			p.TeamColor = BrickColor.new("Sea green")
			
			p:LoadCharacter()
			
			event.OnServerEvent:Connect(Clicked)


			
		
	end

	end
15

Put the event in the global scope, not in the function

Sorry for late response.

1 Like
16

What does that mean? I don’t understand

17 
local p = game.Players.LocalPlayer

local function Clicked()
    if p:GetRankInGroup(9536284) >= 3 then
	   p.TeamColor = BrickColor.new("Sea green")
	   p:LoadCharacter()
   end
end

event.OnServerEvent:Connect(Clicked)
1 Like
18

Ok, I will use this. But what if I have multiple teams? I copy and paste?

19 
local function Clicked(player, teamColor)
    if p:GetRankInGroup(9536284) >= 3 then
	   p.TeamColor = teamColor
	   p:LoadCharacter()
   end
end

event.OnServerEvent:Connect(Clicked)
20

Duplicate function called on line 7