From 385ab302f5157b82ccefc967f8fe7012e3dd76e6 Mon Sep 17 00:00:00 2001
From: SabinDeus <sabin@sabindeus.com>
Date: Mon, 14 Jun 2021 21:35:40 -0400
Subject: [PATCH] Changes from live version of Turn In plus alterations for BC
 Classic APIs

---
 TurnIn.lua   | 23 ++++++++++++++++-------
 TurnIn.toc   |  4 ++--
 TurnIn.xml   | 37 +++++++++++++++++++++++++++++++------
 TurnInUI.lua |  6 +++---
 4 files changed, 52 insertions(+), 18 deletions(-)

diff --git a/TurnIn.lua b/TurnIn.lua
index 19a633f..9046336 100644
--- a/TurnIn.lua
+++ b/TurnIn.lua
@@ -507,7 +507,7 @@ function TI_HandleGossipWindow(gorq)
 							for i2,v2 in ipairs(GossipOptions) do
 								if (v2.name == current.name) then
 									TI_debug(i1.."-Gossip Match Found: "..current.name..", "..current.type);
-									SelectGossipOption(i2);
+									C_GossipInfo.SelectOption(i2);
 									return;
 								end
 							end
@@ -533,12 +533,20 @@ function TI_HandleGossipWindow(gorq)
 				TI_availnumber = TI_availnumber + 1;
 				SAvQ(TI_availnumber-1);
 				TI_debug("Selecting Available Quest ".. TI_availnumber-1);
