diff ObjectiveTracker/ObjectiveTracker.lua @ 39:92534dc793f2

- restore the previous QuestLogSelection after pulling for selection-restricted quest data; fixes icon mixups while quest map is open - moved progressbar builders into the schema environment, with all the other Frame.lua functions; prep for configuration access - relegate the various removal events to a framescript in their corresponding blocks; this takes care of resolving dead frames
author Nenue
date Thu, 21 Apr 2016 16:43:37 -0400
parents 1f8f9cc3d956
children 03ed70f846de
line wrap: on
line diff
--- a/ObjectiveTracker/ObjectiveTracker.lua	Thu Apr 21 11:36:41 2016 -0400
+++ b/ObjectiveTracker/ObjectiveTracker.lua	Thu Apr 21 16:43:37 2016 -0400
@@ -264,8 +264,9 @@
 end
 
 local Event = {}
+
 Event.QUEST_LOG_UPDATE =  function()
-  return OBJECTIVE_TRACKER_UPDATE_MODULE_QUEST + OBJECTIVE_TRACKER_UPDATE_MODULE_BONUS_OBJECTIVE
+  return OBJECTIVE_TRACKER_UPDATE_QUEST
 end
 local GetNumQuestWatches, AddQuestWatch, SetSuperTrackedQuestID = GetNumQuestWatches, AddQuestWatch, SetSuperTrackedQuestID
 Event.QUEST_ACCEPTED = function(questLogIndex, questID, added)
@@ -277,10 +278,6 @@
   AddQuestWatch(questID)
 end
 
-Event.QUEST_REMOVED = function(questLogIndex, questID)
-  return OBJECTIVE_TRACKER_UPDATE_QUEST, questID, false
-end
-
 Event.QUEST_WATCH_LIST_CHANGED = function(questID, added)
   if ( added == true ) then
     if ( not IsQuestTask(questID) ) then
@@ -377,7 +374,8 @@
   ['RemoveAutoQuestPopUp'] = 'RemoveAutoQuestPopUp',
   ['AddTrackedAchievement'] = 'AddTrackedAchievement',
   ['RemoveTrackedAchievement'] = 'RemoveTrackedAchievement',
-  ['SetSuperTrackedQuestID'] = 'SetSuperTrackedQuestID'
+  ['SetSuperTrackedQuestID'] = 'SetSuperTrackedQuestID',
+  ['SelectQuestLogEntry'] = 'SelectQuestLogEntry',
 }
 local VeneerData
 
@@ -403,9 +401,6 @@
   local xp = GetQuestLogRewardXP();
   local playerTitle = GetQuestLogRewardTitle();
   ProcessQuestLogRewardFactions();
-  if previousSelection then
-    SelectQuestLogEntry(previousSelection)
-  end
 
   local rewards = {}
   local texture, name, isTradeskillSpell, isSpellLearned, hideSpellLearnText, isBoostSpell, garrFollowerID = GetQuestLogRewardSpell(questID)
@@ -416,6 +411,9 @@
       texture = texture,
     })
   end
+  if previousSelection then
+    SelectQuestLogEntry(previousSelection)
+  end
 
   t.numCurrencies = GetNumQuestLogRewardCurrencies(questID)
   for i = 1, t.numCurrencies do