Load winch settings when re-opening settings tab

DREDGE-Mods · Jul 12, 2024 · 11c0d66 · 11c0d66
1 parent b16ff94
commit 11c0d66
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/src/components/appcontext.tsx b/src/components/appcontext.tsx
@@ -176,11 +176,13 @@ export const AppProvider = (props: React.PropsWithChildren) => {
     }
 
     const updateWinchConfig = () => {
-        invoke("update_winch_config", {
-            "json": JSON.stringify(state.winchConfig, null, 2),
-            dredgePath: state.dredgePath})
-            .then(reloadMods)
-            .catch(genericHandleError);
+        if (state.winchConfig != null) {
+            invoke("update_winch_config", {
+                "json": JSON.stringify(state.winchConfig, null, 2),
+                dredgePath: state.dredgePath})
+                .then(reloadMods)
+                .catch(genericHandleError);
+        }
     }
 
     // fetch dredge path on mount

diff --git a/src/components/settings/settings.tsx b/src/components/settings/settings.tsx
@@ -14,8 +14,17 @@ export const Settings = (props: {path_correct?: boolean}) => {
 
     const context = useContext(AppContext);
 
+    const [shouldCheckWinchConfig, setshouldCheckWinchConfig] = useState(true);
     const [path, setPath] = useState("");
 
+    // Opening the settings tab will try reloading everything: Checks if winch config has been setup by the modloader yet
+    useEffect(() => {
+        if (context.state.winchConfig == null && shouldCheckWinchConfig) {
+            context?.reloadMods()
+        }
+        setshouldCheckWinchConfig(false);
+    });
+
     // Fetch DREDGE path from context; only called once on load
     useEffect(() => {
         if (path === "") {
@@ -109,7 +118,6 @@ export const Settings = (props: {path_correct?: boolean}) => {
                     onChange={(e) => {
                         const value = e.target.value;
                         setPath(value);
-
                     }}
                     value={path}
                 />