✓ ANSWER ACCEPTED

(Solved!) Not Adding Items To Inventory? No Output Errors!

Asked Feb 7, 2019471 views2 answers

The script isn't adding the items to the inventory as asked and there's no output errors!

There's probably something way wrong with this script but here it is!

--//ITS A LOCAL SCRIPT\--

local player = game.Players.LocalPlayer
local template = script.Parent.Parent.Template

wait(player.CharacterAdded)

function addItems()
	local invItems = player.Inventory:GetChildren()
	if invItems == "TestItem" then
		local clonedTemp = template:Clone()
		clonedTemp.Parent = script.Parent
		clonedTemp.InfoValues.Gives.Value = 1
		clonedTemp.InfoValues.Name.Value = "TestItem"
		clonedTemp.Visible = true
	end
end

addItems()

AnonymousFeb 7, 2019Edited Feb 9, 2019

2 answers

✓ Accepted answer

Despite incapaxx's comment (I literally have no idea what he's talking about; he's probably high again), :GetChildren() returns a table. This means that invItems will never be equal to TestItem. If you printed it, you'd get a TableId outputted. I'm assuming Inventory is a folder or any data container that you've created, and you want to scan inside it for the so called TestItem. To accomplish this, you'll have to loop through the table returned by :GetChildren(), i.e. invItems.

local player = game.Players.LocalPlayer
local template = script.Parent.Parent.Template

wait(player.CharacterAdded)

function addItems()
    local invItems = player.Inventory:GetChildren()
    for _,v in pairs(invItems) do
       if v.Name == "TestItem" then
           local clonedTemp = template:Clone()
           clonedTemp.Parent = script.Parent
           clonedTemp.InfoValues.Gives.Value = 1
           clonedTemp.InfoValues.Name.Value = "TestItem"
           clonedTemp.Visible = true
       end
    end
end

addItems()

Miniller did actually have the right idea. He simply didn't elaborate on it enough.

##Accept and upvote if this helps!

AnonymousFeb 7, 2019Edited Feb 7, 2019Permalink

5 comments

It's not working

AnonymousFeb 7, 2019#

is there a child in Inventory called TestItem and are there any errors?

AnonymousFeb 7, 2019#

changed line 9, my bad. try now unless there are different errors

AnonymousFeb 7, 2019#

i'm just better than u and i'm not high

AnonymousFeb 7, 2019#

It works now

AnonymousFeb 7, 2019#

Problem 01:Line 08 is not working, because player.Inventory:GetChildren() returns a table. Problem 02:It's Backpack, not Inventory, Problem 03: I don't think wait(player.CharacterAdded) is working, because there you need to write a number... For that, you need to add an event like this:

player.CharacterAdded:Connect(function()
	--put all your code (except the line01) there!
end)

For the other problems I said, the backpack and the table thing, use

for i = 1, #player.Backpack:GetChildren() do
	print(player.Backpack:GetChildren()[i])
end
--This code will print all tools in backpack.

instead of what you used. Also, where is this script at? I need to know that! Tell me where is the script, and I can help you to re-script that.

AnonymousFeb 7, 2019Edited Feb 7, 2019Permalink

3 comments

invItems will only hold the first return value of unpack

AnonymousFeb 7, 2019#

Then how to detect all items?

AnonymousFeb 7, 2019#

incapaz is rarted

AnonymousFeb 7, 2019#

This discussion is preserved as it was. New answers, comments, and votes are closed.