Default aura setup not working #6


  • Defect
  • Fixed
Closed
Assigned to _ForgeUser117147
  • _ForgeUser1539023 created this issue Feb 18, 2009

    What steps will reproduce the problem?
    1. Delete your SV file (if one exists)
    2. Load Grid2, it will create a new profile

    What is the expected output? What do you see instead?
    E.g. for a priest, the default setup should show the "Renew"-buff and the "Weakened soul"-debuff, but it doesn't.

    What version of the product are you using?
    r48

    Please provide any additional information below.
    The default setup uses spell ids, probably to make it independent from the users locale, but the StatusAura module expects the name of the (de-)buff to be provided. I think there are two possibilities to fix this:
    1. Make the StatusAura module work with spell ids. The advantage from this is that profiles would be protable between locales. The big downside of this solution is that users need to provide a spell id if they want to create a new aura status, something that should not be expected from an average user.
    2. Translate the spell ids to the spell name in the current locale on profile creation. This can be done easily with the GetSpellInfo function like in this example for priests:

        elseif class == "PRIEST" then
            setup.buffs.renew = { (GetSpellInfo(139)), true, 1, 1, 1, }
            setup.debuffs.weakened = { (GetSpellInfo(6788)), 1, 0, 0, }

  • _ForgeUser1539023 added the tags New Defect Feb 18, 2009
  • _ForgeUser117147 posted a comment Feb 19, 2009

    Fixed in Revision50.

    Create(De)BuffStatus() API now accepts a number as a spell name and will call GetSpellInfo() if it's the case.
  • _ForgeUser117147 removed a tag New Feb 19, 2009
  • _ForgeUser117147 added a tag Fixed Feb 19, 2009
  • _ForgeUser117147 closed issue Feb 19, 2009

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