stuck with little lua

[Mazzop]

So i wanna do new feature for my addon, i wanna display Thunderforged (+) next to dropped item, for example if boss drop items [A B B+ A C], i have list in chat
1. Ax2
2. B
3. B+
4. C
Thing is, sometimes i fail to detect Thunderforge, and have no idea why.

if LootTooltipTextLeft2:GetText() ~= nil then
                if not (LootTooltipTextLeft2:GetText() == ITEM_HEROIC or LootTooltipTextLeft2:GetText() == ITEM_BIND_ON_PICKUP or LootTooltipTextLeft2:GetText():match(USE_COLON) or LootTooltipTextLeft2:GetText():match(ITEM_LEVEL)) then
                    upgrade[i] = LootTooltipTextLeft2:GetText()
                end
            end

Sometimes it work, sometimes dont.
If i do print after first "if", and there are tooltip line 2 for given item, it wont fire either
I was thinking i should track GET_ITEM_INFO_RECEIVED, but that cannot be case because when i click for same windows second time, its not working wither



My whole current lua file in attachment
(it is absolutly terrible piece of code, i would write it now with table of arrays and table.remove todetect duplicate drops, that would save me all those loops, problems with iteration i guess)

If someone would peek, i would appreciate

[Gethe]

Have you thought about using ilvl to determine Thunderforged status?

local _, link, ilvl
for index = 1, GetNumLootItems() do
    link = GetLootSlotLink(index)
    _, _, _, ilvl = GetItemInfo(link)
    if (ilvl == 528) or (ilvl == 541) then
        --this item is Thunderforged
    else
        --this item is not Thunderforged
    end
end

[Mazzop]

I will have too, just wondering why GetText() dont always work, because it work sometimes, if it was totally bad then i would not have partial results

[semlar]

If it's working sometimes then you're trying to access the font string before the tooltip has been populated.

The text is not available immediately after setting the item unless you already have it cached.

You need to request item information from the server and wait for a response before processing the data.

[Gethe]

Well mining the tooltip, to me, looks a bit messy overall, so I try to avoid it. That aside i've had trouble using more than one "or" conditional in a single statement like you have, so that's also something to look at.

[semlar]

Regardless of how you get it, item information is not immediately available, even to api functions like GetItemInfo, until it's been cached.

Be careful how you use them.

[Clamsoda]

Just to avoid confusion, the 428 in Gethe's post should be 528.

[Gethe]

Originally Posted by Clamsoda Just to avoid confusion, the 428 in Gethe's post should be 528.

Yeah, not sure how that slipped through... /fixed