Solved Vocation based addon

[Caduceus]

I am attempting to make a quest chest to give a player addons based on vocation. The chest opens, gives the storage ID. No addon rewarded. No errors on console. Using tfs 1.2, how bad am I off?

local storage = 15485

local outfits = {
    {138,3}, -- 1
    {148,3}, -- 2
    {137,3}, -- 3
    {139,3}  -- 4
}

local msg = ''
function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if player:getStorageValue(storage) < 1 then
        for vocId, items in ipairs(outfits) do
            if player:getVocation():getBase():getId() == vocId then
                for i = 1, #items do
                    --add players addon based on Vocation/sex (player:addItem(items[i]))
                    doPlayerAddOutfit(items[i])
                end
                msg = "Yay, free shit!"
            end
        end
        player:setStorageValue(storage, 1)
    else
        msg = "chest is empty."
    end
    player:sendTextMessage(MESSAGE_INFO_DESCR, msg)
    return true
end

 

[Non Sequitur]

I assumed the 3 in the outfits table is the addon number:

local outfits = {
    {138,3}, -- 1
    {148,3}, -- 2
    {137,3}, -- 3
    {139,3}  -- 4
}

local msg = ''
function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if player:getStorageValue(storage) < 1 then
        local outfit = outfits[player:getVocation():getBase():getId()]
        player:addOutfitAddon(outfit[1], outfit[2])
        msg = "Yay, free shit!"
        player:setStorageValue(storage, 1)
    else
        msg = "chest is empty."
    end
    player:sendTextMessage(MESSAGE_INFO_DESCR, msg)
    return true
end

 

[Peonso]

You should add both male and female counterparts.

 

[Caduceus]

Something like this I assume. Thanks

local storage = 15485

local outfits = {
    {female = 138, male = 130, addon = 3}, -- 1
    {female = 148, male = 144, addon = 3}, -- 2
    {female = 137, male = 129, addon = 3}, -- 3
    {female = 139, male = 131, addon = 3}  -- 4
}

local msg = ''
function onUse(player, item, fromPosition, target, toPosition, isHotkey)
    if player:getStorageValue(storage) < 1 then
        local outfit = outfits[player:getVocation():getBase():getId()]
        player:addOutfitAddon(outfit.female, outfit.addon)
        player:addOutfitAddon(outfit.male, outfit.addon)
        msg = "Yay, free shit!"
        player:setStorageValue(storage, 1)
    else
        msg = "chest is empty."
    end
    player:sendTextMessage(MESSAGE_INFO_DESCR, msg)
    return true
end