basically the parent clones into workspace and clicking it should buy it
game.Players.PlayerAdded:Connect(function(plr)
Money = plr.leaderstats.Money
script.Parent.ClickDetector.MouseClick:Connect(function()
if Money.Value < 30 then
print('not enough money')
else
Money.Value = Money.Value - 30
script.Parent:Destroy()
end
end)
end)
You're doing something wrong, other people can click the brick and use your OWN money, so instead of doing that you just add a single event.
script.Parent.ClickDetector.MouseClick:Connect(function(plr)
if not plr then return end -- End the function if there's no player
Money = plr.leaderstats.Money
if Money.Value < 30 then
print('not enough money')
else
Money.Value = Money.Value - 30
script.Parent:Destroy()
end
end)
Done! Now you listen with one connection instead of more, and now it only does it to the player that clicked it! If this helped mark this as the answer!
This discussion is preserved as it was. New answers, comments, and votes are closed.
11 comments
ah, i see what you mean, now im back to the problem where the else isnt working
Is the money value a number value?
Does it at least destroy the part?
money value is a int value, it prints not enough money when clicked with enough money
Can you show an image of the value in explorer and output ?
https://imgur.com/a/sJeFGpB is that what you mean?
Is the money value set locally? If it is set it server-side
i set it globally (what i think you saying) still nothing
Try printing the value on clicking
its always printing 0
You should set it from the server and not client