HizurosWoWAddOns / FarmHud

Turn your minimap into a hud for farming ore / herb.
GNU General Public License v3.0
5 stars 6 forks source link

[BUG]: 11.0.0.55818 Prepatch not working #17

Open lostmimic opened 3 months ago

lostmimic commented 3 months ago

WoW client

WoW Retail

What happened?

Does not load with prepatch due to below error (possibly more but wont get past that point)

Lua Error Message?

2x FarmHud/FarmHud.lua:138: table index is nil
[string "@FarmHud/FarmHud.lua"]:138: in function `GetTrackingTypes'
[string "@FarmHud/FarmHud_Options.lua"]:587: in function `RegisterOptions'
[string "@FarmHud/FarmHud.lua"]:998: in function <FarmHud/FarmHud.lua:996>

Locals:
num = 25
(for index) = 1
(for limit) = 25
(for step) = 1
i = 1
name = <table> {
 type = "spell"
 name = "Find Fish"
 active = true
 spellID = 43308
 subType = -1
 texture = 133888
}
textureId = nil
active = nil
objType = nil
objLevel = nil
objId = nil
(*temporary) = <table> {
}
(*temporary) = <table> {
 name = <table> {
 }
 index = 1
}
(*temporary) = "table index is nil"
ns = <table> {
 addon = "FarmHud"
 modules = <table> {
 }
 addon_short = "FH"
 L = <table> {
 }
 QuestArrowToken = <table> {
 }
 debugMode = false
}
numTrackingTypes = 25
trackingTypes = <table> {
}

Other addons?

Disabled everything but Farmhud, Bugsack and Buggrabber

lostmimic commented 3 months ago

Looks like that API changed to return a table object with all the data instead of as 6 separate objects?

lostmimic commented 3 months ago

Yep, this change to lines 137 and 138 of FarmHud.lua fixes the error and makes the addon function again. (Note use whatever naming convention you tend to use, I just used obj for the returned table)

            local obj = C_Minimap.GetTrackingInfo(i);
            trackingTypes[obj.texture] = {index=i,name=obj.name,active=obj.active,level=obj.subType};
hizuro commented 3 months ago

Thank you for reporting. Update coming soon.

Greetings Hizuro