Merge pull request #126 from Saifullah-dev/125-fix-initial-path-isnt-working-correctly

Saifullah-dev · web-flow · commit 487b888e0ee4 · 2024-12-30T05:46:11.000+05:00
fix(initialPath): change initial path's logic to wait for the files r…
diff --git a/frontend/src/contexts/FileNavigationContext.jsx b/frontend/src/contexts/FileNavigationContext.jsx
@@ -23,7 +23,7 @@ export const FileNavigationProvider = ({ children, initialPath }) => {
   }, [files, currentPath]);
 
   useEffect(() => {
-    if (!isMountRef.current) {
+    if (!isMountRef.current && Array.isArray(files) && files.length > 0) {
       setCurrentPath(files.some((file) => file.path === initialPath) ? initialPath : '');
       isMountRef.current = true;
     }

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ export const FileNavigationProvider = ({ children, initialPath }) => {`
`23`	`23`	`}, [files, currentPath]);`
`24`	`24`
`25`	`25`	`useEffect(() => {`
`26`		`- if (!isMountRef.current) {`
	`26`	`+ if (!isMountRef.current && Array.isArray(files) && files.length > 0) {`
`27`	`27`	`setCurrentPath(files.some((file) => file.path === initialPath) ? initialPath : '');`
`28`	`28`	`isMountRef.current = true;`
`29`	`29`	`}`