If I’m not mistaken, this is setting a players whole Datastore key to the value of toSaveData[1], this would completely erase any other values that were previously stored, or trying to store.
Secondly, you set a table with all of the values, but the values within this table will not update because you are not setting the table values upon the IntValue’s value changing. This would result in you storing the values that the player has joined with.
I would recommend you saving the players data upon PlayerRemoving. This would reduce the chance of getting throttled if an IntValue’s value were to change repeatedly. You should make the following edit:
local PlayerInfoTable = {}
game.Players.PlayerAdded:Connect(function(player)
PlayerInfoTable[player.UserId] = {clan.Value, money.Value, speed.Value, strength.Value, endurance.Value, stamina.Value, hunger.Value, rpname.Value, style.Value}
-------SAVE DATA ONE MORE TIME (WHEN THE PLAYER IS LEAVING THE GAME)-------
clan.Changed:Connect(function(value)
PlayerInfoTable[player.UserId][1] = value -- repeat changed event for the other IntValues
end)
end)
game.Players.PlayerRemoving:Connect(function(player)
local data = PlayerInfoTable[player.UserId]
if data then
sentodata:UpdateAsync(player.UserId, function(oldData)
return data
end)
end
end)
++ you should read up on this post:
PSA: Don't use Instance.new() with parent argument