Attempt To Index Nil with 'userid'

[Ma7adi805 (III)]

local admins = {2428180234, 2428233997, game.CreatorId} --UserId of the admins
local reason = game:GetService("ReplicatedStorage"):WaitForChild("SoftShutdown")
script.Parent.MouseButton1Click:Connect(function(Caller)
	local caller = Caller.UserId
	if table.find(admins, caller) then
		if reason.SoftShutdownReason.Value=="" then
			script.Parent.Text="Enter Reason"
		elseif reason.SoftShutdownReason.Value~="" then
			reason:FireServer()
			
			
		end
	end
	
end)

Apparently the error is on the variable caller

[seancool07777]

Is script.Parent a UI object?
If so, you should get the player from game.Players.LocalPlayer instead.
You are clearly meant to be in a local script, so do it the suggested way instead.

@Ma7adi805

local caller = Caller.UserId

Replace that with:

local caller = game.Players.LocalPlayer.UserId

Also, you should handle the authorization on the server. Otherwise, exploiters can shutdown the server. Do not trust the client.

[Ma7adi805 (III)]

I am using a script currently to Fire the server, do I still can fire the server in local?? script.parent is a button

[seancool07777]

You can only FireServer in a local script. UI objects are to be controlled in local scripts.

[Ma7adi805 (III)]

That worked, thank you so, I am a new dev and I keep forgetting everything