Updates the quest abandoning system a little. Updates some quest/NPC …

…information for Shadowlands beta.

Grail-Quests-_classic_/Grail-Quests-_classic_.lua

@@ -4408,5 +4408,6 @@ end
 --	35395	1.13.5	2020-07-30	2020-0x-xx
 --	35753	1.13.5	2020-09-01	2020-09-03
 --	36035	1.13.5	2020-09-24	2020-xx-xx
+--	36307	1.13.5	2020-10-19	2020-10-21
 
 end

Grail-Quests-_retail_-enUS/Grail-Quests-_retail_-enUS.lua

@@ -33462,6 +33462,7 @@ G[61885]="Eyegor's Special Friends"
 G[61947]='Lurking In The Shadows'
 G[61948]='Airborne Defense Force'
 G[62085]='...Why Me?'
+G[62166]="Tal-Inara's Call"
 G[62287]='Observing Arenas'
 G[62740]='Dark Aspirations'
 G[62785]='I Could Be A Contender'

Grail-Quests-_retail_/Grail-Quests-_retail_.lua

@@ -33479,7 +33479,7 @@ G[59199]='K4096 L3355647 A:160037 T:160037 P:59198 E34003'
 G[59200]='K4096 L3355647 A:160037 T:160037 P:59199 E34003'
 G[59202]='K4096 L3553023 A:168381 T:164843 P:59009+59231+60733 E34714'
 G[59203]='L3947775 A:159830 T:159830 P:59879 E34714'
-G[59206]='L3618815 A:164545 T:162928 P:60737 E34714'
+G[59206]='K4096 L3618815 A:164545 T:162928 P:60737 E34714'
 G[59209]='E34137'	-- TODO: Yoshimo: Revendreth
 G[59210]='K4096 L3487231 A:162487 T:164326 P:59172 E34714'
 G[59211]='A:-181525 T:-1525 P:a59211'
@@ -33978,8 +33978,8 @@ G[61631]='K262144 A:-511565 T:-1565 P:a61631'
 G[61667]='K262144 A:-351536 T:-1536 P:a61667 E35598'
 G[61689]='K262144 A:-251533 T:-1533 P:a61689 E35213'
 G[61708]='K262144 A:-281536 T:-1536 P:a61708 E35598'
-G[61715]='L3618815 A:162928 T:173384 P:59206 E35167'
-G[61716]='L3618815 A:173384 T:700071 P:61715 E35360'
+G[61715]='K4096 L3684607 A:162928 T:173384 P:59206 E35167'
+G[61716]='K4096 L3684607 A:173384 T:700071 P:61715 E35360'
 G[61718]='T:157294'
 G[61729]='T:164741 P:59319 E35282'
 G[61783]='K278528 A:-141533 T:-1533 P:a61783'

Grail/Grail.lua

@@ -515,6 +515,7 @@
 --			Adds slash command "/grail treasures" which toggles the old method of LOOT_CLOSED to record information when looting.
 --			Adds GetCurrencyInfo() which works around issues for which Blizzard API to use.
 --			Ensures AzeriteLevelMeetsOrExceeds() checks to make sure API used are present.
+--			Reworks quest abandoning to use events instead of the old routines.
 --
 --	Known Issues
 --
@@ -1268,12 +1269,6 @@ experimental = false,	-- currently this implementation does not reduce memory si
 						QuestFrameCompleteQuestButton:SetScript("OnClick", function() self:_QuestRewardCompleteButton_OnClick() end);
 					end
 
-					self.origAbandonQuestFunction = SetAbandonQuest
-					SetAbandonQuest = function() self:_QuestAbandonStart() end
-
-					self.origConfirmAbandonQuestFunction = AbandonQuest
-					AbandonQuest = function() self:_QuestAbandonStop() end
-
 					--	For the choice of types of quest on Isle of Thunder the following function is eventually
 					--	called with anId which is associated with the button in the UI.
 					local newSendQuestChoiceFunction = function(anId) self:_SendQuestChoiceResponse(anId) end
@@ -2019,7 +2014,7 @@ end,
 				-- this happens for both abandon and turn-in
 				-- and turn-in is first, so we can know we are abandoning or not
 				if nil == self.questTurningIn then
-					self:_StatusCodeInvalidate({ tonumber(questId) }, self.delayQuestRemoved)
+					self:_QuestAbandon(questId)
 				end
 				self.questTurningIn = nil
 			end,
@@ -3353,6 +3348,8 @@ end,
 			local functionKey = "+"
 			if "Complete" == callbackType then
 				functionKey = "="
+			elseif "Abandon" == callbackType then
+				functionKey = "-"
 			end
 			local message = strformat("%s %s(%d)", functionKey, Grail:QuestName(questId) or "NO NAME", questId)
 			if "Accept" == callbackType or "Complete" == callbackType then
@@ -9591,19 +9588,10 @@ print("end:", strgsub(controlTable.something, "|", "*"))
 		end,
 
 		--	Internal Use.
-		--	Routine used to hook the function for abandoning a quest.  This is needed because the events that Blizzard issues
-		--	are not adequate for our desired tasks.  One of three needed for abandoning.
-		_QuestAbandonStart = function(self)
-			self.abandoningQuestIndex = GetQuestLogSelection()
-			self.origAbandonQuestFunction()
-		end,
+		_QuestAbandon = function(self, questId)
+			questId = tonumber(questId)
+			if nil == questId then return end
 
-		--	Internal Use.
-		--	Routine used to hook the function for abandoning a quest.  This is needed because the events that Blizzard issues
-		--	are not adequate for our desired tasks.  One of three needed for abandoning.
-		_QuestAbandonStop = function(self)
-			local questTitle, level, questTag, suggestedGroup, isHeader, isCollapsed, isComplete, isDaily, questId = self:GetQuestLogTitle(self.abandoningQuestIndex)
-			self.origConfirmAbandonQuestFunction()
 			if nil ~= self.quests[questId] then
 				self:_MarkQuestInDatabase(questId, GrailDatabasePlayer["abandonedQuests"])
 			end
@@ -9625,6 +9613,9 @@ print("end:", strgsub(controlTable.something, "|", "*"))
 				end
 				self:_StatusCodeInvalidate(questsToInvalidate)
 			end
+
+			self:_StatusCodeInvalidate({ questId }, self.delayQuestRemoved)
+
 			self:_PostDelayedNotification("Abandon", questId, self.abandonPostNotificationDelay)
 		end,