Remote event returning player instead of number?

Help and Feedback Scripting Support
,
#1

I’m trying to make a shop sell system, but transfering the money to the leaderstats isn’t working.
the value “Money” isn’t actually the number but the player for some reason.

Client script:

script.Parent.InputBegan:Connect(function(obj)
	if obj.UserInputType == Enum.UserInputType.MouseButton1 then
		local target = game.Players.LocalPlayer.Inventory:FindFirstChild(script.Parent.Title.Text)
		game.ReplicatedStorage.AddMoney:FireServer(game.Players.LocalPlayer,tonumber(target))
		target:Destroy()
		script.Parent:Destroy()
	end
end)

Server script: (only important part)

game.ReplicatedStorage.AddMoney.OnServerEvent:Connect(function(plr,money)
	plr.leaderstats.Money.Value = plr.leaderstats.Money.Value + money
	local ui = Instance.new("ScreenGui")
	ui.Parent = plr.PlayerGui
	local text = Instance.new("TextLabel")
	text.TextColor3 = Color3.fromRGB(170, 170, 170)
	text.TextScaled = true
	text.Font = Enum.Font.FredokaOne
	text.BackgroundTransparency = 1
	text.Position = UDim2.new(math.random(1,10)/10,0,1,0)
	text.Parent = ui
	text.Text = "+"..money
	text.Size = UDim2.new(0, 200,0, 50)
	local tweenp = game:GetService("TweenService"):Create(text,TweenInfo.new(2,Enum.EasingStyle.Sine),{Position = UDim2.new(text.Position.X,0,0.5,0)})
	local tweent = game:GetService("TweenService"):Create(text,TweenInfo.new(2,Enum.EasingStyle.Linear),{TextTransparency = 1})
	tweenp:Play()
	tweent:Play()
	game:GetService("Debris"):AddItem(ui,2)
end)
#2 
	game.ReplicatedStorage.AddMoney:FireServer(tonumber(target))
#3

When a client fires the server, the server automatically receives the player as the first argument, you don’t need to pass it. Replace that line with:

game.ReplicatedStorage.AddMoney:FireServer(tonumber(target))
#4

If i print it from the server side it returns nil for some reason now.

#5

Did you try printing tonumber(target) on the client side first? It might not be the remote event where the problem is

#6

Done. And they both print nil.

#7

Oh no found the problem!:
In the LocalScript the target is set as :FindFirstChild(script.Parent.Title.**Text**)
Its supposed to be:
:FindFirstChild(script.Parent.Title).**Text**

#8

Just realised i cant set bold in code texts.

#9

Now it says “attempt to index nil with ‘Text’”

#10

That’s probably because :FindFirstChild() couldn’t find what you were looking for

#11

Nevermind. Its still wrong i’m reading the code wrong

#12

Well we know the findfirstchild isn’t finding it now.

#13

What are you trying to do? What are you trying to find?

#14

A Numbervalue in a folder located inside the player object.

#15

Its trying to get the value of that and then add that to the players balance.

#16

Are you sure, that when the script runs, the number value will even be there?

#17

Yes i am.
It needs to be there, im even checking in the explorer

#18

And how is script.Parent.Title that number value?

#19

It’s finding it perfectly its even deleting it after the click, its because of the remote event then.

#20

Its the name of it.

charcharcha