+				TI_ResetPointers();
 				return;
 			elseif(current.type == "activequest" and #ActiveQuests > 0 and TI_activenumber <= #ActiveQuests) then
-				TI_activenumber = TI_activenumber + 1;
-				SAcQ(TI_activenumber-1);
-				TI_debug("Selecting Active Quest ".. TI_activenumber-1);
-				return;
+				while TI_activenumber <= #ActiveQuests do
+					TI_activenumber = TI_activenumber + 1;
+					if ActiveQuests[TI_activenumber-1].isComplete then
+						TI_debug("Selecting Active Quest ".. TI_activenumber-1 .. " " .. ActiveQuests[TI_activenumber-1].name );
+						SAcQ(TI_activenumber-1);
+						TI_ResetPointers();
+						return;
+					else
+						TI_debug("Active Quest ".. TI_activenumber-1 .. " " .. ActiveQuests[TI_activenumber-1].name .. " is not complete")
+					end
+				end
 			elseif(#GossipOptions > 0) then
 				for j,val in ipairs(GossipOptions) do
 					if(val.type == current.type) then
@@ -678,7 +686,8 @@ function TI_TabulateGossipActiveQuests(...)
 	end
 	return x;
 end
-	
+
+
 --[[this function stolen from WhisperCast by Sarris, whom I love dearly for his contribution to Paladins everywhere. 
 ]]
 function TI_copyTable( src )
@@ -701,4 +710,4 @@ function toggle(arg)
 	else
 		return true;
 	end
-end
\ No newline at end of file
+end
diff --git a/TurnIn.toc b/TurnIn.toc
index 2002b53..d46b5c5 100644
--- a/TurnIn.toc
+++ b/TurnIn.toc
@@ -1,5 +1,5 @@
-## Interface: 11302
-## X-Compatible-With: 11302
+## Interface: 20501
+## X-Compatible-With: 90002
 ## Title: Turn In
 ## Notes: Automates the selection of quest and gossip options.
 ## SavedVariables: TI_status, TI_NPCDB, TI_NPCIndex
diff --git a/TurnIn.xml b/TurnIn.xml
index 1862cc7..b779dee 100644
--- a/TurnIn.xml
+++ b/TurnIn.xml
@@ -2,7 +2,7 @@
 	<Script file="TurnIn.lua" />
 	<Script file="TurnInUI.lua" />
 	
-	<Frame name="TI_OptionFrameBoxTemplate" virtual="true">
+	<Frame name="TI_OptionFrameBoxTemplate" virtual="true" inherits="BackdropTemplate">
 		<!-- shamelessly stolen from blizzard -->
 		<Layers>
 			<Layer level="BACKGROUND">
@@ -17,6 +17,9 @@
 				</FontString>
 			</Layer>
 		</Layers>
+		<KeyValues>
+			<KeyValue key="backdropInfo" value="BACKDROP_TOOLTIP_16_16_5555" type="global"/>
+		</KeyValues>
 		<Backdrop bgFile="Interface\Tooltips\UI-Tooltip-Background" edgeFile="Interface\Tooltips\UI-Tooltip-Border" tile="true">
 			<EdgeSize>
 				<AbsValue val="16"/>
@@ -30,6 +33,7 @@
 		</Backdrop>
 		<Scripts>
 			<OnLoad>
+				self:ApplyBackdrop();
 				self:SetBackdropBorderColor(0.4, 0.4, 0.4);
 				self:SetBackdropColor(0.5, 0.5, 0.5);
 			</OnLoad>
@@ -333,7 +337,7 @@
 		</Frames>
 	</Frame>
 	
-	<Frame Name="TI_TempNPCListWindow" parent="UIParent" frameStrata="DIALOG" toplevel="true" enableMouse="true" movable="true" hidden="true">
+	<Frame Name="TI_TempNPCListWindow" parent="UIParent" frameStrata="DIALOG" toplevel="true" enableMouse="true" movable="true" hidden="true" inherits="BackdropTemplate">
 		<Size>
 			<AbsDimension x="200" y="130" />
 		</Size>
@@ -403,6 +407,9 @@
 				</Anchors>
 			</Frame>
 		</Frames>
+		<KeyValues>
+			<KeyValue key="backdropInfo" value="BACKDROP_TOOLTIP_16_16_5555" type="global"/>
+		</KeyValues>
 		<Backdrop bgFile="Interface\DialogFrame\UI-DialogBox-Background" edgeFile="Interface\Tooltips\UI-Tooltip-Border" tile="true">
 			<EdgeSize>
 				<AbsValue val="16"/>
@@ -415,6 +422,11 @@
 			</BackgroundInsets>
 		</Backdrop>
 		<Scripts>
+			<OnLoad>
+				self:ApplyBackdrop();
+				self:SetBackdropBorderColor(0.6, 0.6, 0.6);
+				self:SetBackdropColor(0.2, 0.2, 0.2);
+			</OnLoad>
 			<OnMouseUp>
 				if ( self.isMoving ) then
 					self:StopMovingOrSizing();
@@ -436,7 +448,7 @@
 		</Scripts>
 	</Frame>
 	
-	<Frame Name="TI_OptionsFrame" frameStrata="DIALOG" toplevel="true" enableMouse="true" enableMouseWheel="true" movable="true" parent="UIParent">
+	<Frame Name="TI_OptionsFrame" inherits="BackdropTemplate" frameStrata="DIALOG" toplevel="true" enableMouse="true" enableMouseWheel="true" movable="true" parent="UIParent">
 		<TitleRegion>
 			<Size>
 				<AbsDimension x="128" y="64"/>
@@ -450,6 +462,9 @@
 			</Anchors>
 		 </TitleRegion>
 		
+		<KeyValues>
+			<KeyValue key="backdropInfo" value="BACKDROP_DIALOG_32_32" type="global"/>
+		</KeyValues>
 		<Backdrop bgFile="Interface\DialogFrame\UI-DialogBox-Background" edgeFile="Interface\DialogFrame\UI-DialogBox-Border" tile="true">
 			<BackgroundInsets>
 				<AbsInset left="11" right="12" top="12" bottom="11"/>
@@ -463,9 +478,17 @@
 		</Backdrop>
 		
 		<Scripts>
-			<OnShow>
-				
-			</OnShow>
+			<OnLoad>
+				self:ApplyBackdrop()
+                self:EnableMouse(true)
+                self:RegisterForDrag("LeftButton")
+            </OnLoad>
+            <OnDragStart>
+                self:StartMoving()
+            </OnDragStart>
+            <OnDragStop>
+                self:StopMovingOrSizing()
+            </OnDragStop>
 		</Scripts>
 		<Size>
 			<AbsDimension x="300" y="500"/>
@@ -658,6 +681,7 @@
 			<Frame name="TI_OptionsPriorityContainer" inherits="TI_OptionFrameBoxTemplate">
 				<Scripts>
 					<OnLoad>
+						self:ApplyBackdrop();
 						self:SetBackdropBorderColor(0.4, 0.4, 0.4);
 						self:SetBackdropColor(0.15, 0.15, 0.15);
 						getglobal(self:GetName().."Title"):SetText("Gossip Options Priority List");
@@ -865,6 +889,7 @@
 			<Frame name="TI_OptionsNPCContainer" inherits="TI_OptionFrameBoxTemplate">
 				<Scripts>
 					<OnLoad>
+						self:ApplyBackdrop();
 						self:SetBackdropBorderColor(0.4, 0.4, 0.4);
 						self:SetBackdropColor(0.15, 0.15, 0.15);
 						getglobal(self:GetName().."Title"):SetText("NPC Database");
diff --git a/TurnInUI.lua b/TurnInUI.lua
index e47cf94..81b47d8 100644
--- a/TurnInUI.lua
+++ b/TurnInUI.lua
@@ -20,16 +20,16 @@ function TI_PopulateOptions(arg)
 				if(v.icon) then
 					iconpath = v.icon;
 				else
-					iconpath = "Interface\\GossipFrame\\ActiveQuestIcon";
+					iconpath = "Interface/GossipFrame/ActiveQuestIcon";
 				end
 			elseif(v.type == "availquest") then
 				if(v.icon) then
 					iconpath = v.icon;
 				else
-					iconpath = "Interface\\GossipFrame\\AvailableQuestIcon";
+					iconpath = "Interface/GossipFrame/AvailableQuestIcon";
 				end
 			else
-				iconpath = "Interface\\GossipFrame\\" .. v.type .. "GossipIcon";
+				iconpath = "Interface/GossipFrame/" .. v.type .. "GossipIcon";
 			end
 			getglobal(entry .. "_Icon"):SetTexture(iconpath);
 			getglobal(entry .. "_Check"):SetChecked(v.state);