How do I make a stat increasing tool?

removing holdername = nil might work and make it holdername. Otherwise, you can try this:

Step 1: Make a RemoteEvent called 'Give' and put it in ReplicatedStorage. Step 2: Make a script and name it 'Function' and place it in ServerScriptService. Here's the code for that:

local remote = game.ReplicatedStorage.Give

remote.OnServerEvent:Connect(function(Player)
    local plr = Player
    plr.leaderstats.Cash.Value = plr.leaderstats.Cash.Value + 1
end)

Step 3: Clearly the leaderboard in ServerScriptService:

game.Players.PlayerAdded:connect(function(p)
    local stats = Instance.new("IntValue")
    stats.Name = "leaderstats"
    stats.Parent = p

    local money = Instance.new("IntValue")
    money.Name = "Cash"
    money.Value = 0
    money.Parent = stats
end)

Step 4: Now create a screengui in StarterGui and name it 'popUp' Step 5: Then insert a 'TextLabel' (You can use a ImageLabel but u will have to change the code in the tool later on) into the popUp and set these properties:

• Visible: False
• TextScaled: True
• BackgroundTransparency = 1
• and any other settings should be okay, expect TextTransparency.

Step 6: Now name the text label 'ImageLabel' If you decided to use a ImageLabel instead, this step isn't needed.

Step 7: Create a tool with a handle.

Create a tool and name it 'Cash Giver' or whatever you please. then place a part inside it and name it handle, customize that handle, then place the tool in StarterPack.

Step 8: Add scripts and tool settings

Make sure the tool has these settings:

• RequiresHandle: True
• CanBeDropped: False
• Enabled: True
• ManualActivation: False
• Archivable: True

Now let's make the scripts:

Script 1: add a local script and name it 'Detector' then insert this code into it:

script.Parent.Unequipped:Connect(function()
    script.Parent.LocalScript.Disabled = true
end)

script.Parent.Equipped:Connect(function()
    script.Parent.LocalScript.Disabled = false
end)

Script 2: add a local script and do not name it, then insert this code into it:

local remote = game.ReplicatedStorage.Give

script.Parent.Equipped:Connect(function()
    if script.Parent.Equipped then
                game.Players.LocalPlayer:GetMouse().Button1Down:Connect(function()
                local plr = game.Players.LocalPlayer
                plr.PlayerGui.popUp.ImageLabel.Position = UDim2.new(0, plr:GetMouse().X, 0, plr:GetMouse().Y)
                plr.PlayerGui.popUp.ImageLabel.Visible = true
                script.Disabled = true
                remote:FireServer()
                for i = 1, 10 do
                    wait()
                    plr.PlayerGui.popUp.ImageLabel.TextTransparency = plr.PlayerGui.popUp.ImageLabel.TextTransparency - 0.1
                end
                wait(0.5)
                for i = 1, 10 do
                    wait()
                 plr.PlayerGui.popUp.ImageLabel.TextTransparency = plr.PlayerGui.popUp.ImageLabel.TextTransparency + 0.1
                end
                plr.PlayerGui.popUp.ImageLabel.Visible = false
                script.Disabled = false
            end)
        end
    end)

If you plan on using a ImageLabel instead, it's very simple to change:

-- Old:
plr.PlayerGui.popUp.ImageLabel.TextTransparency = plr.PlayerGui.popUp.ImageLabel.TextTransparency - 0.1
-- New:
plr.PlayerGui.popUp.ImageLabel.ImageTransparency =  plr.PlayerGui.popUp.ImageLabel.ImageTransparency - 0.1  
-- and clearly change where the + is instead of the -, change the wording to ImageTransparency.

Please accept if this worked! Also please give credit.

Explanation: 1.You need to capitalize the C's in Connect because that is the new form of connect. 2.Your holdername is being named Backpack because you need one more parent for the player. 3.You need to use :FindFirstChild to get any player in players for variables.

holdername = nil;

script.Parent.Equipped:Connect(function()
    holdername = script.Parent.Parent.Parent.Name;
end)

script.Parent.Unequipped:Connect(function()
    holdername = nil;
end)

script.Parent.Activated:Connect(function()
    -- game.Players:FindFirstChild(holdername).leaderstats["Cheezburgers"].Value = game.Players:FindFirstChild(holdername).leaderstats["Cheezburgers"].Value + 1;
    -- 14:33:38.767 - holdername is not a valid member of Players
    -- Judging by this, I cannot detect a player by their name using a string variable/value.
end)

script.Parent.Deactivated:Connect(function()
    print(holdername.." ate a cheezburger!")
end)