From: Liza Carvelli <liza@carvel.li>
Date: Sun, 9 Mar 2025 21:41:54 +0000 (+0100)
Subject: Fix Questionable ignoring level check for next MSQ during load screens, thus potentia... 
X-Git-Tag: v4.25~9
X-Git-Url: https://git.jacobcasper.com/?a=commitdiff_plain;h=cb205b2a28db4823f619ae54919f36efbae38623;p=Questionable.git

Fix Questionable ignoring level check for next MSQ during load screens, thus potentially resetting your current quest progress when loading into another zone
---

diff --git a/Questionable/Functions/QuestFunctions.cs b/Questionable/Functions/QuestFunctions.cs
index fc906e00..f0a41ecb 100644
--- a/Questionable/Functions/QuestFunctions.cs
+++ b/Questionable/Functions/QuestFunctions.cs
@@ -291,10 +291,14 @@ internal sealed unsafe class QuestFunctions
         else if (!IsReadyToAcceptQuest(currentQuest))
             return default;
 
+        var currentLevel = _clientState.LocalPlayer?.Level;
+
+        // are we in a loading screen?
+        if (currentLevel == null)
+            return default;
+
         // if we're not at a high enough level to continue, we also ignore it
-        var currentLevel = _clientState.LocalPlayer?.Level ?? 0;
-        if (currentLevel != 0 &&
-            _questRegistry.TryGetQuest(currentQuest, out Quest? quest)
+        if (_questRegistry.TryGetQuest(currentQuest, out Quest? quest)
             && quest.Info.Level > currentLevel)
             return default;