Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
why does this short script not work?
Please make your question title relevant to your question content. It should be a one-sentence summary in question form.
1 | btn1 = script.Parent |
2 | player = game.Players.LocalPlayer |
3 | btn1.MouseButton1Down:connect(function() |
4 | game.Lighting.tools.Dribbler:GetChildren().Parent = player.Backpack |
5 | end) |
please help
3 answers
Log in to vote
2
What your issue is line 4. The variables being set to local won't change much. I'm trying to understand what you're trying to accomplish. You're either a: Trying to clone 1 tool into the player's backpack b: Trying to clone a set of tools into the player's backpack Since it's hard to tell which you're going for, I'll offer both solutions.
Solution A:
1 | btn1 = script.Parent |
2 | player = game.Players.LocalPlayer |
3 |
4 | btn1.MouseButton1Down:connect(function() |
5 | game.Lighting.tools.Dribbler:clone().Parent = player.Backpack |
6 | end) |
Solution B:
1 | btn1 = script.Parent |
2 | player = game.Players.LocalPlayer |
3 |
4 | btn1.MouseButton1Down:connect(function() |
5 | for i,v in pairs(game.Lighting.tools.Dribbler:GetChildren()) do |
6 | v:clone().Parent = player.Backpack |
7 | end |
8 | end) |
Any questions? Feel free to comment!
Log in to vote
0
You're trying to set a Table's parent. You can't do that.
In order to set all the Children's Parent, you need to set a Generic For loop, something like this:
1 | for _,v in pairs(game.Lighting.tools.Dribbler:GetChildren()) do |
2 | v:Clone().Parent = player.Backpack |
3 | end |
Also, I'd recommend using game.ReplicatedStorage instead of game.Lighting. Using game.Lighting as an asset storage is deprecated.
Log in to vote
-1
Perhaps set the variables as local variables like so
1 | local btn1 = script.Parent |
2 | local player = game.Players.LocalPlayer |
3 | btn1.MouseButton1Down:connect(function() |
4 | game.Lighting.tools.Dribbler:GetChildren().Parent = player.Backpack |
5 | end) |