From 0413b2c56e1baa80e3474b542af28122e3a58e9a Mon Sep 17 00:00:00 2001 From: p3lim Date: Wed, 12 Jan 2011 13:01:01 +0100 Subject: [PATCH] Clarify the notes --- Monomyth.lua | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/Monomyth.lua b/Monomyth.lua index 487b6b7..3579042 100644 --- a/Monomyth.lua +++ b/Monomyth.lua @@ -71,9 +71,8 @@ addon:Register('QUEST_COMPLETE', function() for index = 1, GetNumQuestChoices() do local link = GetQuestItemLink('choice', index) if(not link) then - -- QUEST_COMPLETE can be a bit early, and so the items - -- are not added to the frame yet (cache missing perhaps). - -- either cache items or queue then try again + -- Item is not located in the cache yet, let it request it + -- from the server and run this again after its received return end @@ -92,9 +91,8 @@ addon:Register('QUEST_AUTOCOMPLETE', function() local quest, type = GetAutoQuestPopUp(index) if(type == 'COMPLETE') then - -- can sometimes fail to complete - -- the quest is not considered complete by the quest system - -- perhaps queue and try again later + -- The quest may not be considered complete by the server + -- We should check then queue and try again when it is ShowQuestComplete(GetQuestLogIndexByID(quest)) end end @@ -106,8 +104,8 @@ addon:Register('BAG_UPDATE', function(bag) for slot = 1, GetContainerNumSlots(bag) do local _, id, active = GetContainerItemQuestInfo(bag, slot) if(id and not active) then - -- can sometimes cause a disconnect - -- most likely because the item is not cached yet + -- We should check if the item is cached yet + -- The negative result of this is a disconnect UseContainerItem(bag, slot) end end -- 1.7.9.5