Keep getting "Players:GetUserIdFromNameAsync() failed: Unknown user"

RedKylegamer123 · August 26, 2021, 1:36am

What do you want to achieve?
A successful leaderboard without any issues.
What is the issue?
Whenever the leaderboard updates it gets this error

image755×57 3.47 KB

and it doesn’t put my friend on the leaderboard but it puts me on it.
What solutions have you tried so far?
Looked it up and couldn’t find anything useful. I printed the “data.key” to see what could be making the error. It shows up my userId and “-1” which I don’t know where that number came from.

local dss = game:GetService("DataStoreService")
local coinsLead = dss:GetOrderedDataStore("coinsLead")
local holder = workspace:WaitForChild("Map"):WaitForChild("Town Square"):WaitForChild("Leaderboards"):WaitForChild("Leaderboard1"):WaitForChild("Backdrop"):WaitForChild("SurfaceGui"):WaitForChild("Holder")

local function updateLeaderboard()
	local success, err = pcall(function()
		local data = coinsLead:GetSortedAsync(false, 5)
		local coinsPage = data:GetCurrentPage()
		for rank, data in pairs(coinsPage) do 
			print(data.key)
			local name = game.Players:GetNameFromUserIdAsync(data.key)
			local coins = data.value
			local isOnLeaderboard = false
			for i, v in pairs(holder:GetChildren()) do
				if v.Player.Text == name then
					isOnLeaderboard = true
					break
				end
			end

			if coins and isOnLeaderboard == false then
				local newLbFrame = game.ReplicatedStorage:WaitForChild("LeaderboardFrame"):Clone()
				newLbFrame.PlayerTL.Text = name
				newLbFrame.CoinsTL.Text = coins
				newLbFrame.RankTL.Text = "#" .. rank
				newLbFrame.Position = UDim2.new(0,0,newLbFrame.Position.Y.Scale + (.08 * #holder:GetChildren()), 0)
				newLbFrame.Parent = holder
			end
		end
	end)
	if not success then
		warn(err)
	end
end

while true do
	wait(5)
	for _, player in pairs(game.Players:GetPlayers()) do
		coinsLead:SetAsync(player.UserId, player.leaderstats.Coins.Value)
	end

	for _, frame in pairs(holder:GetChildren()) do
		frame:Destroy()
	end

	updateLeaderboard()
	print("Leaderboard updated")

	wait(120)

end

Nimcbyte · August 26, 2021, 1:50am

how about

nameFinder = getUsernameFromUserId(data.key)
name = game.Players:FindFirstChild(nameFinder)

RedKylegamer123 · August 26, 2021, 1:53am

It says “GetUsernameFromUserId is not a valid member of Players “Players””

Nimcbyte · August 26, 2021, 1:54am

edited it try it now it should work

RedKylegamer123 · August 26, 2021, 1:57am

Did the same thing

local nameFinder = game.Players:getUsernameFromUserId(data.key)
			local name = game.Players:FindFirstChild(nameFinder)

Nimcbyte · August 26, 2021, 1:59am

Oh wait my bad lol I completely told you the wrong thing
Ok I have an alternative

Players:GetPlayerByUserId(data.key)

RedKylegamer123 · August 26, 2021, 2:01am

Another error came from it.

What my code looks like now:

local dss = game:GetService("DataStoreService")
local coinsLead = dss:GetOrderedDataStore("coinsLead")
local holder = workspace:WaitForChild("Map"):WaitForChild("Town Square"):WaitForChild("Leaderboards"):WaitForChild("Leaderboard1"):WaitForChild("Backdrop"):WaitForChild("SurfaceGui"):WaitForChild("Holder")

local function updateLeaderboard()
	local success, err = pcall(function()
		local data = coinsLead:GetSortedAsync(false, 5)
		local coinsPage = data:GetCurrentPage()
		for rank, data in pairs(coinsPage) do 
			print(data.key)
			local nameFinder = game.Players:GetPlayerByUserId(data.key)
			local name = game.Players:FindFirstChild(nameFinder)
			local coins = data.value
			local isOnLeaderboard = false
			for i, v in pairs(holder:GetChildren()) do
				if v.Player.Text == name then
					isOnLeaderboard = true
					break
				end
			end

			if coins and isOnLeaderboard == false then
				local newLbFrame = game.ReplicatedStorage:WaitForChild("LeaderboardFrame"):Clone()
				newLbFrame.PlayerTL.Text = name
				newLbFrame.CoinsTL.Text = coins
				newLbFrame.RankTL.Text = "#" .. rank
				newLbFrame.Position = UDim2.new(0,0,newLbFrame.Position.Y.Scale + (.08 * #holder:GetChildren()), 0)
				newLbFrame.Parent = holder
			end
		end
	end)
	if not success then
		warn(err)
	end
end

while true do
	wait(5)
	for _, player in pairs(game.Players:GetPlayers()) do
		coinsLead:SetAsync(player.UserId, player.leaderstats.Coins.Value)
	end

	for _, frame in pairs(holder:GetChildren()) do
		frame:Destroy()
	end

	updateLeaderboard()
	print("Leaderboard updated")

	wait(120)

end

Nimcbyte · August 26, 2021, 2:03am

Oh I see now it says string expected as the error so,

local StringKey = tostring(data.key)
Players:GetPlayerByUserId(StringKey)

RedKylegamer123 · August 26, 2021, 2:07am

Still showing the save thing

local StringKey = tostring(data.key)
			local nameFinder = game.Players:GetPlayerByUserId(StringKey)
			local name = game.Players:FindFirstChild(nameFinder)

Nimcbyte · August 26, 2021, 2:22am

I can’t seem to pinpoint the issue here, but remove the Findfirstchild part.

Is the user ID value correct?

RedKylegamer123 · August 26, 2021, 2:27am

Tested it again, I don’t know what happened but the output changed after I removed FindFirstChild.

Nimcbyte · August 26, 2021, 2:29am

wait show me the script as it is right now

RedKylegamer123 · August 26, 2021, 2:33am

local dss = game:GetService("DataStoreService")
local coinsLead = dss:GetOrderedDataStore("coinsLead")
local holder = workspace:WaitForChild("Map"):WaitForChild("Town Square"):WaitForChild("Leaderboards"):WaitForChild("Leaderboard1"):WaitForChild("Backdrop"):WaitForChild("SurfaceGui"):WaitForChild("Holder")

local function updateLeaderboard()
	local success, err = pcall(function()
		local data = coinsLead:GetSortedAsync(false, 5)
		local coinsPage = data:GetCurrentPage()
		for rank, data in pairs(coinsPage) do 
			print(data.key)
			local StringKey = tostring(data.key)
			local nameFinder = game.Players:GetPlayerByUserId(StringKey)
			local name = game.Players.nameFinder
			local coins = data.value
			local isOnLeaderboard = false
			for i, v in pairs(holder:GetChildren()) do
				if v.Player.Text == name then
					isOnLeaderboard = true 
					break
				end
			end

			if coins and isOnLeaderboard == false then
				local newLbFrame = game.ReplicatedStorage:WaitForChild("LeaderboardFrame"):Clone()
				newLbFrame.PlayerTL.Text = name
				newLbFrame.CoinsTL.Text = coins
				newLbFrame.RankTL.Text = "#" .. rank
				newLbFrame.Position = UDim2.new(0,0,newLbFrame.Position.Y.Scale + (.08 * #holder:GetChildren()), 0)
				newLbFrame.Parent = holder
			end
		end
	end)
	if not success then
		warn(err)
	end
end

while true do
	wait(5)
	for _, player in pairs(game.Players:GetPlayers()) do
		coinsLead:SetAsync(player.UserId, player.leaderstats.Coins.Value)
	end

	for _, frame in pairs(holder:GetChildren()) do
		frame:Destroy()
	end

	updateLeaderboard()
	print("Leaderboard updated")

	wait(120)

end

Nimcbyte · August 26, 2021, 2:36am

local coinsLead = dss:GetOrderedDataStore("coinsLead")
local holder = workspace:WaitForChild("Map"):WaitForChild("Town Square"):WaitForChild("Leaderboards"):WaitForChild("Leaderboard1"):WaitForChild("Backdrop"):WaitForChild("SurfaceGui"):WaitForChild("Holder")

local function updateLeaderboard()
	local success, err = pcall(function()
		local data = coinsLead:GetSortedAsync(false, 5)
		local coinsPage = data:GetCurrentPage()
		for rank, data in pairs(coinsPage) do 
			print(data.key)
			local StringKey = tostring(data.key)
			local nameFinder = game.Players:GetPlayerByUserId(StringKey)
			local coins = data.value
			local isOnLeaderboard = false
			for i, v in pairs(holder:GetChildren()) do
				if v.Player.Text == name then
					isOnLeaderboard = true 
					break
				end
			end

			if coins and isOnLeaderboard == false then
				local newLbFrame = game.ReplicatedStorage:WaitForChild("LeaderboardFrame"):Clone()
				newLbFrame.PlayerTL.Text = name
				newLbFrame.CoinsTL.Text = coins
				newLbFrame.RankTL.Text = "#" .. rank
				newLbFrame.Position = UDim2.new(0,0,newLbFrame.Position.Y.Scale + (.08 * #holder:GetChildren()), 0)
				newLbFrame.Parent = holder
			end
		end
	end)
	if not success then
		warn(err)
	end
end

while true do
	wait(5)
	for _, player in pairs(game.Players:GetPlayers()) do
		coinsLead:SetAsync(player.UserId, player.leaderstats.Coins.Value)
	end

	for _, frame in pairs(holder:GetChildren()) do
		frame:Destroy()
	end

	updateLeaderboard()
	print("Leaderboard updated")

	wait(120)

end

use this instead

RedKylegamer123 · August 26, 2021, 2:45am

I’m not sure how you want me to replace it because dss (data store service) isn’t defined and “name” isn’t defined.

Nimcbyte · August 26, 2021, 2:57am

local name = game.Players:GetPlayerByUserId(StringKey)
just put that where the other one is

XDify01 · August 26, 2021, 3:07am

You should first check if the usernames are valid (try printing them and searching them)

RedKylegamer123 · August 26, 2021, 3:08am

Doing this again. I don’t understand why it isn’t working. I tried putting my userid directly but it gave the same error.

My code currently:

local dss = game:GetService("DataStoreService")
local coinsLead = dss:GetOrderedDataStore("coinsLead")
local holder = workspace:WaitForChild("Map"):WaitForChild("Town Square"):WaitForChild("Leaderboards"):WaitForChild("Leaderboard1"):WaitForChild("Backdrop"):WaitForChild("SurfaceGui"):WaitForChild("Holder")

local function updateLeaderboard()
	local success, err = pcall(function()
		local data = coinsLead:GetSortedAsync(false, 5)
		local coinsPage = data:GetCurrentPage()
		for rank, data in pairs(coinsPage) do 
			print(data.key)
			local StringKey = tostring(data.key)
			local name = game.Players:GetPlayerByUserId(StringKey)
			local coins = data.value
			local isOnLeaderboard = false
			for i, v in pairs(holder:GetChildren()) do
				if v.Player.Text == name then
					isOnLeaderboard = true 
					break
				end
			end

			if coins and isOnLeaderboard == false then
				local newLbFrame = game.ReplicatedStorage:WaitForChild("LeaderboardFrame"):Clone()
				newLbFrame.PlayerTL.Text = name
				newLbFrame.CoinsTL.Text = coins
				newLbFrame.RankTL.Text = "#" .. rank
				newLbFrame.Position = UDim2.new(0,0,newLbFrame.Position.Y.Scale + (.08 * #holder:GetChildren()), 0)
				newLbFrame.Parent = holder
			end
		end
	end)
	if not success then
		warn(err)
	end
end

while true do
	wait(5)
	for _, player in pairs(game.Players:GetPlayers()) do
		coinsLead:SetAsync(player.UserId, player.leaderstats.Coins.Value)
	end

	for _, frame in pairs(holder:GetChildren()) do
		frame:Destroy()
	end

	updateLeaderboard()
	print("Leaderboard updated")

	wait(120)

end

Abcreator · August 26, 2021, 3:08am

This won’t work as it requires the player to be in-game.

RedKylegamer123 · August 26, 2021, 3:10am

How would I print them? I don’t see where to get the usernames.