Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
Money button script for tycoon not working for some reason using can collide script?
I'm making a money giver that turns off and on to do this I'm using a simple can collide off and on but for some reason with this script in it. it will not give you the money here are the scripts.
1 | while true do |
2 | wait(450) |
3 | script.Parent.CanCollide = true |
4 | script.Parent:Remove() |
5 | wait(200) |
6 | script.Parent.CanCollide = false |
7 | script.Parent:Remove() |
8 | end |
01 | deb = false |
02 | script.Parent.Touched:connect(function(part) |
03 | if game.Players:FindFirstChild(part.Parent.Name) then |
04 | if deb == false then |
05 | deb = true |
06 | local plr = game.ServerStorage.PlayerMoney:FindFirstChild(part.Parent.Name) |
07 | script.Parent.BrickColor = BrickColor.new("Bright red") |
08 | plr.Value = plr.Value + script.Parent.Parent.Amount.Value |
09 | wait(script.Parent.Parent.TimeToWait.Value) |
10 | deb = false |
11 | script.Parent.BrickColor = BrickColor.new("Bright green") |
12 | end |
13 | end |
14 | end) |
0
Simple fix. `:Remove() ` is deprecated and no longer works. To delete, you now use `:Destroy()` CreationNation1 459 — 4y
1 answer
Log in to vote
0
Both comments are false: :Remove works just fine (though it is deprecated). :Remove is identical to setting .Parent = nil, if don't you want to destroy something (though in this case, I don't think you want to remove or destroy anything.)
If you merge your scripts, it's much easier to get the behaviour you want:
01 | local deb = false |
02 | local enabled = true |
03 | script.Parent.Touched:connect(function(part) |
04 | if game.Players:FindFirstChild(part.Parent.Name) then |
05 | if enabled and not deb then |
06 | deb = true |
07 | local plr = game.ServerStorage.PlayerMoney:FindFirstChild(part.Parent.Name) |
08 | script.Parent.BrickColor = BrickColor.new("Bright red") |
09 | plr.Value = plr.Value + script.Parent.Parent.Amount.Value |
10 | wait(script.Parent.Parent.TimeToWait.Value) |
11 | deb = false |
12 | script.Parent.BrickColor = BrickColor.new("Bright green") |
13 | end |
14 | end |
15 | end) |
View all 24 lines...