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.
01 | script.Parent.ClickDetector.MouseClick:Connect( function (plr) |
02 | if not plr then return end |
04 | Money = plr.leaderstats.Money |
05 | if Money.Value < 30 then |
06 | print ( 'not enough money' ) |
08 | Money.Value = Money.Value - 30 |
09 | script.Parent:Destroy() |
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!