FireClient not firing? (extra text to look like a question)
so i got this script that when a player presses a certain button if they have the right amount of experience it will fireclient and then it is supposed to take away that set amount of experience and set their speed value to true but it doesnt it also doesnt print any errors
script:
01 | game.Players.PlayerAdded:Connect(function(plr) |
02 | local skill = plr.Experience.Skill |
03 | local speed = plr.Speed |
04 | script.Parent.Activated:Connect(function() |
05 | if skill.Value >=20000 then |
06 | local Twenty = speed.Twenty.Value |
07 | game.ReplicatedStorage.SpeedEvents.Twenty:FireClient(plr,Twenty) |
08 | else |
09 | script.Parent.Parent.Parent.XpWarning.Visible = true |
10 | wait(2.5) |
11 | script.Parent.Parent.Parent.XpWarning.Visible = false |
12 | end |
13 | end) |
14 | end) |
localscript:
1 | game.ReplicatedStorage.SpeedEvents.Twenty.OnClientEvent:Connect(function(Twenty) |
2 | game.Players.LocalPlayer.Experience.Skill.Value = game.Players.LocalPlayer.Experience.Skill.Value - 20000 |
3 | Twenty = true |
4 | end) |
still quite new at remote events
0
0
1 answer
With "FilteringEnabled" enabled, things change up a bit. And I'm pretty sure that you cannot change values using localscripts as only the client can see this and would probably cause issues.
Some of it needs re-editing and seeing that it's a tool your trying to use, your localscript should be the one firing a RemoteEvent rather than the other way around. Something to keep in mind is that when you fire a RemoteEvent or RemoteFunction, it will always give you the player who fired the RemoteEvent/RemoteFunction as the first parameter.
For your localscript, you don't need much code to run it since all your doing is activating your tool. Here's the simple localscript you need with some good notes (put it into the tool):
1 | script.Parent.Activated:Connect(function() |
2 | game.ReplicatedStorage.SpeedEvents.Twenty:FireServer() |
3 | --Make sure that what you're firing is a RemoteEvent and *NOT* a RemoteFunction. |
4 | --Also, you don't want any "if" statements in here because the "client" (or the user) can still fire the RemoteEvent whether if it meets the requirements or not (as an exploit issue fix) |
5 | end) |
Your script is supposed to be the one doing all of the work (such as doing "if" statements and stuff). Also since the first parameter of your function is which player fired the RemoteEvent, you can use it to check for anything within it (it works like "game.Players.Localplayer") but you cannot use "game.Players.LocalPlayer" with a script since it runs on the server and not on the client so it wouldn't know which player your talking about. Here's the modified script:
01 | game.ReplicatedStorage.SpeedEvents.Twenty.OnServerEvent:Connect(function(plr) |
02 | local skill = plr.Experience.Skill |
03 | local speed = plr.Speed |
04 |
05 | if skill.Value >=20000 then |
06 | plr.Experience.Skill.Value = plr.Experience.Skill.Value - 20000 |
07 | speed.Twenty.Value = true |
08 | else |
09 | script.Parent.Parent.Parent.XpWarning.Visible = true |
10 | wait(2.5) |
11 | script.Parent.Parent.Parent.XpWarning.Visible = false |
12 | end |
13 | end) --The player is given always as the first parameter, even when you input parameters yourslef such as ":FireServer(money, exp)" and will always come out somthing like ".OnServerEvent:Connect(function(Player, money, exp)" |
If you have any questions or issues, please contact me. ;)