From: Liza Carvelli <liza@carvel.li>
Date: Tue, 12 Aug 2025 17:38:52 +0000 (+0200)
Subject: Handle errors if your only tracked quests are low priority
X-Git-Tag: v6.1~11
X-Git-Url: https://git.jacobcasper.com/?a=commitdiff_plain;h=716bf637680baed3a5ead2bcd2a263aa5b247a02;p=Questionable.git

Handle errors if your only tracked quests are low priority
---

diff --git a/Questionable/Functions/QuestFunctions.cs b/Questionable/Functions/QuestFunctions.cs
index 70783c75..9d99f2bb 100644
--- a/Questionable/Functions/QuestFunctions.cs
+++ b/Questionable/Functions/QuestFunctions.cs
@@ -157,14 +157,14 @@ internal sealed unsafe class QuestFunctions
             }
         }
 
-        if (trackedQuests.Count > 0)
+        if (_configuration.General.SkipLowPriorityDuties && trackedQuests.Count > 0)
         {
-            if (_configuration.General.SkipLowPriorityDuties)
-            {
-                var lowPriorityQuests = _questRegistry.LowPriorityContentFinderConditionQuests;
-                trackedQuests.RemoveAll(x => lowPriorityQuests.Any(y => x.Quest == y.QuestId && x.Sequence == y.Sequence));
-            }
+            var lowPriorityQuests = _questRegistry.LowPriorityContentFinderConditionQuests;
+            trackedQuests.RemoveAll(x => lowPriorityQuests.Any(y => x.Quest == y.QuestId && x.Sequence == y.Sequence));
+        }
 
+        if (trackedQuests.Count > 0)
+        {
             // if we have multiple quests to turn in for an allied society, try and complete all of them
             var (firstTrackedQuest, firstTrackedSequence) = trackedQuests.First();
             EAlliedSociety firstTrackedAlliedSociety =