BAndysc/WoWDatabaseEditor

Action Invoker source/target doesn’t work with `wait` meta action

Aberta

#33 aberto em 3 de fev. de 2021

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)C# (168 forks)auto 404
bughelp wanted

Métricas do repositório

Stars
 (561 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Describe the bug ActionInvoker is null after wait action.

Version (commit hash) d2ad04b3f53abf5738ab2e178f3a2796d52e66fe

To Reproduce Steps to reproduce the behaviour:

  1. Create a script
  2. Create any event which has „invoker”
  3. Add action wait
  4. Add any action with invoker source
  5. Test in game - doesn’t work, because invoker is null there

Expected behaviour Action Invoker should work as expected after wait action.

Proposed solution Best option would be to change TC to pass invoker here https://github.com/TrinityCore/TrinityCore/blob/master/src/server/game/AI/SmartScripts/SmartScript.cpp#L1542

Another option would be to change invoker to storedTarget in the editor, but there is a risk, this storedTargetID will be taken already. We can inform user during exporting that some storedTarget will be reserved?

Guia do colaborador

Direção de pesquisa
Explore como o editor gera smart scripts a partir de tabelas de ações, focando em como 'invoker' é atribuído como origem/destino. Depois examine a saída gerada para um script contendo uma ação de espera para ver por que invoker se torna nulo. Considere os links para o SmartScript.cpp do TrinityCore para entender o comportamento no lado do servidor.
Pilha de tecnologia
csharp
Domain
game devtooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC#Smart Scripts (SAI)
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65