From bb1612a231fb97157a3422deada73d8a591071a2 Mon Sep 17 00:00:00 2001
From: Clara Ni <clara.ni@outlook.fr>
Date: Fri, 6 Dec 2024 17:47:27 +0100
Subject: [PATCH] fixup! front: fetch pathSteps coordinates if pathfinding
 failed when editing a train

---
 .../hooks/useSetupItineraryForTrainUpdate.ts  | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/front/src/applications/operationalStudies/hooks/useSetupItineraryForTrainUpdate.ts b/front/src/applications/operationalStudies/hooks/useSetupItineraryForTrainUpdate.ts
index 57e6caa0fd0..a3bd63dc521 100644
--- a/front/src/applications/operationalStudies/hooks/useSetupItineraryForTrainUpdate.ts
+++ b/front/src/applications/operationalStudies/hooks/useSetupItineraryForTrainUpdate.ts
@@ -92,15 +92,13 @@ const useSetupItineraryForTrainUpdate = (
       const tracks = await getTrackSectionsByIds(trackSectionIds);
 
       // get operational points
-      let ops: SearchResultItemOperationalPoint[] = [];
       const searchPayload = buildOpSearchQuery(infraId, [trainSchedule]);
-
-      if (searchPayload) {
-        ops = (await postSearch({
-          searchPayload,
-          pageSize: 1000,
-        }).unwrap()) as SearchResultItemOperationalPoint[];
-      }
+      const ops = searchPayload
+        ? ((await postSearch({
+            searchPayload,
+            pageSize: 1000,
+          }).unwrap()) as SearchResultItemOperationalPoint[])
+        : [];
 
       // match path steps with track sections or operational points
       const pathStepsWithCoordinates = trainSchedule.path.map((step, index) => {
@@ -116,15 +114,13 @@ const useSetupItineraryForTrainUpdate = (
           let op: SearchResultItemOperationalPoint | undefined;
           if ('uic' in step) {
             op = ops.find((o) => o.uic === step.uic);
-            name = `${op?.name} - ${step.uic}`;
           } else if ('trigram' in step) {
             op = ops.find((o) => o.trigram === step.trigram);
-            name = `${op?.name} - ${step.trigram}`;
           } else {
             op = ops.find((o) => o.obj_id === step.operational_point);
-            name = `${op?.name} - ${step.operational_point}`;
           }
           coordinates = op?.geographic.coordinates;
+          name = `${op?.name}`;
         }
 
         return {
@@ -134,7 +130,6 @@ const useSetupItineraryForTrainUpdate = (
         };
       });
 
-      // update le store
       dispatch(updatePathSteps(pathStepsWithCoordinates));
     };