Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
(ANSWERED) How come my script only gives the player the item ONCE?
Introduction
Hi there! I made a script that gives a player an item when they click it, and it works. But it only gives them the item once and then stops working. Here is my script:
1 | local tool = game.ServerStorage.Card |
2 | local klone = tool:clone |
3 | script.Parent.ClickDetector.MouseClick:connect (function(plr) |
4 | if klone.Parent ~= plr.Backpack then |
5 | klone.Parent = plr.Backpack |
6 | else |
7 | end |
8 | end) |
Conclusion
Can somebody help me?
0
I mean if they already have the item, it won't give them another one right? Unless your talking about if the player has the item, looses the item, and then tries to get the item back. If thats the case, idk ;-;. fighterkirbyzx 102 — 5y
0
Put local Klone inside the function. That should fix it fighterkirbyzx 102 — 5y
2 answers
Log in to vote
0
So in this script, there is an if-then statement that checks if you currently have the item. If you remove the statement, then it should fix it.
1 | local tool = game.ServerStorage.Card |
2 | local klone = tool:clone |
3 | script.Parent.ClickDetector.MouseClick:connect (function(plr) |
4 | klone.Parent = plr.Backpack |
5 | else |
6 | end) |
0
Also it only clones klone once so it probably needs to have local klone INSIDE the function so it clones every time they clicks it fighterkirbyzx 102 — 5y
0
try doing tool:Clone() instead of tool:clone User_3805 6 — 5y
Log in to vote
0
Here you go.
01 | script.Parent.ClickDetector.MouseClick:connect (function(plr) |
02 | local tool = game.ServerStorage.Card:Clone() |
03 | if tool.Parent ~= plr.Backpack then |
04 | tool.Parent = plr.Backpack |
05 | else |
06 | tool:Destroy() |
07 | warn("This player already has the tool!") |
08 | return |
09 | end |
10 | end) |