How can i save player accessories?

Help and Feedback Scripting Support
#1

Hello, i have problem with save player accessories and here my code:

local DataStoreService = game:GetService("DataStoreService")
local myDataStore = DataStoreService:GetDataStore("AccessoriesSaver")

game.Players.PlayerAdded:Connect(function(Player)
	
	local Data
	
	local Folder = Instance.new("Folder", Player)
	Folder.Name = "Items"
	
	pcall(function()
		
		Data = myDataStore:GetAsync(Player.UserId)
		
		for _, Item in pairs(Data) do
			local Value	= Instance.new("StringValue", Folder)
			Value.Name = Item
		end
		
		Player.CharacterAdded:Connect(function()
			for i, accessory in pairs(Folder:GetChildren()) do
				game.ServerStorage.Accessories[accessory.Name]:Clone().Parent = Player.Character
			end
		end)
	end)
end)

game.ReplicatedStorage.SaveEvent.OnServerEvent:Connect(function(Player)
	pcall(function()
		
		local Wear = {}
		
		for _, Item in pairs(Player.Items) do
			table.insert(Wear, Item.Name)
		end
		
		myDataStore:SetAsync(Player.UserId, Wear)
	end)
end)
2 Likes
#2

Are there any errors in the output?

1 Like
#3

No but the item value not saving and i don’t know why

#4

Have you enabled API in your studio?

1 Like
#5

Yes, but does not saving value

#6

Is it not saving when the player leaves? Or does it not load.

#7

Exactly when the player leaves

#8

Should use PlayerRemoved Event instead of a remote event.

#9

i guess he need use game.Players.PlayerRemoving:Connect(function()

#10

Try changing it to this script real quick so we can see the error.

game.ReplicatedStorage.SaveEvent.OnServerEvent:Connect(function(Player)
	local success, errorMessage = pcall(function()
		
		local Wear = {}
		
		for _, Item in pairs(Player.Items) do
			table.insert(Wear, Item.Name)
		end
		
		myDataStore:SetAsync(Player.UserId, Wear)
	end)
    if success then
      print("it works woohoo!")
    else
      error(errorMessage)
    end
end)
#11

I checked and it still doesn’t work

#12

Print the table when the player leaves.

#13

That’s the result

image

#14

Ah i know now do Player.Items:GetChildren().

#15

What’s the 33th line? Can you give us a screen shot?

#16

Here screenshot:

image

#17

Did changing it to :GetChildren() Work?

#18

Theres a typo, change it to:

for _, Item in pairs(Player.Items:GetChildren()) do
#19

No, the error said table error. (Nevermind)

#20

That’s works thank you so much