Addon system bug reports

[the_real_seebs]

We could write that, but it'd be hard to convince people to use it in a library.

I say: We should have this as an API change, and yes, even as a breaking change. It is just that worth fixing.

[Naifu]

We could write that, but it'd be hard to convince people to use it in a library.

I say: We should have this as an API change, and yes, even as a breaking change. It is just that worth fixing.

I second this

Cheers
N.

[Baanano]

Utility.Auction.Cost errors out when the 4th parameter (buyout price) is nil. This worked ok before 1.8.

BTW, when you have time to fix it, you could try to add a variant that accepts stack size as parameter so we don't have to split stacks *before* knowing how much will they cost to auction .

[Kreiri]

We could write that, but it'd be hard to convince people to use it in a library.

I say: We should have this as an API change, and yes, even as a breaking change. It is just that worth fixing.

This. And another breaking change, please: frame events handlers should be stored as event tables too.

[Naifu]

The following code illustrates a problem with changing textures:

local context = UI.CreateContext("nkText")
local frame = UI.CreateFrame("Frame", 'test', context)
frame:SetPoint ('CENTER', UIParent, 'CENTER')
frame:SetWidth (200)
frame:SetHeight (200)

frame.texture = UI.CreateFrame("Texture", 'texture', frame)
frame.texture:SetPoint ("TOPLEFT", frame, "TOPLEFT")
frame.texture:SetTexture ('nkTest', 'textureGrain.png')
frame.texture:SetWidth (200)
frame.texture:SetHeight (200)

frame.tint = UI.CreateFrame("Frame", 'tint', frame.texture)
frame.tint:SetPoint ("TOPLEFT", frame, "TOPLEFT")
frame.tint:SetBackgroundColor(.5, 0, 0, 0.5)
frame.tint:SetWidth (200)
frame.tint:SetHeight (200)

frame:SetAlpha (0.2)

function frame.Event:MouseIn() frame:SetAlpha(1)  end
function frame.Event:MouseOut() frame:SetAlpha(0.2)  end

frame.texture:SetTexture ('nkTest', 'textureGrain.png')

What this does is create a frame and add two children: a texture and another frame to 'tint' the texture.

Now if you change a texture after creating it and then change the alpha level of the parent frame the texture will appear as it is e.g. the tint frame is loosing it's color. I'm using black & white textures in nkThings and this bug leads to them being displayed in all their b&w glory.

If you remove the line where the texture is set again you'll see that the coloring is no longer lost. Didn't find any way to work around this behaviour.

Cheers
N.

[Kreiri]

Naifu, did you try setting layers for your texture and/or tint frames?

frame.texture = UI.CreateFrame("Texture", 'texture', frame)
frame.texture:SetPoint ("TOPLEFT", frame, "TOPLEFT")
frame.texture:SetTexture ('nkTest', 'textureGrain.png')
frame.texture:SetWidth (200)
frame.texture:SetHeight (200)
frame.texture:SetLayer(-1)  --- add this line and see if it works

[Naifu]

Nope doesn't work Reapplying the parent on the 'tint' frame doesn't work either.

Cheers
N.

[ZorbaTHut]

The following code illustrates a problem with changing textures:

I've heard there's a glitch with texture ordering. I haven't gotten a chance to look at it yet, unfortunately. Doing something to kick the textures around and re-order them may work as a temporary fix.

[Naifu]

I've heard there's a glitch with texture ordering. I haven't gotten a chance to look at it yet, unfortunately. Doing something to kick the textures around and re-order them may work as a temporary fix.

Oki doki I'll try around some more

Cheers
N.

[Naifu]

The API is no firing Event.Unit.Available if the player is in a raid.

If the player is not in a raid and ports (or enters a dungeon) the following events will happen:

Event.Buff.Add for all buffs but temporary buffs
Event.Unit.Available
Event.Buff.Add for all buffs but temporary buffs
Event.Buff.Remove for all buffs but temporary buffs
Event.Buff.Add for all buffs including temporary buffs

I dunno the reason for this strange firing of events but that's the way it is. Now for the fun part. This happens as soon as you group with another player, turn your group into a raid and port (or enter instance):

Event.Buff.Add for all buffs but temporary buffs
Event.Buff.Remove for all buffs but temporary buffs
Event.Buff.Add for all buffs including temporary buffs

No Event.Unit.Available is being launched in raid situations. If recalculating the units buffs on Event.Unit.Available is the right solution to update buffs this is kinda hard to do if the event doesn't fire Oh and important to mention is that the buffs added in the event are getting a new ID. As the 'old' ones are never removed you get duplicate information for temporary buffs you cannot map to any valid buff information.

Cheers
N.

[ZorbaTHut]

The API is no firing Event.Unit.Available if the player is in a raid.

Aha, I see what the issue is here. The same issue occurs in a group. I'll get that fixed.

There may be a workaround by relying on Event.Unit.Detail.Zone to let you know when things need to be refreshed, or alternatively, just doing a full refresh every five seconds or so. Obviously this isn't ideal and isn't intended as a long-term solution, but it may be a viable short-term fix.

[Kreiri]

Everything spams Event.Map.Detail.Coord every time. Everything. Vendors, trainers, bankers, footholds, whatever fire one Event.Map.Detail.Coord after another with all coordinates the same.

I made a simple function to print details of "locations" passed in Event.Map.* handler and got 112 KB of noncombat log in three seconds, and all coords for given id were the same.

Also, footholds give Event.Map.Change when player is moving. The coordinates retrieved in Event.Map.Change handler differ starting from second digit after period.

[ZorbaTHut]

Everything spams Event.Map.Detail.Coord every time. Everything. Vendors, trainers, bankers, footholds, whatever fire one Event.Map.Detail.Coord after another with all coordinates the same.

I'm having trouble reproducing this. Can you give me an exact location where this occurs? 'Cause I'm not seeing this at all right now

[Kreiri]

I was in College of Planar Studies/Manufactory in Meridian.

Edit: I don't know if it matters, but I'm on EU server.

[ZorbaTHut]

I was in College of Planar Studies/Manufactory in Meridian.

Edit: I don't know if it matters, but I'm on EU server.

Hrm.

If you can get it to happen again, can you paste half a dozen lines in?