search

HizurosWoWAddOns / Broker_Everything

A collection of module to display informations about own character, friends, guild and more on minimap or panel addons like Bazooka, ChcolateBar, TitanPanel and more.
GNU General Public License v3.0
10 stars 6 forks source link

[fix] Remove LUA error introduced in 10.2.6 in shared.lua #45

Closed lowlee closed 6 months ago

lowlee commented 6 months ago

Changes call to GetItemSpell() to use item ID rather than item link

Fixes #44

PedjaSenpai commented 6 months ago

You deleted local info = itemsByID[id]; that needs to be present for this change to work as intended

zaphon commented 6 months ago

I did something similar to fix it (while retaining info which is used in the for loop if it is a spell).

                local _, info = GetItemInfo(id)
                local _, spell = GetItemSpell(info);
                if spell then
hizuro commented 6 months ago

Thanks for help.

Strange. The code doesn't seem correct to me even before the 10.2.6 update. Did I sleep while programming? 🫣