[Fixed] Ilvl on equipped slots

[Lanseb]

Hey all,

Ilvl for items has been bugged for a few weeks. Greens all show as 680 for some reason, and some of the pvp gear from boxes would show as 800 even though it wasn't. Here's an example:



Also, the Artifact's base ilvl (750) is shown: it doesn't show the real ilvl, which in this case is 770ish.

Just a few things I noticed! Thanks for all the awesome work!

[max41479]

lanseb, fixed. https://github.com/Shestak/ShestakUI/pull/291
TODO: Add an option in menu for disabling this code.

[Wetxius]

It's not good solution. All addon than tried find item level via scanning tooltip is use cpu expensive.

Добавлено через 1 час 3 минуты
http://www.wowhead.com/item=132910/leywalker-bracers
It's show 680 item level, and GetItemInfo return is 680.
So without tooltip scan it's unreal to show right level. But we use item check in 3 places to show Our item level, Inspect target item level, show bags item level.

[max41479]

It's not perfect, but it's the only way.

Bugs :(

[hide]

bugs fixed.

[Wetxius]

Just review this code for 110 level.
https://github.com/Shestak/ShestakUI...el.lua#L23-L47

[Tryllemann]

Gotten tired of this code part, so i added LibItemUpgradeInfo-1.0 instead into my addon. Posibility?

[Wetxius]

LibItemUpgradeInfo ~= use tooltip scanning.

Добавлено через 1 час 9 минут
As I tested LibItemUpgradeInfo and other tooltip scan addon, it's show wrong level for heirloom items, because in tooltip level is scaled to player level. So if I 12 level, it shows 17 item level.
We can true heirloom level only with this code https://github.com/Shestak/ShestakUI...el.lua#L23-L47
Just need update for 110 level.

For all other items like PvP and artifact we can use tooltip scanning and it show correct level.

[F5Hellbound]

I have reached level 110 and some item level will not be correctly timed

see pic

i have downloaded the last update from GitHub

[max41479]

More bugs

Fixed.

[Wetxius]

So, subject bug is fixed.