How would I go about making a "Ability Shop" [Solved]

[skyss_0fly]

I am after someone who knows how to basically have a script, not a local script inside a button, that upon click if. the player who clicks, has say 100 kills then they can equip the ability “kinetic”

this is what I got, doesn’t work tho, as always

local plrs = game:GetService("Players")

script.Parent.MouseButton1Click:Connect(function(plr)
	local player = plrs:GetPlayerFromCharacter(plr)
	if player.leaderstats.Kills.Values >= 50 then
		player.leaderstats.Ability.Value = "Kinetic"
	end
    
	
end)

Btw it doesn’t work because this error is what appears:
“Attempt to index nil with leaderstats” on line 5

[SOLVED]
thanks to:
xE_nte
EveningNighmare

[xE_nte]

@skyss_0fly

“Attempt to index nil with leaderstats” on line 5

That basicly means it can’t find the leaderstats folder of the player. (I guess you create it in another script) But the real problem ist

@skyss_0fly

local player = plrs:GetPlayerFromCharacter(plr)

You already get the player (plr) in the MouseButton1Click Funktion. So most likely player will be nil in your script. Try replacing player.leaderstats with plr.leaderstats

[skyss_0fly]

thanks I will try that, I appreciate this

[skyss_0fly]

Update, tried that and still same issue

code now:

script.Parent.MouseButton1Click:Connect(function(plr)
	if plr.leaderstats.Kills.Values >= 50 then
		plr.leaderstats.Ability.Value = "Kinetic"
	end
    
	
end)

[xE_nte]

try debugging the code with adding print(plr) or print(plr.Name), also do you create the leaderstats in another script?

[skyss_0fly]

yep, here’s leader stats script (in serverscriptsevice)

game.Players.PlayerAdded:Connect(function(plr)
	local leaderstats = Instance.new("Folder")
	leaderstats.Parent = plr
	leaderstats.Name = "leaderstats"
	
	local tempkills = Instance.new("IntValue")
	tempkills.Parent = leaderstats
	tempkills.Name = "KillsRound"
	tempkills.Value = 0
	
	local totalkills = Instance.new("IntValue")
	totalkills.Parent = leaderstats
	totalkills.Name = "Kills"
	totalkills.Value = 0
	
	local roundsplayed = Instance.new("IntValue")
	roundsplayed.Parent = leaderstats
	roundsplayed.Name = "Rounds"
	roundsplayed.Value = 0
	
	local ability = Instance.new("StringValue")
	ability.Parent = leaderstats
	ability.Name = "Ability"
	ability.Value = "Default"
end)

[EveningNighmare (ikm)]

MouseButton1Cliick event doesn’t return Player as an argument. You have to use LocalPlayer (and dont change leaderstats on the client side lol)

[skyss_0fly]

this script is not local script

[EveningNighmare (ikm)]

Find player in a different way, you cannot get player from Mousebtn1click event

[xE_nte]

@EveningNighmare

MouseButton1Cliick event doesn’t return Player as an argument.

woops forgot about that
You could the get the player by doing local player=script.ParentParent.Parent…
(depends on where the button is located)

[skyss_0fly]

ok well ill give this a try. let ya know if it works

[EveningNighmare (ikm)]

just use script:FindFirstAncestorOfClass(“Player”), it is a lot of shorter and better optimized

[skyss_0fly]

this works so now the code is:

script.Parent.MouseButton1Click:Connect(function(plr)
	local player = script:FindFirstAncestorOfClass("Player")
	if player.leaderstats.Kills.Value >= 50 then
		player.leaderstats.Ability.Value = "Diamond"
	end
    
	
end)

Posting this for others to find

[EveningNighmare (ikm)]

if someones confused how it works: its because this script is descendant of player