Can someone help me with the playergui error, it seems unavoidable?
I'm trying to make a GUI pop up for people who touch the hitbox of a shop, and I put the localscript for it inside a folder that is in the playergui, which is my script storage. Here is the code
01 | local last = 0 |
02 |
03 | workspace.Shop.CrateShop.CrateShopTouch.Hitbox.Touched:Connect(function(hit) |
04 | if tick() - last <= 1 then |
05 |
06 | else |
07 | local hum = hit.Parent:FindFirstChild("Humanoid") |
08 | if hum ~= nil then |
09 | local plr = hum.Parent.Name |
10 | local plrC = game.Players:FindFirstChild(plr) |
11 | if plrC.PlayerGui.ScreenGUI:FindFirstChild("CrateShopMenu") == nil then |
12 | local newCrateMenu = game.ReplicatedStorage.Storage.GUIObjects.CrateShopMenu:Clone() |
13 | newCrateMenu.Parent = plrC.PlayerGui.ScreenGUI |
14 | elseif plrC.PlayerGui.ScreenGUI:FindFirstChild("CrateShopMenu") ~= nil then |
15 |
16 | end |
17 | end |
18 | end |
19 | last = tick() |
20 | end) |
When the local player touches it, it's doesn't send any errors their way, but it sends out an error for everyone else. If it isn't the local player that is touching it, it sends out something like this "PlayerGui is not a valid member of Player" "Begin" "Players.[LocalPlayerName].PlayerGui.Scripts.ShopTouch Line 11" "End"
where localplayername is, it just shows the name of your player, no matter who touched it.
0
0
1 answer
For what I know, Touch only works in server script. For this, I recommend that you use RemoteEvents with FireClient. On the client event you need to use a Server Script and LocalScript
Your errors: Do not use .Touched in local scripts (for me it never works.), It is not recommended to clone GUIs by a server script, only by the client is the best way in my opinion. (From what I know, You can put GUI's through the server, however it is recommended to put by Client).
Example:
SERVER
01 | local event = game.ReplicatedStorage.RemoteEvent |
02 |
03 | script.Parent.Touched:Connect(function(hit) |
04 | if hit.Parent:FindFirstChild("Humanoid") then |
05 | local player = game:GetService("Players"):FindFirstChild(hit.Parent.Name) |
06 | if player then |
07 | event:FireClient(player) |
08 | end |
09 | elseif hit.Parent.Parent:FindFirstChild("Humanoid") then |
10 | local player = game:GetService("Players"):FindFirstChild(hit.Parent.Parent.Name) |
11 | if player then |
12 | event:FireClient(player) |
13 | end |
14 | end |
15 | end |
CLIENT
1 | game.ReplicatedStorage.RemoteEvent.OnClientEvent:Connect(function() |
2 | warn("Fired!") |
3 | end) |
Here is your script fixed:
Part Touch - Server Script(Script) - (put in part)
01 | local last = 0 |
02 |
03 | local event = game:GetService("ReplicatedStorage").RemoteEvent -- Location of event. |
04 |
05 | function Touch(hit) |
06 | if tick() - last <= 1 then |
07 |
08 | else |
09 | if hit.Parent:FindFirstChild("Humanoid") then |
10 | local player = game:GetService("Players"):FindFirstChild(hit.Parent.Name) |
11 | if player then |
12 | event:FireClient(player) |
13 | end |
14 | elseif hit.Parent.Parent:FindFirstChild("Humanoid") then |
15 | local player = game:GetService("Players"):FindFirstChild(hit.Parent.Parent.Name) |
GetFiredEvent - LocalScript - (put in StarterGui)
01 | repeat wait() until game.Players.LocalPlayer |
02 | local player = game.Players.LocalPlayer -- Get localPlayer (ONLY WORKS IN LocalScript) |
03 | local playerGui = player.PlayerGui -- Get playerGUI |
04 |
05 | local GUI = game:GetService("ReplicatedStorage").Storage.GUIObjects.CrateShopMenu -- Location of the object. |
06 |
07 | local GUI_Holder_Name = "ScreenGUI" -- Put the name of GUI or location. |
08 |
09 | game.ReplicatedStorage.RemoteEvent.OnClientEvent:Connect(function() -- On fire event |
10 |
11 | playerGui:WaitForChild(tostring(GUI_Holder_Name)) -- Get GUI HOLDER NAME |
12 |
13 | local Holder = playerGui:FindFirstChild(tostring(GUI_Holder_Name)) -- GET Holder GUI |
14 |
15 | if Holder then -- IF FIND Holder GUI |
Hope it helped! :)