Bug: direct link to plant location does not work (first time)

src/features/projectsV2/ProjectSnippet/microComponents/ImageSection.tsx

@@ -41,9 +41,7 @@ const ImageSection = (props: ImageSectionProps) => {
   const isEmbed = embed === 'true';
 
   const handleBackButton = () => {
-    if (setPreventShallowPush) {
-      setPreventShallowPush(true);
-    }
+    if (setPreventShallowPush) setPreventShallowPush(true);
 
     const previousPageRoute = sessionStorage.getItem('backNavigationUrl');
     const defaultRoute = `/${locale}`;

src/features/projectsV2/ProjectsContext.tsx

@@ -314,7 +314,8 @@
       page !== 'project-details' ||
       singleProject === null ||
       selectedSite !== null ||
-      (requestedPlantLocation && requestedSite)
+      (requestedPlantLocation && requestedSite) ||
+      plantLocations === null
     )
       return;
 
@@ -353,21 +354,23 @@
     selectedPlantLocation,
     selectedSite,
     hasNoSites,
+    plantLocations,
   ]);
   useEffect(() => {
+    if (requestedPlantLocation && plantLocations === null) return;
     if (
       !router.isReady ||
       page !== 'project-details' ||
       singleProject === null ||
       selectedPlantLocation !== null ||
       preventShallowPush
     )
       return;
     // Handle the case where a direct link requests a specific site (via URL query)
     // This will update the site param based on the requestedSite. If the requested site ID is invalid,
     // it falls back to the default (first) site.
     if (requestedSite && selectedSite === null && !hasNoSites) {
       const index = singleProject.sites?.findIndex(
         (site) => site.properties.id === requestedSite
       );
 
@@ -398,6 +401,7 @@
     requestedSite,
     router.isReady,
     selectedPlantLocation,
+    plantLocations,
     preventShallowPush,
     hasNoSites,
   ]);

src/features/projectsV2/ProjectsMap/SingleProjectView.tsx

@@ -17,13 +17,13 @@ interface Props {
 }
 
 const SingleProjectView = ({ mapRef }: Props) => {
+  const router = useRouter();
   const { singleProject, selectedSite, selectedPlantLocation, plantLocations } =
     useProjects();
+  const { ploc: requestedPlantLocation, site: requestedSite } = router.query;
   if (singleProject === null) return null;
-
   const { isSatelliteView, handleViewStateChange, setIsSatelliteView } =
     useProjectsMap();
-  const router = useRouter();
 
   const sitesGeojson = useMemo(() => {
     return {
@@ -34,7 +34,6 @@ const SingleProjectView = ({ mapRef }: Props) => {
   }, [singleProject?.sites]);
   const hasNoSites = sitesGeojson.features.length === 0;
   // Zoom to plant location
-
   useEffect(() => {
     if (!router.isReady || selectedPlantLocation === null) return;
     const { geometry } = selectedPlantLocation;
@@ -57,11 +56,16 @@ const SingleProjectView = ({ mapRef }: Props) => {
         zoomToLocation(handleViewStateChange, lon, lat, 20, 4000, mapRef);
       }
     }
-  }, [selectedPlantLocation, router.isReady]);
+  }, [selectedPlantLocation, router.isReady, requestedPlantLocation]);
 
   // Zoom to project site
   useEffect(() => {
-    if (!router.isReady || selectedPlantLocation !== null) return;
+    if (
+      !router.isReady ||
+      selectedPlantLocation !== null ||
+      Boolean(requestedPlantLocation)
+    )
+      return;
     if (sitesGeojson.features.length > 0 && selectedSite !== null) {
       zoomInToProjectSite(
         mapRef,
@@ -72,6 +76,7 @@ const SingleProjectView = ({ mapRef }: Props) => {
       );
     } else {
       const { lat: latitude, lon: longitude } = singleProject.coordinates;
+      if (!(singleProject.sites?.length === 0)) return;
 
       if (typeof latitude === 'number' && typeof longitude === 'number') {
         // Zoom into the project location that has no site
@@ -85,9 +90,17 @@ const SingleProjectView = ({ mapRef }: Props) => {
         );
       }
     }
-  }, [selectedSite, sitesGeojson, router.isReady, selectedPlantLocation]);
+  }, [
+    selectedSite,
+    sitesGeojson,
+    router.isReady,
+    selectedPlantLocation,
+    requestedPlantLocation,
+    requestedSite,
+  ]);
 
   useEffect(() => {
+    if (plantLocations === null) return;
     const hasNoPlantLocations = !plantLocations?.length;
     const isSingleProjectLocation = hasNoPlantLocations && hasNoSites;
     // Satellite view will be:

src/utils/projectV2.ts

@@ -251,14 +251,12 @@ export const areMapCoordsEqual = (
 export const getLocalizedPath = (path: string, locale: string): string => {
   // Strip query parameters if present
   const pathWithoutQuery = path.split('?')[0];
-
   // Remove trailing slash if present
   const cleanPath = pathWithoutQuery.endsWith('/')
     ? pathWithoutQuery.slice(0, -1)
     : pathWithoutQuery;
-
   // Handle root path special case
-  if (cleanPath === '' || cleanPath === '/') {
+  if (cleanPath === '' || cleanPath === '/' || cleanPath === `/${locale}`) {
     return `/${locale}`;
   }