Scripting [Ajuda] função onDeath

[Guizek69 9]

Bom galera aí vai minha duvida:

Utilizo atualmente um TFS 0.4 e implantei o AUTO LOOT recentemente.
Fiz diversas modificações e agora esse sistema me atende.

Porem, há um problema - Ele assim como outros do mercado, trabalha com essa função onDeath:
 

Citar

function onDeath(cid, corpse, deathList)
local killer,pos = deathList[1],getCreaturePosition(cid)
addEvent(loot,1,killer,pos)
return true
end

Isso faz com que o jogador a dar o ultimo hit na criatura fique com o loot da lista, mesmo que nao seja o "dono do corpo morto" ou corpseOwner.

Obviamente nem preciso dizer que isso esta errado e vai gerar muitos problemas no meu servidor.

A questao é: existe alguma função no TFS 0.4 que verifica o dono do corpo morto para eu implantar uma regra nessa função? Ou entao que diga o jogador que deu mais dano e apenas ele possa "lootear"?!

Aguardo amigos.

@WooX

[Lurk 67]

@Guizek69 do mesmo jeito q existe a função do cara não poder abrir o corpo de um bixo q ele n matou vc deve poder implementar isso no script tb, eu lembro q tinha visto aqui no tibiaking um post de um cara ensinando como ver todas as funcoes do seu tfs, mas enfim, provavelmente nas suas sources deve ter algo relacionado a owner, vc vai ter q transformar isso em função lua (caso já não exista) e dai adicionar no seu script, tipo um if not isOwner(cid) then return false end

[Guizek69 9]

Na source, a função em actions.cpp é essa:
 

Citar

Line 523:         if(container->getCorpseOwner() && !player->canOpenCorpse(container->getCorpseOwner())
    Line 523:         if(container->getCorpseOwner() && !player->canOpenCorpse(container->getCorpseOwner())
    Line 524:             && g_config.getBool(ConfigManager::CHECK_CORPSE_OWNER))
    Line 525:             return RET_YOUARENOTTHEOWNER;
    Line 599:         creatureId) || action->hasOwnErrorHandler());

Encontrei uma função lua para TFS 1.2 feita pelo @luanluciano93:
 

Citar

local corpse = Tile(target:getPosition()):getTopDownItem()
    if not corpse or not corpse:isContainer() then
        return false
    end

    if corpse:getType():isCorpse() and corpse:getAttribute(ITEM_ATTRIBUTE_CORPSEOWNER) == player:getId() then
		return true
    end

Vou tentar converter para TFS 0.4.
Se alguem puder ajudar, eu agradeço.

[Lurk 67]

@Guizek69 n sei nada de tfs 1.x mas acho que eh convertível sim, vc provavelmente já tentou mas ainda assim vale apontar, tenta colocar esse getCorpseOwner() no seu script e ve se resolve

[Tottin 25]

Se você quer trabalhar somente com o player que pegou o loot você pode utilizar a função da seguinte forma:
function onDeath(cid, corpse, lastHitKiller, mostDamageKiller)
local killer = lastHitKiller[1] --- a atribuição em uma variável do player que pegou o frag é totalmente opcional.

Dai caso queira realizar alguma ação em cima desse player pode fazer da seguinte forma:
if isPlayer(killer) then --- Não lembro ao certo se tem monstros entram na lista (lastHitKiller) então só por precaução eu recomendaria fazer uma checagem se quem fragou é 1 player.
doPlayerAddItem(killer, 13981, 1)
end

[Guizek69 9]

1 hora atrás, Tottin disse:

Se você quer trabalhar somente com o player que pegou o loot você pode utilizar a função da seguinte forma:
function onDeath(cid, corpse, lastHitKiller, mostDamageKiller)
local killer = lastHitKiller[1] --- a atribuição em uma variável do player que pegou o frag é totalmente opcional.

Dai caso queira realizar alguma ação em cima desse player pode fazer da seguinte forma:
if isPlayer(killer) then --- Não lembro ao certo se tem monstros entram na lista (lastHitKiller) então só por precaução eu recomendaria fazer uma checagem se quem fragou é 1 player.
doPlayerAddItem(killer, 13981, 1)
end

