Questie

Questie

116M Downloads

Error on Light Leather for AQ War Effort - Darrowshire

tsheeley opened this issue ยท 5 comments

commented

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.

image

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.

commented

@tsheeley As explained in the commit message, this issue should be resolved. If you continue having problems, please re-open the issue.

commented

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.

commented

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...

commented

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.

commented

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.