Lucas, vc ta verificando o elemento vocnova sem especificar em qual elemento da tabela voc ela esta. Vc pode fazer como o whitewolf falou ou rodar um loop na tabela

Hummm e como seria para fazer tal loop?

Assim

local voc = {
{vocatual = 1, vocnova = 5, preco = 2, item = 2160, nome = "0 k"},
{vocatual = 2, vocnova = 6, preco = 2, item = 2160, nome = "0 k"},
{vocatual = 3, vocnova = 7, preco = 2, item = 2160, nome = "0 k"},
{vocatual = 4, vocnova = 8, preco = 2, item = 2160, nome = "0 k"}
}

function onSay(cid, words, param, channel)

    for index, vocation in pairs(voc) do
        
        if getPlayerVocation(cid) == vocation.vocatual then
            if doPlayerRemoveItem(cid, vocation.item, vocation.preco) then
                doPlayerSetVocation(cid, vocation.vocnova)
                doPlayerSendTextMessage(cid, 22, "Voce foi promovido para "..getPlayerVocationName(cid))
                doSendMagicEffect(getThingPos(cid), 13)
            else
                doPlayerSendCancel(cid, "Voce nao possui "..vocation.preco.." "..getItemNameById(vocation.item))
            end
        end
    end
    return true
end

Resolvido? Se sim clique em melhor resposta!