What am I doing wrong that's not saving local function saveInventory(Player)?

local DataStore = game:GetService("DataStoreService"):GetDataStore("InventoryKey_Version" .. game.ServerScriptService.DataStoreService.Service.Version.Value)
local SlotDataStore = game:GetService("DataStoreService"):GetDataStore("SlotKey_Version" .. game.ServerScriptService.DataStoreService.Service.Version.Value)

local HTTP = game:GetService("HttpService")
local itemModule = require(game.ReplicatedStorage.Assets.Objects.Utility.Items)

game:GetService("Players").PlayerAdded:Connect(function(Player)
    local newData = DataStore:GetAsync(Player.UserId)
    local slotData = SlotDataStore:GetAsync(Player.UserId)

    local Folder = Instance.new("Folder")
    Folder.Parent = game.ServerStorage.Assets.Inventory
    Folder.Name = Player.Name

    local toolFolder = Instance.new("Folder")
    toolFolder.Parent = Folder
    toolFolder.Name = "Tools"

    local Slot1 = Instance.new("Folder")
    Slot1.Parent = toolFolder
    Slot1.Name = "Slot1"

    local Slot2 = Instance.new("Folder")
    Slot2.Parent = toolFolder
    Slot2.Name = "Slot2"

    local Slot3 = Instance.new("Folder")
    Slot3.Parent = toolFolder
    Slot3.Name = "Slot3"

    local Slot4 = Instance.new("Folder")
    Slot4.Parent = toolFolder
    Slot4.Name = "Slot4"

    local Slot5 = Instance.new("Folder")
    Slot5.Parent = toolFolder
    Slot5.Name = "Slot5"

    local Slot6 = Instance.new("Folder")
    Slot6.Parent = toolFolder
    Slot6.Name = "Slot6"

    if slotData then
        slotData = HTTP:JSONDecode(slotData)

        wait(0.5)

        if slotData[game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot1:GetChildren()] then
            print(slotData)
            for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot1:GetChildren()) do
                if v:IsA("StringValue") then
                    v.Name = v.Value
                end
                for i, v2 in pairs(v:GetChildren()) do
                    if v:IsA("IntValue") then
                        v.Name = v.Amount.Value
                    end
                end
            end
        end

        if slotData[game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot2:GetChildren()] then
            for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot2:GetChildren()) do
                if v:IsA("StringValue") then
                    v.Name = v.Value
                end
                for i, v2 in pairs(v:GetChildren()) do
                    if v:IsA("IntValue") then
                        v.Name = v.Amount.Value
                    end
                end
            end
        end

        if slotData[game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot3:GetChildren()] then
            for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot3:GetChildren()) do
                if v:IsA("StringValue") then
                    v.Name = v.Value
                end
                for i, v2 in pairs(v:GetChildren()) do
                    if v:IsA("IntValue") then
                        v.Name = v.Amount.Value
                    end
                end
            end
        end

        if slotData[game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot4:GetChildren()] then
            for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot4:GetChildren()) do
                if v:IsA("StringValue") then
                    v.Name = v.Value
                end
                for i, v2 in pairs(v:GetChildren()) do
                    if v:IsA("IntValue") then
                        v.Name = v.Amount.Value
                    end
                end
            end
        end

        if slotData[game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot5:GetChildren()] then
            for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot5:GetChildren()) do
                if v:IsA("StringValue") then

                    v.Name = v.Value
                end
                for i, v2 in pairs(v:GetChildren()) do
                    if v:IsA("IntValue") then
                        v.Name = v.Amount.Value
                    end
                end
            end
        end

        if slotData[game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot6:GetChildren()] then
            for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot6:GetChildren()) do
                if v:IsA("StringValue") then
                    v.Name = v.Value
                end
                for i, v2 in pairs(v:GetChildren()) do
                    if v:IsA("IntValue") then
                        v.Name = v.Amount.Value
                    end
                end
            end
        end
    end

    if newData then
        newData = HTTP:JSONDecode(newData)
        wait(0.5)

        if newData.Apple ~= nil then
            if newData.Apple > 0 then
                local Apple = Instance.new("StringValue")
                Apple.Parent = Folder
                Apple.Name = "Apple"
                local Amount = Instance.new("IntValue")
                Amount.Parent = Apple
                Amount.Name = "Amount"
                Amount.Value = newData.Apple
            end
        end

        if newData["Magic Potion"] ~= nil then
            if newData["Magic Potion"] > 0 then
                local MP = Instance.new("StringValue")
                MP.Parent = Folder
                MP.Name = "Magic Potion"
                local Amount = Instance.new("IntValue")
                Amount.Parent = MP
                Amount.Name = "Amount"
                Amount.Value = newData["Magic Potion"]
            end
        end

        if newData["Lesser Healing Potion"] ~= nil then
            if newData["Lesser Healing Potion"] > 0 then
                local LHP = Instance.new("StringValue")
                LHP.Parent = Folder
                LHP.Name = "Lesser Healing Potion"
                local Amount = Instance.new("IntValue")
                Amount.Parent = LHP
                Amount.Name = "Amount"
                Amount.Value = newData["Lesser Healing Potion"]
            end
        end

        if newData["Healing Potion"] ~= nil then
            if newData["Healing Potion"] > 0 then
                local HP = Instance.new("StringValue")
                HP.Parent = Folder
                HP.Name = "Lesser Healing Potion"
                local Amount = Instance.new("IntValue")
                Amount.Parent = HP
                Amount.Name = "Amount"
                Amount.Value = newData["Healing Potion"]
            end
        end

        if newData["Greater Healing Potion"] ~= nil then
            if newData["Greater Healing Potion"] > 0 then
                local GHP = Instance.new("StringValue")
                GHP.Parent = Folder
                GHP.Name = "Greater Healing Potion"
                local Amount = Instance.new("IntValue")
                Amount.Parent = GHP
                Amount.Name = "Amount"
                Amount.Value = newData["Greater Healing Potion"]
            end
        end     

        if newData["Magnus Shirt"] ~= nil then
            if newData["Magnus Shirt"] > 0 then
                local MS = Instance.new("StringValue")
                MS.Parent = Folder
                MS.Name = "Magnus Shirt"
                local Amount = Instance.new("IntValue")
                Amount.Parent = MS
                Amount.Name = "Amount"
                Amount.Value = newData["Magnus Shirt"]
            end
        end

        if newData["Magnus Pants"] ~= nil then
            if newData["Magnus Pants"] > 0 then
                local MPTS = Instance.new("StringValue")
                MPTS.Parent = Folder
                MPTS.Name = "Magnus Pants"
                local Amount = Instance.new("IntValue")
                Amount.Parent = MPTS
                Amount.Name = "Amount"
                Amount.Value = newData["Magnus Pants"]
            end
        end 
    end
