Attempt to perform arithmetic (sub) on number and nil

Help and Feedback Scripting Support
#21

Indeed he has, and he has to change that approach. :slight_smile:

#22

tonumber(gems) is nil, which means that whatever gems is, it can’t be converted to a number. This is probably due to an error with RemoteEvent arguments.
Change this line here:
AddTrails.OnServerEvent:Connect(function(gems)
into this:
AddTrails.OnServerEvent:Connect(function(player, gems)
and let me know how it goes.

#23

Yeah, he probably assigned player as gems.

#24

Ok, I tried that, but it gives me the same exact error.

#25

Try to remove all the tonumber() and approach to @JarodOfOrbiter method

#26

OnServerEvent gets the player parameter first not gems. Do:

AddTrails.OnServerEvent:Connect(function(player, gems)
	addTrails(gems)
end)
#27

He tried that and got error again.

#28

Maybe try

button.MouseButton1Click:Connect(function()
      local gems = 50
      AddTrail:FireServer(game.Players.LocalPlayer, gems)
end)

AddTrails.OnServerEvent:Connect(function(player, gems)
#29

You’re calling the player parameter in this function without defining the player. Is player already defined?

#30

Yeah I think it would be defined and currency is a number, that means it is defined, gems is nil.

#31

I mean Curreny.Value could also be nil

#32

Yes it is defined, I used it to set the database for the trails (the thing I’m working on)

1 Like
#33

Nope, it is a number, error is saying gems is nil.

#34

Maybe try this method, Idk if it will work.

#35

Can you try printing gems? I’m not sure by can you remove the tonumber before gems, it won’t make a difference but try print just “gems” without tonumber.

#36

Print gems, we need to know what gems is.

#37

He tried it and it was nil though.

#38

This is most likely going to work but instead rather than passing the player parameter when firing, pass the player parameter when connecting the event. It solved a few solutions for me in the past because player trying to be accessed and the player given by the remote event might be different since it’s a server script etc.

#39

It says that it is nil

(character min)

#40

I dont know if this will actually work though.