Quantcast

Clarify the notes

p3lim [01-12-11 - 12:01]
Clarify the notes
Filename
Monomyth.lua
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