Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Improvement] Show the real spells used by NPC #126

Closed
BarbzYHOOL opened this issue Aug 17, 2017 · 3 comments
Closed

[Improvement] Show the real spells used by NPC #126

BarbzYHOOL opened this issue Aug 17, 2017 · 3 comments
Labels
Feature request i want something that is missing

Comments

@BarbzYHOOL
Copy link

BarbzYHOOL commented Aug 17, 2017

  1. Replace the tab "Used by" in the spell category by "Available when mind controlled for".
    Replace the tab "Abilities" in the NPC category by "Spells when mind controlled".

Because the spells listed there are only for. They are extracted from the "creature_template" table.
Maybe they are a relic of the past, or simply incomplete but they don't show the real spells from the NPC.

You can just try possessing Azuregos, he will have the same spells found on aowow, but if you fight him he will use different spells. His spells are defined into the core directly (c++) but for most NPCs it's in smart_scripts

  1. List which spells are used by the NPC by querying the "smart_scripts" table (when action_type = 11 = cast spell).

Note : not all spells are listed there, because some are directly coded into the core (c++).

@Sarjuuk Sarjuuk added the Feature request i want something that is missing label Feb 23, 2020
@Killyana
Copy link

Killyana commented Feb 24, 2020

We already have this implemented, for example: https://tcubuntu.northeurope.cloudapp.azure.com/aowow/?npc=15981

Even if the npc don't have any spell on:

SELECT * FROM   creature_template_spell  WHERE CreatureID IN (15981);

Spells from SAI are in: Abilities

@Sarjuuk
Copy link
Owner

Sarjuuk commented Feb 26, 2020

added in 54b20c3
most of it was already in place.

@Sarjuuk Sarjuuk closed this as completed Feb 26, 2020
@BarbzYHOOL
Copy link
Author

nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request i want something that is missing
Projects
None yet
Development

No branches or pull requests

3 participants