end)

local function saveInventory(Player)

    print('SaveInventory has been called')

    local Inv = {}
    local Char = {}
    local Tools = {}

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name):GetChildren()) do
        if v:FindFirstChild("Amount") then
            Inv[v.Name] = v.Amount.Value
        end
    end

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot1:GetChildren()) do
        if v:IsA("StringValue") then
            Char[v.Name] = v.Value
        end
        for i, v2 in pairs(v:GetChildren()) do
            if v:IsA("IntValue") then
                Char[v.Name] = v.Amount.Value
            end         
        end
    end 

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot2:GetChildren()) do
        if v:IsA("StringValue") then
            Char[v.Name] = v.Value
        end
        for i, v2 in pairs(v:GetChildren()) do
            if v:IsA("IntValue") then
                Char[v.Name] = v.Amount.Value
            end         
        end
    end 

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot3:GetChildren()) do
        if v:IsA("StringValue") then
            Char[v.Name] = v.Value
        end
        for i, v2 in pairs(v:GetChildren()) do
            if v:IsA("IntValue") then
                Char[v.Name] = v.Amount.Value
            end         
        end
    end 

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot4:GetChildren()) do
        if v:IsA("StringValue") then
            Char[v.Name] = v.Value
        end
        for i, v2 in pairs(v:GetChildren()) do
            if v:IsA("IntValue") then
                Char[v.Name] = v.Amount.Value
            end         
        end
    end 

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot5:GetChildren()) do
        if v:IsA("StringValue") then
            Char[v.Name] = v.Value
        end
        for i, v2 in pairs(v:GetChildren()) do
            if v:IsA("IntValue") then
                Char[v.Name] = v.Amount.Value
            end         
        end
    end 

    for i, v in pairs(game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name).Tools.Slot6:GetChildren()) do
        if v:IsA("StringValue") then
            Char[v.Name] = v.Value
        end
        for i, v2 in pairs(v:GetChildren()) do
            if v:IsA("IntValue") then
                Char[v.Name] = v.Amount.Value
            end         
        end
    end 

    local suc, error = pcall(function()
        DataStore:SetAsync(Player.UserId, HTTP:JSONEncode(Inv))
    end)

    local suc1,error1 = pcall(function()
        SlotDataStore:SetAsync(Player.UserId, HTTP:JSONEncode(Char))
    end)

    if not suc then
        print(error.. ' Inventory Variable')
    end

    if not suc1 then
        print(error1.. ' Char Variable')
    end
end

game.Players.PlayerRemoving:Connect(function(Player)
    saveInventory(Player)

    if game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name) then
        game.ServerStorage.Assets.Inventory:FindFirstChild(Player.Name):Destroy()
    end
end)

game:BindToClose(function()
    for _, Client in ipairs(game:GetService("Players"):GetPlayers()) do
        spawn(function()
            saveInventory(Client)
        end)
    end
end)