Leaderstats not being vaild

Help and Feedback Scripting Support
1

This script makes it so if you Hit the part that it would open up a certain GUI depending on how much of a certain leaderstats you have (Coins).

But I get this error.

Error21

local part = script.Parent

part.Touched:Connect(function(hit)
	local plr = hit.Parent:FindFirstChild('Humanoid')
	if hit.Parent:FindFirstChild('Humanoid')and plr.leaderstats.Coins.Value <= 100 then
		plr.PlayerGui.HaveCoins.Visible = true
	elseif hit.Parent:FindFirstChild('Humanoid')and plr.leaderstats.Coins.Value >= 100 then
		plr.PlayerGui.NotHaveCoins.Visible = true
	end
end)
2

Where are your leaderstats located - in the player Model or within the Players service? With the assumption that your leaderstats are found within the Player Object (of Players service) then use this function:

local player = Players:GetPlayerFromCharacter(part.Parent)

1 Like
3

You’re trying to retrieve leaderstats from your humanoid. That isn’t how it works. Since it’s located in the player, you must get the player from the character model. So, do this:
local plr = game.Players:GetPlayerFromCharacter(hit.Parent)

2 Likes
4

Ok at @ItzMeZeus_IGotHacked and @kboy100

I did you edits and not I got this error.

Error22
Error221340Ă—41 2.7 KB

New Script

local part = script.Parent

part.Touched:Connect(function(hit)
	local plr = game.Players:GetPlayerFromCharacter(hit.Parent)
	if hit.Parent:FindFirstChild('Humanoid')and plr.leaderstats.Coins.Value <= 100 then
		plr.PlayerGui.HaveCoins.Visible = true
	elseif hit.Parent:FindFirstChild('Humanoid')and plr.leaderstats.Coins.Value >= 100 then
		plr.PlayerGui.NotHaveCoins.Visible = true
	end
end)
5

You spelt HaveCoins, not NotHaveCoins.

6

Where in the script? Like in which line?

7

Can’t you notice it? It’s literally at around middle of the code.

8

If I change its the same error but now with is saying HaveCoins and not NotHaveCoins

9

When you create the leaderstats folder, where are you parenting it to?

Please also provide some code.

10

The Player of in this case Madden21fan (Me)

11

Maybe you forgot to name the ScreenGui, if there is one

local part = script.Parent

part.Touched:Connect(function(hit)
	local plr = game.Players:GetPlayerFromCharacter(hit.Parent)
	if not plr then				return				end
	
	local Screen = plr.PlayerGui[""] -- ScreenGui name
	if plr.leaderstats.Coins.Value <= 100 then
		Screen.HaveCoins.Visible = true
	elseif plr.leaderstats.Coins.Value >= 100 then
		Screen.NotHaveCoins.Visible = true
	end
end)
12

There is one. It is there I spent like 30 min on it…

13

you do know that if coins is 100, then both if statements will run

14

I don’t know, this is the script he gave :woman_shrugging:

15

It should be >=100 and <100

You should have helped me correct my mistake.

16

So this script works.But it gives up the same message no matter how many coins you have.

local part = script.Parent

part.Touched:Connect(function(hit)
	local plr = game.Players:GetPlayerFromCharacter(hit.Parent)
	if not plr then				return				end

	local Screen = plr.PlayerGui["TeleporterGUI"] -- ScreenGui name
	if plr.leaderstats.Coins.Value < 100 then
		Screen.NotHaveCoins.Visible = true
	elseif plr.leaderstats.Coins.Value >= 100 then
		Screen.HaveCoins.Visible = true
	end
end)

P.S. So I have like a 1,400 coins and I change it to 99 and I wonder if it didn’t register.

Ok with a little bit of print magic apparently it didn’t register.