O problema é exatamente esse. O auto loot nao avalia todos os jogadores que participaram no combate para x criatura.
Ele determina que o loot é de quem deu o ultimo hit e acabou.

Se EU der 99% do dano em um dl e alguem der hit kill e dropar dsm: EU fico como dono do corpo, mas o a dsm é retirada automaticamente pelo sistema e concedida ao jogador do "ks".

Por isso preciso de uma função que determine se sou ou nao o dono do corpo ou se sou ou nao quem deu mais dano para dai proceder com todo o resto.

[Tottin 25]

Meu querido, eu acabei de te mandar exatamente isso, ele vai enviar o loot pra quem fragou o monstro. Se o jogo te reconheceu como "dono do loot" então o loot vai ser enviado pra você, Vou ser gente boa e vou montar a função pra tu. Isso possivelmente vai resolver o seu problema, se não funcionar, então seu servidor reconhece o "dono do loot" de outra forma. Seria interessante descobrir qual.

function onDeath(cid, corpse, lastHitKiller, mostDamageKiller)

local killer = lastHitKiller[1]
pos = getCreaturePosition(cid)
addEvent(loot,1,killer,pos)
return true
end

[Guizek69 9]

4 minutos atrás, Tottin disse:

Meu querido, eu acabei de te mandar exatamente isso, ele vai enviar o loot pra quem fragou o monstro. Se o jogo te reconheceu como "dono do loot" então o loot vai ser enviado pra você, Vou ser gente boa e vou montar a função pra tu. Isso possivelmente vai resolver o seu problema, se não funcionar, então seu servidor reconhece o "dono do loot" de outra forma. Seria interessante descobrir qual.

function onDeath(cid, corpse, lastHitKiller, mostDamageKiller)

local killer = lastHitKiller[1]
pos = getCreaturePosition(cid)
addEvent(loot,1,killer,pos)
return true
end

O corpseOwner para criaturas é sempre quem deu mais dano em tal, nao quem deu o hit final.
Acho que esta confundindo com o frag player system.
 

[Tottin 25]

É, bem provável que eu esteja confundindo, são sistemas "bem similares" né, mas td bem.

[WooX 691]

7 horas atrás, Guizek69 disse:

existe alguma função no TFS 0.4 que verifica o dono do corpo morto para eu implantar uma regra nessa função?

A resposta é não, pelo menos não uma função Lua.

 

4 horas atrás, Guizek69 disse:

Encontrei uma função lua para TFS 1.2 feita pelo @luanluciano93:
 

Citar

local corpse = Tile(target:getPosition()):getTopDownItem()
    if not corpse or not corpse:isContainer() then
        return false
    end

    if corpse:getType():isCorpse() and corpse:getAttribute(ITEM_ATTRIBUTE_CORPSEOWNER) == player:getId() then
		return true
    end

Vou tentar converter para TFS 0.4.
Se alguem puder ajudar, eu agradeço.

Tenta buscar pelo atributo "corpseowner" com a função getItemAttribute, pode ser que funcione.

 

2 horas atrás, Tottin disse:

function onDeath(cid, corpse, lastHitKiller, mostDamageKiller)

Eu não me recordo da existência dos parâmetros lastHitKiller e mostDamageKiller, pelo menos não no TFS 0.4. Mas se o callback realmente retornar estes valores, a solução pro problema ta ai, só utilizar o mostDamageKiller que vai retornar quem deu mais dano.

 

Edit: me lembrei que um conhecido estava passando por este mesmo problema a algum tempo atrás, eu cheguei a postar a solução que ele encontrou em outro tópico, pode dar uma olhada.

 

 

Editado Julho 19, 2020 por WooX (veja o histórico de edições)

[Guizek69 9]

Em 19/07/2020 em 19:45, WooX disse:

A resposta é não, pelo menos não uma função Lua.

 

Tenta buscar pelo atributo "corpseowner" com a função getItemAttribute, pode ser que funcione.

 

