self.Power.frequentUpdates = true for pets

[zork]

When I try adding self.Power.frequentUpdates = true for unit == "pet" I get a Lua error.

Screenshot:
http://img2.abload.de/img/wowscrnsho...09_213av4p.jpg

Am I doing something wrong? Pets are allowed to use frequentupdates or not?

[p3lim]

Its a bug haste forgot during the new event system.

[haste]

Originally Posted by zork When I try adding self.Power.frequentUpdates = true for unit == "pet" I get a Lua error. Screenshot: http://img2.abload.de/img/wowscrnsho...09_213av4p.jpg Am I doing something wrong? Pets are allowed to use frequentupdates or not?

Is the power frame correctly parented? It seems to me that you have a power frame that isn't. Since your UNIT_MAXMANA is nil on the parent that is.

If it is: Could you do a dump on the parent of the power frame? or tell me what layout this is on so I can test myself.

Also it's really better to post possible bug issues on my portal. I always go through them before I start looking it posts. In this case there isn't enough information to do a fix, but if it had then I would have to do an extra minor version "just" for this.

Originally Posted by p3lim Its a bug haste forgot during the new event system.

Most likely not, but based on your comment you've already had this issue without reporting it.

[p3lim]

Originally Posted by haste Most likely not, but based on your comment you've already had this issue without reporting it.

I havent, but looking at the code it seems it still uses the old event system:

local OnPowerUpdate
do
	local UnitMana = UnitMana
	OnPowerUpdate = function(self)
		if(self.disconnected) then return end
		local power = UnitMana(self.unit)

		if(power ~= self.min) then
			self.min = power

			self:GetParent():UNIT_MAXMANA("OnPowerUpdate", self.unit)
		end
	end
end

[haste]

That code is still valid for the new system.

[zork]

I will post the code on the Bug Report Portal.

Edit haste's tip was correct. I was using self.Power = CreateFrame("StatusBar", nil, self.Health), after I changed that to self.Power = CreateFrame("StatusBar", nil, self) it worked.

[Scound]

I have experienced the exact same issue, but for all my powerframes - I already posted it in "oUF - Layout discussion": http://www.wowinterface.com/forums/s...=18363&page=50

I suppose referencing the powerbar to the frame itself insted of the healthbar might do the trick for me too - will check later today. Thanks.

However, why isn't it possible to parent the healthbar-frame to the powerbar one?

[haste]

Originally Posted by Scound However, why isn't it possible to parent the healthbar-frame to the powerbar one?

frequentUpdates makes oUF call the function which updates power/health on a poll basis instead of event. It does however still use events for some updates, so oUF needs to be able to call the event function.

It's possible to solve this in a different manner now, but I haven't seen a reason to change it. Except for the two cases you have here.

[Scound]

Well, tried it and everything is working perfectly now - Thanks...