My equipped tool is not saving?

I have a script that is supposed to save what the player has equipped. When a player equips a tool, it works. It is a GUI button that connects to a remote event. There are no errors, but however it does not save the players last equipped primary and secondary. API services are enabled in the place. This is the script:

local new = Instance.new

local datastore = game:GetService("DataStoreService"):GetDataStore("LocalPlayerPrimary")
local updateAsync = datastore.UpdateAsync
local datastore2 = game:GetService("DataStoreService"):GetDataStore("LocalPlayerSecondary")
local updateAsync2 = datastore2.UpdateAsync

local loadData1 = function(player)
	local id = player.UserId
	local data

	local success,output = pcall(function()
		data = datastore:GetAsync(id) 
	end)

	assert(success,output)

	if not data then
		data = {
			PrimaryEquipped = "Laser Gun",
		}
	end	


	player.PrimaryEquipped.Value = data.PrimaryEquipped
end

local loadData2 = function(player)
	local id = player.UserId
	local data

	local success,output = pcall(function()
		data = datastore2:GetAsync(id) 
	end)

	assert(success,output)

	if not data then
		data = {
			SecondaryEquipped = "Speed Coil",
		}
	end	


	player.SecondaryEquipped.Value = data.SecondaryEquipped
end

local saveData1 = function(player)
	local id = player.UserId
	local data = {
		PrimaryEquipped = "",
	}



	data.PrimaryEquipped = player.PrimaryEquipped.Value

	local success,output = pcall(updateAsync,datastore,id,function(old)
		return data
	end)

	assert(success,output)
end

local saveData2 = function(player)
	local id = player.UserId
	local data = {
		SecondaryEquipped = "",
	}



	data.SecondaryEquipped = player.SecondaryEquipped.Value

	local success,output = pcall(updateAsync2,datastore2,id,function(old)
		return data
	end)

	assert(success,output)
end

updatePri = function(player)
	
	local clone = game.ReplicatedStorage.Items:FindFirstChild(player.PrimaryEquipped.Value)
	local tool = clone:Clone()
	tool.Parent = player:WaitForChild("Backpack")


	
end

updateSec = function(player)

	local clone = game.ReplicatedStorage.Items:FindFirstChild(player.SecondaryEquipped.Value)
	local tool = clone:Clone()
	tool.Parent = player:WaitForChild("Backpack")



end

local onJoin = function(player)

	-- // Equipped Character Value
	local p = new("StringValue")
	p.Parent = player
	p.Name = "PrimaryEquipped"

	local s = new("StringValue")
	s.Parent = player
	s.Name = "SecondaryEquipped"
	
	loadData1(player)
	loadData2(player)

end



game.Players.PlayerAdded:Connect(onJoin)
game.Players.PlayerRemoving:Connect(saveData1, saveData2)

Can anyone help? Thank you.