How do I fix these tools?

[NotAid_n]

So I have these two tools that for some reason won’t let me grab it twice and randomly gives it to other people.

Here are all the scripts:

Plate Of Steak Script:

local tool = game.Lighting["Raw Big Boi Steak"]
local klone = tool:clone()

script.Parent.ClickDetector.MouseClick:connect(function(plr)
     if klone.Parent ~= plr.Backpack then
         klone.Parent = plr.Backpack
     end
 end)

Plate Of Wings Script:

local tool = game.ServerStorage["Raw Chicken Wing"]
local klone = tool:clone()

script.Parent.ClickDetector.MouseClick:connect(function(plr)
     if klone.Parent ~= plr.Backpack then
         klone.Parent = plr.Backpack
     end
 end)

The Tools don’t have any scripts (Accept some that don’t matter like animation)

[JollyGameCrazy (ArtificialIntellect)]

Why do you have tools under lighting?

[ramdom_player201]

You only defined the tool once and so it is trying to parent the same tool. Make it define the tool in the function

[NotAid_n]

I don’t understand. Can you explain?

[ramdom_player201]

@AidanPlaysYT_Real

local tool = game.Lighting["Raw Big Boi Steak"]
local klone = tool:clone()

script.Parent.ClickDetector.MouseClick:connect(function(plr)
     if klone.Parent ~= plr.Backpack then
         klone.Parent = plr.Backpack
     end
 end)

‘Klone’ is defined once. But this means that the 2nd time the function is activated, it will try to parent the exact same tool

[ramdom_player201]

Move ‘local Klone’ into the function

script.Parent.ClickDetectore.MouseClick:Connect(function(plr)
    local klone = tool:Clone()
    -- rest of code

[NotAid_n]

@ramdom_player201

local klone = tool:Clone()

lemme see if that works, thank you.

[NotAid_n]

it didn’t work at all. Maybe make it a local script?

[ThoseNamesAreGood]

Because you’re checking if it is bigger than nil, It checks 1 time and since nothing is in you’re inventory but the 2 time it is already in you’re inventory and you’re checking if it is nil.

Try remove the if nil.

[ramdom_player201]

Did you move it into the function like I said? Because it should work if you did that…

[NotAid_n]

the script right now:

local tool = game.ServerStorage["Raw Chicken Wing"]

script.Parent.ClickDetector.MouseClick:connect(function(plr)
	local klone = tool:clone()
	if klone.Parent ~= plr.Backpack then
		klone.Parent = plr.Backpack
	end
end)

[ramdom_player201]

Yes, that is what I meant, does it work now?

[NotAid_n]

No, i tried it like that and it didnt work.

[ramdom_player201]

Are there any errors in the output?

[NotAid_n]

No, No errors appeared when i tested.

edit: Ima try putting it as a LocalScript

[ramdom_player201]

Maybe add print statements, to see which parts of the code runs successfully

[Synnwave]

If the tools are in serverstorage, don’t use a local script, use a remote event

[NotAid_n]

one of them is in serverstorage the other was in lighting