Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
Gui giver with lighting?
So...I'm trying to turn this script into a giver that i can use on a Gui Button. But i cant figure out how i can make the Click function work with a gui button?
Script Below...
01 | local itemname = "Ggs" |
02 |
03 | local item = game.Lighting:findFirstChild(tostring(itemname)) |
04 |
05 | local trigger = script.Parent |
06 |
07 | enabled = true |
08 |
09 | function onClick(plyr) |
10 |
11 | if plyr.Backpack:findFirstChild(tostring(itemname)) == nil and enabled == true then |
12 |
13 | enabled = false |
14 |
15 | trigger.BrickColor = BrickColor.new("Black") |
View all 28 lines...
2 answers
Log in to vote
1
Sorry if I'm wrong, but the local variables have to be declared inside the function. Here is the new code:
01 | enabled = true |
02 |
03 | function onClick(plyr) |
04 |
05 | if plyr.Backpack:findFirstChild(tostring(itemname)) == nil and enabled == true then |
06 |
07 | local itemname = "Ggs" |
08 |
09 | local item = game.Lighting:findFirstChild(tostring(itemname)) |
10 |
11 | local trigger = script.Parent |
12 |
13 | enabled = false |
14 |
15 | trigger.BrickColor = BrickColor.new("Black") |
View all 28 lines...
But wait! You haven't declared what "plyr" is. I would do this: (THIS SCIPT MUST BE A LOCALSCRIPT FOR IT TO WORK!!!) Oh, and remove "plyr" from the parentheses from the function.
1 | local plyr = LocalPlayer |
2 | local Backpack = plyr:WaitForChild("Backpack") |
3 | -- Put this with all your local variables |
Also, why are you using ~~~~~~~~~~~~~~~~~ tostring ~~~~~~~~~~~~~~~~~ ?
0
To correct you, local variables do not need to be used in the specific function you use it in. If it isn't in any 'end needed' functions, then it can be used ANYWHERE. Second, your logic is wrong. plyr would be the MOUSE not the actual player, and anyway, 'LocalPlayer' isn't declared! It's supposed to be 'game.Players.LocalPlayer'. deputychicken 226 — 9y
0
Sorry. I wrote this at 11:00 PM fight4money -2 — 9y
Log in to vote
0
You don't have to change much, other than plyr.Backpack and put it in a "LocalScript". Here is the GUI Version:
01 | local itemname = "Ggs" |
02 |
03 | local item = game.Lighting:findFirstChild(tostring(itemname)) |
04 |
05 | local trigger = script.Parent |
06 |
07 | local enabled = true |
08 |
09 | -- function onClick(plyr) This would be declared as the mouse instead of the player. |
10 | function onClick() |
11 | local plyr = game.Players.LocalPlayer -- This is a way of getting the 'Player' (In the 'Players' Section of game) |
12 | if plyr.Backpack:findFirstChild(tostring(itemname)) == nil and enabled == true then |
13 |
14 | enabled = false |
15 |
View all 28 lines...