Problem with Clicking Gui for item

Help and Feedback Scripting Support
#1

Hello, i have this issue where when a player(only that player) clicks a button, they recieve a tool.
The problem is it does not give the tool and i get this error:
Players.wanu10.PlayerGui.ScreenGui2.TextButton.Script:7: attempt to index nil with ‘Backpack’
here is my code:

local tool = game.ServerStorage.Tool


script.Parent.MouseButton1Click:Connect(function(plr)
	local Tool1 = tool:Clone()
	print(plr)
	Tool1.Parent = plr

end)
2 Likes
#2

what type of script is this because if it is a local script here is a few problems

#3

its a normal script i used for.

#4

also the MouseButton1Click event doesn’t have a player has an argument

#5

what do you mean im new to coding.

#6

What you should do instead is you should detect the MouseButton1Click event on the client - local script and then send a remote event onto the server where it would give them the tool

#7

uhmmmm what’s a client? do you mean network client?

#8

oh so basically you can’t do

script.Parent.MouseButton1Click:Connect(function(player)
end)
-- the player would be nil
#9

client i meaning like a local script

#10

so what can i do? . ---------------

#11

what should i put in the local script???

#12

sorry im very new to scripting.

#13

do i just type what i typed before and then send something to the server?

#14 
--local script
-- script parented to the button

local remote = game.ReplicatedStorage.RemoteEvent -- using remote event; you would need to add this

script.Parent.MouseButton1Click:Connect(function()
    remote:FireServer()
end) 


-- server script (ServerScriptService)
local remote = game.ReplicatedStorage.RemoteEvent

remote.OnServerEvent:Connect(function(player)
    local tool = game.ServerStorage.Tool:Clone()
    
    tool.Parent = player.Backpack
end)
#15

you would need to do something like this. You are new to scripting so you probably don’t understand remote events so here a link that can help u

1 Like
#16

thanks so much. have a good day.

#17

i do have a question tho, now that that works i tried to make it so when you click it a certain value goes away.
for now i need a value to be - 3 when crafted.

here is the code you gave me and i modified it.

local remote = game.ReplicatedStorage.RemoteEvent

remote.OnServerEvent:Connect(function(player, plr)
	print(plr)
	local wood = plr.leaderstats.Wood
	local tool = game.ServerStorage.Tool:Clone()

	tool.Parent = player.Backpack
	wood.Value = wood.Value - 3
end)
#18

urr i see some problems

remote.OnServerEvent:Connect(function(player, plr) -- you don't need (plr) because u got (player)
end)
#19 
local remote = game.ReplicatedStorage.RemoteEvent

remote.OnServerEvent:Connect(function(plr)
	print(plr)
	local wood = plr.leaderstats.Wood
	local tool = game.ServerStorage.Tool:Clone()

	tool.Parent = plr.Backpack
	wood.Value = wood.Value - 3
end)
1 Like
#20

omg this helped tysm 1!!!