I need help debugging a ban system that relies on a datastore + improving it

Help and Feedback Scripting Support
#21

Alright I’ve found the problem, it seems that it was printing “BAN A PLAYER”. That only means that there is something wrong in the LocalScript.

However, even then, it still keeps printing “UserId not founded”.

1 Like
#22

Can you show me the GUI in the explorer?

#23 
-- VARIABLES

local RS = game:GetService("ReplicatedStorage")
local Player = game.Players.LocalPlayer
local SGUI = script.Parent.Parent.Parent
local Button = SGUI.ScrollingFrame:WaitForChild("BanPlayer")
local banEvent = RS.BanEvent
local debounce = false
local PGUI = Player:WaitForChild("PlayerGui")

--EVENT

Button.MouseButton1Click:Connect(function()
	local text = PGUI.AdminPanel.ScrollingFrame.BanBox
	if debounce == false then
		debounce = true
		print("SUCCESS")
		banEvent:FireServer(text.Text)
		task.wait(0.25)
		debounce = false
	end
end)

The text variable is a TextBox, so that I can type in it.

#24

No I mean the explorer window with the GUI, not the script

#25

8b50364d8b20527b801a4279c9ec2b81
The GUI is located in StarterGui. The most important parts here is BanPlayer and BanBox.

#26

Your local script also seems fine. Are you sure there aren’t any duplicates of the BanBox or the AdminPanel?

#27

There are no duplicates of BanBox or AdminPanel in StarterGui.

#28

Maybe change your variables to be “relative” to the script just in case there somehow are duplicates. I am guessing the local script you sent is called ClickScript?

#29

The script I sent is the one inside BanPlayer button.

1 Like
#30

Try this instead

local RS = game:GetService("ReplicatedStorage")
local Player = game.Players.LocalPlayer
local SGUI = script.Parent.Parent.Parent
local Button = script.Parent
local banEvent = RS.BanEvent
local debounce = false

--EVENT

Button.MouseButton1Click:Connect(function()
	local text = SGUI.ScrollingFrame.BanBox
	print(text.Name)
	print(text.Text)
	if debounce == false then
		debounce = true
		print("SUCCESS")
		banEvent:FireServer(text.Text)
		task.wait(0.25)
		debounce = false
	end
end)
#31

It doesn’t look like a lot has changed, since the output is still printing that “UserId not found!” Should I test in game?

#32

Studio should be fine. Did the print(text.Text) inside of the localscript still say “BAN A PLAYER”?

#33

No, it stopped printing “BAN A PLAYER” however “UserId not found” still prints, even after I put in the Player’s name.

#34

What does text.Text print? Also, you can try testing it in game although I am not sure if it will make a difference.

#35

The text.Text prints what I type in the box.

#36

It might be my UserId function, give me a few minutes to test it

#37

Are there any orange lines under the function I sent earlier? I forgot to add an end to it

#38

Yes, but I put a end there so it’s all good.

#39

I just noticed a typo in the script. Also, I tested the UserId function and it worked fine for me.

if target == nil then --i accidentally wrote taget
	warn("UserId not found!")
	return
end
#40

That also was fixed eariler, what should I do to fix the script?