Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
Attempt for index nil with player gui and player not being kicked?
I was making an admin panel and here is the code with 2 errors:
01 | script.Parent.BanButton.MouseButton1Click:Connect(function(player) |
02 |
03 | local playerGui = player.PlayerGui |
04 |
05 | local playerNameBox = script.Parent.PlayerNameBox |
06 | local banReasonBox = script.Parent.BanReasonBox |
07 |
08 | local playerToBan = game.Players:FindFirstChild(playerNameBox.Text) |
09 |
10 | playerToBan.Banned.Value = true |
11 |
12 | playerToBan.Banned.BanReason.Value = banReasonBox.Text |
13 |
14 | if playerNameBox.Text == "Name" then |
15 | playerGui.BanPanel.MainFrame.BanButton.Text = "Please enter a player to ban" |
View all 21 lines...
The first error is on line 15 and 17. "Attempt to index nil with 'PlayerGui'" Another error I got before I added was with the kick part. It is supposed to kick the player in the banReasonBox. Note: This is a local script
What is the problem? Thanks!
3 answers
Log in to vote
1
Your supposed to reference the player using:
1 | local Player = game.Players.LocalPlayer |
Also keep in mind to check if the player thats inputted exists. It might lead to errors later.
1 | if playerToBan then |
2 | playerToBan:Kick(banReasonBox.Text) |
3 | end |
Have a great day!
Log in to vote
0
I don't think player is a parameter of mousebutton1click.
0
you could do local player = script.Parent.Parent.Parent or somethin like that. BulletproofVast 1033 — 4y
Log in to vote
0
at the 3rd line, change
1 | local playerGui = player.PlayerGui |
to
1 | local playerGui = game.Players.LocalPlayer.PlayerGui |
this is because player is not a parameter of Gui.MouseButton1Click event. Instead, it is a parameter for ClickDetector.MouseClick event.