So my goal is to use this script to make it so that when you click on a button, an event is fired. That event could make a bunch of options happen. Here is the script: Local script in gui:

local ReplicatedStorage = game:GetService("ReplicatedStorage")
local event = ReplicatedStorage:WaitForChild("RemoteEvent")

script.Parent.MouseButton1Down:connect(function(billnye)
	script.Parent.Parent.Person.Value = script.Parent.Parent.b.Text
	script.Parent.Parent.Parent.yesgui.a.Visible = false
	script.Parent.Parent.Parent.yesgui.b.Visible = false
	script.Parent.Parent.Parent.yesgui.c.Visible = false
	script.Parent.Parent.Parent.yesgui.d.Visible = false
	event:FireServer(script.Parent.Parent.Parent.Attack.ScrollingFrame.AttackDone.Value, game.Workspace:FindFirstChild(game.StarterGui.yesgui.Person.Value), game.Players:FindFirstChild(game.StarterGui.yesgui.Person.Value))
end)

The variables are values that define what you want to happen to other players, one also has the player name that they want to attack.

ServerScriptService script:

local ReplicatedStorage = game:GetService("ReplicatedStorage")
local event = ReplicatedStorage:WaitForChild("RemoteEvent")

local function yesgogo(attack, playerokay, playerokayplay)
	if attack == "Kill" then
		playerokay.Humanoid.Health = 0
	end
	if attack == "Speed" then
		playerokay.Humanoid.WalkSpeed = 100
	end
	if attack == "Slow" then
		playerokay.Humanoid.WalkSpeed = 3
	end
	if attack == "Jail" then
		local jailCopy = game.Workspace.AdminAssets.Jail:Clone()
		jailCopy.Parent = game.Workspace
		jailCopy.Transparency = 0
		jailCopy.CanCollide = true
		jailCopy.Position = Vector3.new(playerokay.HumanoidRootPart.Position.X, playerokay.HumanoidRootPart.Position.Y, playerokay.HumanoidRootPart.Position.Z)
	end
	if attack == "LevelOP" then
		playerokayplay.leaderstats.Level.Value = 100
	end
	if attack == "Reset" then
		playerokayplay.leaderstats.Level.Value = 1
	end
end

event.OnServerEvent:Connect(yesgogo)

I don't know why these don't effect the player. Please help. Thank you.

4 comments

0

try using math.random and assigning each event to 1 number

0

Just a quick note here, variables are supposed to make it easy to understand what they represent, are you sure your variable choice does that?

0

Could you use more meaningful variables?

0

I should explain what this is for, so I am making admin commands but instead of using it threw chat I am using it threw a gui. When the admin button is clicked options show up. The variables represent the options. After that you type in the player you want to effect. That is another variable. But when I click the kill option and try to use it, it does not work. I don't know why.