StatusShields errors due to shield spellID changes/removed due to 5.0.4 #304


  • Defect
  • Fixed
Closed
Assigned to michaelsp
  • kaluu created this issue Sep 2, 2012

    What steps will reproduce the problem?
    1. Assign "shields" to any indicator
    2. Bugsack reports "1x Grid2-r684\modules\StatusShields.lua:145: table index is nil"

    What is the expected output? What do you see instead?
    "Shields" should appear under current status of the indicator

    What Grid2 version and language of WoW are you using?
    r684 English WoW 5.0.4

    Do you have an error log of what happened?

    1x Grid2-r684\modules\StatusShields.lua:145: table index is nil
    Grid2-r684\modules\StatusShields.lua:145: in function "UpdateDB"
    Grid2-r684\modules\StatusShields.lua:91: in function "OnEnable"
    Grid2-r684\GridStatus.lua:85: in function "RegisterIndicator"
    Grid2-r684\GridIndicator.lua:38: in function "RegisterStatus"
    Grid2-r684\GridSetup.lua:46: in function "SetupStatusMap"
    Grid2-r684\GridSetup.lua:61: in function "Setup"
    Grid2-r684\GridCore.lua:162: in function "LoadConfig"
    Grid2-r684\GridCore.lua:151: in function <Grid2\GridCore.lua:140>
    (tail call): ?
    <in C code>
    <string>:"safecall Dispatcher[1]":9: in function <string>:"safecall Dispatcher[1]":5
    (tail call): ?
    Ace3-r1063\AceAddon-3.0\AceAddon-3.0-11.lua:543: in function "EnableAddon"
    Ace3-r1063\AceAddon-3.0\AceAddon-3.0-11.lua:636: in function <Ace3\AceAddon-3.0\AceAddon-3.0.lua:621>
    <in C code>
    FrameXML\UIParent.lua:299: in function "UIParentLoadAddOn"
    FrameXML\UIParent.lua:322: in function "CombatLog_LoadUI"
    FrameXML\UIParent.lua:692: in function <FrameXML\UIParent.lua:657>

    Locals:
    nil

    Please provide any additional information below.
    This is partially related to ticket 300 due to the shielding part. When I initially saw that StatusShield.lua was the file with the error, I first attempted to look in the configuration for the spells under status/health&heals/shields. Three checkboxes had checks next to them but no spell name (Divine aegis, Sacred Shield, and Savage Defense). I've been leveling a bear druid, and remember that Savage Defense is no longer a shield, but a heal. Looking at the spell IDs of Divine aegis and Sacred shield from line 13 and on, apparently the spellIDs for these were changed. Here is a list of the changes, with current addition you may want to add as well:

    Divine Aegis was changed from 47509 to 47515
    Spirit Shell is 114908 (the acutal absorb, not the buff to activate Spirit Shell which is 109964)
    Sacred Shield was changed from 85285 to 65148(the acutal absorb, not the buff to activate Sacred Shield which is 20925)

    Changing these spellID appear to make everything work again before 5.0.4 was released.

  • kaluu added the tags New Defect Sep 2, 2012
  • kaluu edited description Sep 2, 2012
  • michaelsp removed a tag New Sep 3, 2012
  • michaelsp added a tag Accepted Sep 3, 2012
  • michaelsp removed a tag Accepted Sep 3, 2012
  • michaelsp added a tag Fixed Sep 3, 2012
  • michaelsp closed issue Sep 3, 2012

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