From b6ee61658973fee84e667a35c817a646ac2bcc85 Mon Sep 17 00:00:00 2001
From: Adrian L Lange <git@p3lim.net>
Date: Tue, 6 Aug 2024 19:53:10 +0200
Subject: [PATCH] Skip long dialogue options

---
 gossip.lua | 25 +++++++++++++++++++++----
 1 file changed, 21 insertions(+), 4 deletions(-)

diff --git a/gossip.lua b/gossip.lua
index b6e790b..8703206 100644
--- a/gossip.lua
+++ b/gossip.lua
@@ -39,17 +39,34 @@ function addon:GOSSIP_SHOW()
 		return
 	end
 
+	-- need to iterate all the options first before we can select them
+	local gossipQuests = {}
+	local gossipSkips = {}
+
 	local gossip = C_GossipInfo.GetOptions()
 	for _, info in next, gossip do
 		if DARKMOON_GOSSIP[info.gossipOptionID] and addon:GetOption('paydarkmoonfaire') then
+			-- we can select this one directly since it never interferes with the others
 			C_GossipInfo.SelectOption(info.gossipOptionID, '', true)
-		elseif QUEST_GOSSIP[info.gossipOptionID] and addon:GetOption('autoquestgossip') then
-			C_GossipInfo.SelectOption(info.gossipOptionID)
-		elseif FlagsUtil.IsSet(info.flags, Enum.GossipOptionRecFlags.QuestLabelPrepend) and addon:GetOption('autoquestgossip') then
-			C_GossipInfo.SelectOption(info.gossipOptionID)
+			return
+		elseif QUEST_GOSSIP[info.gossipOptionID] then
+			table.insert(gossipQuests, info.gossipOptionID)
+		elseif FlagsUtil.IsSet(info.flags, Enum.GossipOptionRecFlags.QuestLabelPrepend) then
+			table.insert(gossipQuests, info.gossipOptionID)
+		elseif info.name:sub(1, 11) == '|cFFFF0000<' then
+			-- TODO: this might get a flag in the future
+			table.insert(gossipSkips, info.gossipOptionID)
 		end
 	end
 
+	if #gossipSkips > 0 and addon:GetOption('autoquestgossip') then
+		C_GossipInfo.SelectOption(gossipSkips[1])
+		return
+	elseif #gossipQuests > 0 and addon:GetOption('autoquestgossip') then
+		C_GossipInfo.SelectOption(gossipQuests[1])
+		return
+	end
+
 	if (C_GossipInfo.GetNumActiveQuests() + C_GossipInfo.GetNumAvailableQuests()) > 0 then
 		-- don't automate misc gossip if the NPC is a quest giver
 		return