Ammo Box Help (value and basic scripting)

Help and Feedback Scripting Support
#21

Did you look at the value before and after you ran the script?
Edit: and max ammo is a int value or number value?

#22

it’s a numbervalue and i’m not sure i understand your first question

#23

what happened

also this happens when i add the “end)” at the for i part

#24 
script.Parent.MouseClick:Connect(function(plr)
	for _,tool in ipairs(plr.Backpack:GetChildren()) do

		local MaxAmmo = tool:FindFirstChild("MaxAmmo")

		if tool:IsA("Tool") and MaxAmmo then
print(MaxAmmo.Value)
			MaxAmmo.Value += 12
			print(MaxAmmo.Value)
		end
	end
end)

And see if the print’s value changes.
(The script above is where click detector is the parent of the script)

#25

nothing even happened. i clicked it with the gun unequipped and same problem. there was nothing in the output and the MaxAmmo value didn’t change at all

#26

Well it works for me so that’s confusing.

#27

fake ammo
fake ammo457×1079 70.9 KB

#28

wait maybe because it doesn’t work cause i unequip

#29

Wait…it’s in the character, not in the player (the tool) that’s why is isn’t working.

#30

oh yeah i use this give part to give the tool to the player.

#31

it still doesn’t work even if it put it in starterpack

#32

It should be putting the tool in the player’s backpack, not in the character.

#33

how do i fix that?

the code is still

script.Parent.MouseClick:Connect(function(plr)
	for _,tool in ipairs(plr.Backpack:GetChildren()) do

		local MaxAmmo = tool:FindFirstChild("MaxAmmo")

		if tool:IsA("Tool") and MaxAmmo then
			print(MaxAmmo.Value)
			MaxAmmo.Value += 12
			print(MaxAmmo.Value)
		end
	end
end)
#34

What’s the code for the tool giver?

#35

local Detector = script.Parent.ClickDetector
local Tool = game.ServerStorage.G16A1
local Part = script.Parent

Detector.MouseClick:Connect(function(Player)
local Clone = Tool:Clone()
Clone.Parent = Player.Character
Part:Destroy()
end)

#36

Make it this:

local Detector = script.Parent.ClickDetector
local Tool = game.ServerStorage.G16A1
local Part = script.Parent

Detector.MouseClick:Connect(function(Player)
local Clone = Tool:Clone()
Clone.Parent = Player.BackPack
Part:Destroy()
end)
#37

If you keep the giver script the same make the box script this:

script.Parent.MouseClick:Connect(function(plr)
	for _,tool in ipairs(plr.Character:GetChildren()) do

		local MaxAmmo = tool:FindFirstChild("MaxAmmo")

		if tool:IsA("Tool") and MaxAmmo then
			MaxAmmo.Value += 12
		end
	end
end)
#38

the giver brick script doesn’t work

#39

Whoops, it’s “Backpack”, not “BackPack”

#40

it destroys the part but doesn’t give the tool inside the inventorhy.

edit: nvm it was just the custom inventory gui that made it not work