Error on Light Leather for AQ War Effort - Darrowshire
tsheeley opened this issue ยท 5 comments
When I'm tracking the Light Leather for the War Effort and mouse over the tracker, I get the error below. I tried turning off Tooltips in the options, but it still gives the error.
I've got a GitDesktop clone directly into my Addons folder and then a symlink it to give the proper folder structure and pull 3.7 from that... so I should be using the current version.
@tsheeley As explained in the commit message, this issue should be resolved. If you continue having problems, please re-open the issue.
Your commit message says you were unable to reproduce the error, but isn't it just a matter of a quest not found in the quest table? The quest "Light Leather for the War Effort" isn't in the quest table.
I assumed it was quest 8511 (The Alliance Needs Light Leather!) and tested with that. The quest shows up in the tracker and there are even some notes, though I doubt the marked creatures actually drop light leather as loot...
Could be some servers have significantly different quest names, so we should make sure at least nothing completely breaks in those situations. Obviously we can't correctly handle quests with incorrect names, but we can try to handle errors a bit better.
An error in theses cases shouldn't happen anymore, because the lookup function now returns QuestieLevLookup[realName] or {}
, so if QuestieLevLookup[realName] == nil
then an empty table is returned and used where the function is called as an argument of pairs()
resulting in zero loops.