Eu não me recordo da existência dos parâmetros lastHitKiller e mostDamageKiller, pelo menos não no TFS 0.4. Mas se o callback realmente retornar estes valores, a solução pro problema ta ai, só utilizar o mostDamageKiller que vai retornar quem deu mais dano.

 

Edit: me lembrei que um conhecido estava passando por este mesmo problema a algum tempo atrás, eu cheguei a postar a solução que ele encontrou em outro tópico, pode dar uma olhada.

 

 

A solução foi MUITO mais simples do que eu pensei, agradeço a ajuda de todos vocês, vocês são fod@!!

Segue enredo:

A função onDeath tem duas matrizes: lastHitKiller e mostDamageKiller.

Quando voce usa deathList[1] aplica-se a matriz lastHitKiller, o jogador que causou o hit kill.
Quando voce usa deathList[2] aplica-se a matriz mostDamageKiller ou seja, o maior causador de dano.

Na onPrepareDeath, mesma regra etc..

Citar

function onPrepareDeath(cid, deathList)
	lastHitKiller = deathList[1]
	mostDamageKiller = deathList[2]

Logo minha função ficou assim:
 

Citar

function onDeath(cid, corpse, deathList)

local killer1 = deathList[1]
local killer2 = deathList[2]
local pos = getCreaturePosition(cid)

if killer2 == nil then
addEvent(corpseRetireItems,1,killer1,pos)
else
addEvent(corpseRetireItems,1,killer2,pos)
end
return true
end

Vale a pena lembrar que voce deve criar verificaçoes basicas como if isPlayer(cid) and isMonster(target).
No meu caso, nao foi necessario porque a verificação é feita num evento "combat" e ele joga os players determinado para o onDeath.

RESOLVIDO.
 

Editado Julho 22, 2020 por Guizek69
info (veja o histórico de edições)

[Vodkart 2155]

Era mais facil fazer uma tabela que não executasse nesse tipo de boss...

 

BlockMonsters = {"Demon","Hydra"} -- nome dos mobs
	if isMonster(cid) and not isInArray(BlockMonsters, getCreatureName(cid)) then
		-- sua função do autoLoot
	end

 

[Tottin 25]

Mas se o evento onDeath só ativa em monstros cujo tiveram a tag da script registrada (me corrija se eu estiver errado), não seria 1 tanto quanto desnecessário adicionar uma checagem pra ver se o monstro morto deve ou não ativar a script de autoloot, sendo que bastaria não registrar a tag nos monstros que você não quer que ative a script. Claro que a situação mudaria se a função fosse de onKill.

[Guizek69 9]

22 minutos atrás, Vodkart disse:

Era mais facil fazer uma tabela que não executasse nesse tipo de boss...

 

BlockMonsters = {"Demon","Hydra"} -- nome dos mobs
	if isMonster(cid) and not isInArray(BlockMonsters, getCreatureName(cid)) then
		-- sua função do autoLoot
	end

 

Olá grande mestre!
Perdoe-me minha ignorancia, mas nao entendi bem sua colocação.

Meu problema não está relacionado a criatura x ou y e sim ao fato do lastHitKiller ficar com o loot da tabela mesmo se desse apenas 1 de dano em determinada criatura.
Sendo assim, ele não seria o dono do corpo morto mas ficaria com o loot "rare" de outro consequentemente.
Isso é resolvido buscando a matriz deathList[2] no onDeath - que certifica de dar o loot ao jogador certo ( o que deu mais dano ).

Ficou 100% perfeito agora.

7 minutos atrás, Tottin disse:

Mas se o evento onDeath só ativa em monstros cujo tiveram a tag da script registrada (me corrija se eu estiver errado), não seria 1 tanto quanto desnecessário adicionar uma checagem pra ver se o monstro morto deve ou não ativar a script de autoloot, sendo que bastaria não registrar a tag nos monstros que você não quer que ative a script. Claro que a situação mudaria se a função fosse de onKill.

É o inverso amigo.

O evento ativa em todas as criaturas, exceto as que estao em block list (bosses).

[Vodkart 2155]

@Guizek69 Ah, entendi o que quis dizer!! Desculpa amigo, abraços!