Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
how do i make a giver that works with click detector?
yo i managed to make a giver but now im trying to make it work on click so do you know why this script doesnt work?
01 | local Giver = script.Parent |
02 | local Gear = script.Parent.Handgun |
03 | local idk = false |
04 | function give(Part) |
05 | local hum = Part.Parent:findFirstChild("Humanoid") |
06 | if (hum ~= nil) and idk == false then |
07 | idk = true |
08 | local Location = |
09 | game:GetService("Players"):GetPlayerFromCharacter(hum.Parent) |
10 | local New = Gear:Clone() |
11 | New.Parent = Location.Backpack |
12 | Giver.BrickColor = BrickColor.new("Really red") |
13 | wait(3) |
14 | Giver.BrickColor = BrickColor.new("Medium stone grey") |
15 | idk = false |
16 | end |
17 | end |
18 | Giver.ClickDetector.MouseClick:Connect(give) |
2 answers
Log in to vote
0
You don't put tools in the player. You can either put it in the character or put it in the player's backpack. Here's the fixed script.
1 | local Location = game:GetService("Players"):GetPlayerFromCharacter(hum.Parent).Backpack |
Also, FindFirstChild may need to be capitalized, I dunno.
0
doesnt work TFlanigan 86 — 5y
0
also lmao u just copied those 2 lines from the script TFlanigan 86 — 5y
0
I forgot to edit them joshthegamer456 93 — 5y
0
fixed. joshthegamer456 93 — 5y
Log in to vote
0
It's because you don't understand what MouseClick passes as an argument. MouseClick doesn't pass a part; it passes the player who clicked the ClickDetector.
Edited script:
01 | local Giver = script.Parent |
02 | local Gear = script.Parent.Handgun |
03 | local idk = false |
04 | function give(Player) |
05 | if Player then |
06 | if not idk then |
07 | idk = true |
08 | local New = Gear:Clone() |
09 | New.Parent = Player.Backpack |
10 | Giver.BrickColor = BrickColor.new("Really red") |
11 | wait(3) |
12 | Giver.BrickColor = BrickColor.new("Medium stone grey") |
13 | idk = false |
14 | end |
15 | end |
16 | end |
17 | Giver.ClickDetector.MouseClick:Connect(give) |
Only Touched passes a BasePart as an argument. A ClickDetector's MouseClick event always passes the player who clicked on the ClickDetector.