Crafting Script Deletes 2 Items

Help and Feedback Scripting Support
#21

Oh ok oof, no not because that , its trying to set locked parent

#22

Sorry can you explain what that means? and also could we see your crafting script too I think it its self is the main problem.

#23

Im not sure what that means myself but I did put the script, scroll to the post itself i put it under server script in serverscriptservice

#24 
game.Players.LocalPlayer.Inventory:FindFirstChild("Part"):Destroy()

So I see the inventory also has a copy when you make the axe how many are lost from the inventory?

#25

what do you mean lost from the inventory?

#26

I mean taken away or removed it says it destroys them during crafting so i mean how many are destroyed.

#27

when you craft, it destroyes 1 of the item

#28

Are you 100 percent sure and can you check? Like what if there is a bug in that part of the script too

#29

yeah I am sure, and I checked,

#30

Can you also check after you craft > drop an item (craft then drop item if that wasn`t clear).

#31

Yeah I did that, the one I drop just dissapears

#32

So in your inventory folder also losses a item not the gui the folder right?

#33

idk just nvm this anymore, Ill just rewrite the code , its unfixable it doesnt say where the warning coming from

#34

open output in studio and u can see where the warning is comming from

#35

My potato laptop crashes when I test in studio

#36

He`s already done that but maybe a quick fix can work for this?

#37

Ok i do have a quick fix for this i have a script i was recently working on:

for i,v in pairs(script.Parent:GetChildren()) do
wait()
if v:FindFirstChild("Drop") then
	v.Drop.MouseButton1Click:Connect(function()

		local Part = v.Stats.Item.Value
		local NewPart = workspace.Parent.ReplicatedStorage.DropItems[Part]:Clone()
		local Player = script.Parent.Parent.Parent.Parent
		local character = Player.Character
		local Yax = character.PrimaryPart.Position.Y
		
		NewPart.Parent = workspace
		NewPart.PrimaryPart.CFrame = CFrame.new(NewPart.PrimaryPart.Position, character.PrimaryPart.Position)
		NewPart.PrimaryPart.CFrame = character.PrimaryPart.CFrame:ToWorldSpace(CFrame.new(0,0,-5))

		Player.Inventory[Part].Value = Player.Inventory[Part].Value - 1
	end)
end

end

if you just remove drop in line 3 with frame or remove it entirely then change things like “Part = v.Stats.Item.Value” to another way of identifying item type like text is guess? after this where it states what “NewPart” is you change this to Player.Inventory[Part].

This is a cheep fix for my cheep script i think you should try this FOR NOW it dosnt see like a good idea permanently.

#38

okay thanks but where do I put it?

#39

Put it in the parent of the gui buttons (Item frame?) that drop the item but remember to remove line 3 and add in its place

if v.Name ~= “UIGridLayOut” then

also change part (Your gonna need to figuer out how to define it part needs to be the exact name of the part/item.

define player

and also add if Part:IsA(“Model”) then and if else as you can see this was made to deal with only models.

there is a few things to do so if you have any more problems just ask.