Authors CurseForge

Hey

I recently began using your ag_Unitframes, and so far i've been loving them. However there are at the moment 2 major things i'm concerned about (indeed, so concerned it made my create an account on this forum, so if i missed any forum customs or stuff like that, i apologize):

- ag_Unitframes does not update instantly when shifting out of forms (cant recall atm if it does so when shifting in but im certain it does not when shifting out). For instance; if you are in catform and you have 30% energy, shifting out will cuase the energy-bar (now in fact mana-bar) to still represent 30% while the actual mana it should be displaying might be something completely different.

- There is no Druidbar support. This was one of the major points that made me choose Perl some time ago, and it is still the reason im considering going back. I can see that, to a non-druid, this seems trivial but when the standard Blizzard UI does not offer anything like it and it is vital to success in both PvP and PvE, i sincerely hope you implement such a function before the release of Burning Crusade when we're gonna need it for leveling. Even something as simple (i presume, i only poked around a little in the source, but it looks very well structured i must say) as making that which currently works as the XP-bar show your Mana while shapeshifted would be more than enough :)

Other than that, may i congratulate you on a most successful Addon!

Noticing same things, would be nice if we get some druid love. ;)

Yes, please give us druids some thing that blizz does'nt (love)

I second these requests as a fellow druid! But Blizzard has been given us plenty of love with mangle and lacerate ;)

I've come up with a fix for the first issue (bar not updating when you shift out of forms). The problem was that when your power type (mana/rage/energy) changed, it was only calling a function to update the bar's color, not its values. The issue wasn't apparent when shifting into a form, because once the amount of your mana/rage/energy changes, the bar gets a correct update. But if you shift out of a form, either you'll get a delay until your mana regen kicks in, or if you have full mana, you won't get an update until you use mana.

Anyway, for the fix, change line 176 of ag_UnitClass.lua from this:

self:ObjectRegisterEvent("UNIT_DISPLAYPOWER","StatusBarsColor")

to this:

self:ObjectRegisterEvent("UNIT_DISPLAYPOWER", "UpdatePower")

I didn't notice any adverse effects during my brief testing on my level 42 druid, but I suppose it's possible there are negative effects somewhere... seems safe enough though! :p

Quote from Phanx »

I've come up with a fix for the first issue (bar not updating when you shift out of forms). The problem was that when your power type (mana/rage/energy) changed, it was only calling a function to update the bar's color, not its values. The issue wasn't apparent when shifting into a form, because once the amount of your mana/rage/energy changes, the bar gets a correct update. But if you shift out of a form, either you'll get a delay until your mana regen kicks in, or if you have full mana, you won't get an update until you use mana.

Anyway, for the fix, change line 176 of ag_UnitClass.lua from this:

self:ObjectRegisterEvent("UNIT_DISPLAYPOWER","StatusBarsColor")

to this:

self:ObjectRegisterEvent("UNIT_DISPLAYPOWER", "UpdatePower")

I didn't notice any adverse effects during my brief testing on my level 42 druid, but I suppose it's possible there are negative effects somewhere... seems safe enough though! :p

Thanks alot, I was really getting annoyed with it, I'll test it asap and report back! ;)

With my change, the bar will update to the correct value.

However -- and this didn't occur to me nor did I notice it in the aforementioned "brief testing"! -- in order to get the text to update to show the correct value, you'd need to add "UNIT_DISPLAYPOWER" to each of the Watchdog tags you're using.

So, to add the event to the tags you're using, open ag_WatchdogTags.lua and change any occurrence of:

event = {"UNIT_MANA","UNIT_RAGE","UNIT_FOCUS","UNIT_ENERGY"}

to:

event = {"UNIT_MANA","UNIT_RAGE","UNIT_FOCUS","UNIT_ENERGY","UNIT_DISPLAYPOWER"}

Thanks a lot for the fixes. :D

EDIT: How would someone go about adding these to the release version? It'd be great if AG worked appropriately with our forms.

You're welcome.

Also, for a druid bar, I use SimpleDruidBar: http://www.wowinterface.com/downloads/fileinfo.php?id=6293
Doesn't 100% match my unit frames, but does its job. :P