Weird Bug with a pop up script

Help and Feedback Scripting Support
#1

I have a pop up script that when a remote is fired it displays a text.

This is the part of the script where the bug is at :

ReplicatedStorage.Remotes.statUpRemote.OnClientEvent:Connect(function(amount : string, statName : string)
	if not amount or not statName then return end
	
	print(amount, statName) -- Will print 2, "Wins"
	
	local Template = script.Template:Clone()
	Template.Size = UDim2.new(0.01, 0, 0.01, 0)
	Template.Position = randomPosUI(script.Parent, Template)
	Template.Parent = script.Parent
	Template.Text = string.format("+%i %s", tonumber(amount), tostring(statName)) -- here

end)

local reward = Requirements[tostring(currentPlatform)].Reward * 2
print(reward)
Profiles.PlusStat(player, "Wins", reward) 
ReplicatedStorage.Remotes.statUpRemote:FireClient(player, tostring(reward), "Wins")

The template text is not set the the amount argument but to how much Wins the player has, which is not supposed to happen since I’m not referring the amount of wins.

This doesnt happen when I’m firing the remote event from the command bar, this is very weird.

image

image
image

2 Likes
#2

I don’t get what the problem is, do you want it to print “4 Wins”? Also, just wondering, but why are you sending reward as a string, instead of an integer?

#3

It’s something with the string.format()?

#4

Something to do with the template being cloned? What exactly is the template? Does the remote get fired once or multiple times?

#5

I was honestly thinking the same thing…

I don’t know what is broken to be honest with you, is the 4 wrong or the 2?

1 Like
#6

You need to be clearer with your forum posts so we can help you as much as possible.

What value is incorrect? 4 or 2?
What is the template?

#7

Another thing to mention, protect your remote event. An exploiter can specify what wins to display and it’ll do it accordingly.

#8

Specify the wins from the server?

#9

I want it to print 2 instead of 4,
2 is the amount argument
4 is the amount of wins (that i never refer)

#10

I tried removing it before it make a difference

#11

Template is a text label, the incorrect value is 4

#12

An exploiter cant :FireClient()

#13

I would use a breakpoint and add a watch to the amount variable. Something funky is happening with the print.format.

#14

I’ve tried removing the string and just making one single argument called text and it still prints the wins amount instead of the text argument

#15

Is the 4 the default template text? Where is it getting 4 from?

#16

Make the actual text just "", an empty string. If it’s still 4 then something’s changing it to that number.

#17

The template text is “+2 wins”, the 4 is the amount of wins the player has for some reason…

#18

Still… [Character limit asqdsq]

#19

I’ve tried debugging and added this script :

game.Players.PlayerAdded:Connect(function(plr)
	while task.wait(1) do
		game.ReplicatedStorage.Remotes.statUpRemote:FireClient(plr, 10, "Test")
	end
end)

it shows 10 and its perfectly fine however when it’s in the other script it does that bug…

#20

I’ve managed to find the solution

It was because of the text label template for some reason I still have to figure out why.