Icon-center texture forgotten on /reload #83


  • Defect
  • Fixed
Closed
Assigned to _ForgeUser117147
  • _ForgeUser249224 created this issue Jan 21, 2010

    1. Set a buff status to "Show if missing". In my case it was InnerFire.
    2. Set the indicator for this buff to icon-center.
    3. Cast buff for it to retrieve the texture.
    4. Remove buff to have the icon texture display now that the buff is missing.
    5. Reload the UI.

    Since the buff is still missing I expect it to display the texture for this buff, however it does not display the icon. Recasting and cancelling the buff will make the icon display again until I /reload again. This is using r319.

  • _ForgeUser249224 added the tags New Defect Jan 21, 2010
  • _ForgeUser117147 posted a comment Jan 22, 2010

    Fixed in rev321

  • _ForgeUser117147 removed a tag New Jan 22, 2010
  • _ForgeUser117147 added a tag Fixed Jan 22, 2010
  • _ForgeUser117147 closed issue Jan 22, 2010
  • _ForgeUser249224 posted a comment Jan 22, 2010

    Issue still occurs in r324, even after hitting the reset button.

  • _ForgeUser117147 removed a tag Fixed Jan 22, 2010
  • _ForgeUser117147 added a tag Accepted Jan 22, 2010
  • _ForgeUser117147 reopened issue Jan 22, 2010
  • _ForgeUser65911 posted a comment Mar 17, 2010

    I checked in a fix for this. I think the code needs cleanup though. The fix adds a "missingTexture" and this single texture is returned. This avoids the init issue with the current code where it executes before GridRoster updates with units.

    I think it needs to extend to simply have a single "auraTexture" which is used in either case. Currently an array of up to 89 unit indexed entries are used to return this texture. Seems wrong unless the texture can change (it does not*) and the change is used in the state changed code.

    This is probably something that supports chained boolean logic for statuses in classic Grid.

    Something like raid debuffs could use this scheme so they show the active texture, however it would be better to have a simpler base implementation and then something complicated like this for such statuses to derive from.


    Edited Mar 18, 2010
  • _ForgeUser65911 removed a tag Accepted Mar 17, 2010
  • _ForgeUser65911 added a tag Waiting Mar 17, 2010
  • _ForgeUser65911 posted a comment Mar 19, 2010

    Mmm, ok the poison magic curse & disease statuses use this. Probably still better to simplify the single aura ones.

  • _ForgeUser65911 removed a tag Waiting Mar 19, 2010
  • _ForgeUser65911 added a tag Replied Mar 19, 2010
  • _ForgeUser65911 removed a tag Replied Mar 23, 2010
  • _ForgeUser65911 added a tag Waiting Mar 23, 2010
  • _ForgeUser65911 removed a tag Waiting Mar 24, 2010
  • _ForgeUser65911 added a tag Fixed Mar 24, 2010
  • _ForgeUser65911 closed issue Mar 24, 2010

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