No nil check for QID before calling POI API call.
gameldar opened this issue ยท 3 comments
I should have written down when this occurred - but I'm pretty sure it was while going about things in the Alliance Hellfire quests.
At a guess it was when the following entered the rendered display:
N Silken Thread |N|Buy a Silken thread from Hama. | |L|4291 1|M|54.60,63.70|
There is no QID defined for this step and so we're passing a nil QID to the POI lookup.
I'm guessing the guide should also be fixed?
Date: 2010-08-15 15:49:01
ID: 1
Error occured in: Global
Count: 1
Message: ...ce\AddOns\WoWPro_Leveling\WoWPro_Leveling_Parser.lua line 220:
Usage: QuestPOIGetIconInfo(questID)
Debug:
...ce\AddOns\WoWPro_Leveling\WoWPro_Leveling_Parser.lua:220: RowUpdate()
WoWPro\WoWPro_Broker.lua:61: RowContentUpdate()
WoWPro\WoWPro_Broker.lua:73: UpdateGuide()
WoWPro\WoWPro_Broker.lua:163: CompleteStep()
...ce\AddOns\WoWPro_Leveling\WoWPro_Leveling_Parser.lua:455: AutoCompleteQuestUpdate()
...ce\AddOns\WoWPro_Leveling\WoWPro_Leveling_Parser.lua:376: EventHandler()
WoWPro\WoWPro_Broker.lua:149:
WoWPro\WoWPro_Broker.lua:125
AddOns:
Swatter, v5.8.4723 (CreepyKangaroo)
Ace3, v
Babylonian, v5.1.DEV.130
Bartender4, v4.4.2
Chatter, v1.0
Clique, vwowi:revision
Combuctor, v2.2.2
CombuctorSets, v
Configator, v5.1.DEV.190
DBMCore, v
Gatherer, v3.1.14
Grid, v1.30300.1308
GridStatusAFK, v3.2.0.40
GridStatusDungeonRole, v1.1.1
GridStatusHots, v3.32
GridStatusRaidDebuff, vv3.01
GridStatusRaidIcons, v30300-2
GridStatusRDBC, v1.0
GridStatusRDWotLK, v1.0
GridStatusThreat, v30300-3
kgPanels, vv1.5.2
LauncherMenu, vv1.2
LeatrixGfx, v
LightHeaded, v301
LightHeadedDataC, v301
LightHeadedDataD, v301
LightHeadedDataQIDNames, v301
Livestock, v1.2.2
LoadIT, v
MinimapButtonFrame, v3.1.0
MTarget, v2.1.3
Multishot, vr41
Omen, v3.0.9
OmniCC, v2.5.9
Outfitter, v5.0b8
Postal, v3.3.2
PowerAuras, v3.0.0K
Quartz, v3.0.3.1
RatingBuster, v
Recount, v
SatrinaBuffFrame, v3.1
sct, v6.2
SLDataText, v3.1.4
teksLoot, v3.3.0.9
TidyPlates, v5.12.3
TidyPlatesGrey, v
TidyPlatesThreatPlates, v3.0.4
TomTom, vwowi:revision
WIM, v3.3.7
WoWPro, v1.0.0
WoWProLeveling, v1.0.0
YssDrop, vv1.8.10
BlizRuntimeLib_enUS v3.3.5.30300
(ck=498)
Yes it should and thank you :) Glad it was caught before sending it to the release version next week.