Answered by
6 years ago Edited 6 years ago
Cloning
Instead of storing the cloned audio in a variable, you don't. If you don't use this, you can't reference the cloned object
Deprecated Syntaxes
You are using :connect(). This is deprecated or not recommended to use. Use :Connect()
Why Are You Looping The Players?
You are looping the players even though you have made the onPlayerAdded() to listen for the PlayerAdded event.
The PlayerAdded event already has its own argument, the player, so, you don't need to call the function again. Even though the player resets, it will still work.
Here is the fix:
02 | local Players = game:GetService("Players") |
03 | local serverStorage = game:GetService("ServerStorage") |
05 | local items = serverStorage:WaitForChild("Items") |
06 | local effects = items:WaitForChild("Effects") |
07 | local sounds = effects:WaitForChild("Audio") |
08 | local audio = audio:WaitForChild("Sound") |
10 | local particles = effects:WaitForChild("Particles") |
11 | local particle1 = particles:WaitForChild("ParticleEmitter") |
12 | local particle2 = particles:WaitForChild("ParticleEmitter2") |
14 | local velocityFolder = effects:WaitForChild("Velocity") |
15 | local velocity = velocityFolder:WaitForChild("BodyVelocity") |
18 | function onPlayerAdded(player) |
19 | local newAudio = audio:Clone() |
20 | newAudio.Parent = Players |
23 | Players.PlayerAdded:Connect(onPlayerAdded) |
0