Hekili Priority Helper

Hekili Priority Helper

53M Downloads

Enhancer Shaman

legacy3 opened this issue · 2 comments

commented

Before You Begin

  • I confirm that I have downloaded the latest version of the addon.
  • I am not playing on a private server.
  • I checked for an existing, open ticket for this issue and was not able to find one.
  • I edited the title of this issue (above) so that it describes the issue I am reporting.
  • I am reporting an issue with the default priority included with the specialization (imported or edited priorities are not supported).

Spec

Shaman - Enhancement

Describe the Issue

Hi everyone,

I've identified a potential issue with the Enhancement Shaman implementation. Specifically, it concerns the following state expression:

spec:RegisterStateExpr( "tempest_mael_count", function ()
    return GetSpellCastCount( class.abilities.tempest.id )
end )

After testing, I've found that C_Spell.GetSpellCastCount() consistently returns 0, regardless of the spell ID used. I've verified this in-game using both Tempest and Lightning Bolt IDs:

/run print(C_Spell.GetSpellCastCount(188196))  -- Lightning Bolt
/run print(C_Spell.GetSpellCastCount(454015))  -- Tempest

According to the documentation (https://warcraft.wiki.gg/wiki/API_C_Spell.GetSpellCastCount), this API is intended to return the number of times a spell can be cast based on reagent availability. I suspect this API might not be functioning as the APL author intended.

Please let me know if I'm missing something obvious here. Thanks for your time!

How to Reproduce

  1. Spend Maelstrom towards your Tempest
  2. Run my LUA test commands
  3. See it always being 0

Snapshot (Link)

https://pastebin.com/raw/jn9Hk97n

Raidbots Sim Report (Link)

No response

Additional Information

No response

Contact Information

No response

commented

I think it was just a prototype/test. There is no way to directly access this number via API. The existing weakaura which tracks it is very large.

commented

This currently exists as a placeholder value; I’ll look into updating it but will close this out for now.