Auras by SpellID #271


  • Fixed
  • Enhancment
Closed
Assigned to michaelsp
  • _ForgeUser7773765 created this issue Jan 12, 2012

    I've recently been attempting to finalize a full transition from Grid to Grid2 but I noticed another thing that was missing from Grid2 that I used to rely on in specific circumstances with Grid. Namely, auras based explicitly on SpellID instead of just the aura name. The example that reminded me of this was in Zul'gurub. There are two Frostburn Forumla debuffs with different SpellIDs, one is the one players get from the cauldon that lets them freeze targets and the other is when a player is frozen by a mob. Obviously, I don't care about the former but I want to know when a unit is frozen so I can dispel them quickly. I used to be able to do this with Grid because there was an addon that allowed adding statuses by SpellID.

    Another example I can think of was on heroic Ascendant Council where the source and splash versions of Gravity Core and Static Overload had the same name but different SpellIDs and being able to filter it to only show the source version of the debuff by SpellID was very useful for instructing people how to move during the fight.

  • _ForgeUser7773765 added the tags New Enhancment Jan 12, 2012
  • michaelsp posted a comment Jan 13, 2012

    Using spellids will generate many problems:
    - People could not add custom buffs/debuffs by name (its not posible to translate aura names to spellids , there is no api for this).
    - Many debuffs have a lot of spellIds (normal/heroic/lfr versions or spellId changes depending of the combat phase), this means that many spell Ids must be added only to track the same debuff.

    An hybrid system could be used, supporting spellids and names. But this will increase cpu usage. I dont want to slow down Grid2 only to fix 1 or 2 combats.

    Maybe i could add support to optionally track spellIds in raid debuffs module, but not in grid2 standard auras management.

    Update: Finally i changed my mind, i think (and i wish) the way im going to implement spellids support must not increment the cpu usage too much, this change will fix another issues too.


    Edited Jan 17, 2012
  • michaelsp removed a tag New Jan 17, 2012
  • michaelsp added a tag Accepted Jan 17, 2012
  • michaelsp removed a tag Accepted Jan 19, 2012
  • michaelsp added a tag Started Jan 19, 2012
  • michaelsp removed a tag Started Jan 20, 2012
  • michaelsp added a tag Fixed Jan 20, 2012
  • michaelsp closed issue Jan 20, 2012

To post a comment, please login or register a new account.