front: fix displayed op on manchette

front/src/applications/operationalStudies/__tests__/upsertMapWaypointsInOperationalPoints.spec.ts

@@ -53,7 +53,7 @@ describe('upsertMapWaypointsInOperationalPoints', () => {
           position: 7746000,
         },
         position: 9246000,
-        weight: null,
+        weight: 100,
       },
       {
         id: 'Mid_West_station',
@@ -112,7 +112,7 @@ describe('upsertMapWaypointsInOperationalPoints', () => {
           position: 6481000,
         },
         position: 0,
-        weight: null,
+        weight: 100,
       },
       {
         id: 'Mid_West_station',
@@ -142,7 +142,7 @@ describe('upsertMapWaypointsInOperationalPoints', () => {
           position: 679000,
         },
         position: 4198000,
-        weight: null,
+        weight: 100,
       },
       {
         id: '3',
@@ -157,7 +157,7 @@ describe('upsertMapWaypointsInOperationalPoints', () => {
           position: 883000,
         },
         position: 4402000,
-        weight: null,
+        weight: 100,
       },
     ]);
   });
@@ -186,7 +186,7 @@ describe('upsertMapWaypointsInOperationalPoints', () => {
           position: 6481000,
         },
         position: 0,
-        weight: null,
+        weight: 100,
       },
       {
         id: '2',
@@ -201,7 +201,7 @@ describe('upsertMapWaypointsInOperationalPoints', () => {
           position: 4733000,
         },
         position: 1748000,
-        weight: null,
+        weight: 100,
       },
     ]);
   });

front/src/applications/operationalStudies/helpers/upsertMapWaypointsInOperationalPoints.ts

@@ -15,38 +15,53 @@ export const upsertMapWaypointsInOperationalPoints = (
   t: TFunction
 ): OperationalPoint[] => {
   let waypointCounter = 1;
+  const HIGHEST_PRIORITY_WEIGHT = 100;
 
   return path.reduce(
     (operationalPointsWithAllWaypoints, step, i) => {
-      if (!('track' in step)) return operationalPointsWithAllWaypoints;
-
-      const positionOnPath = pathItemsPositions[i];
-      const indexToInsert = operationalPointsWithAllWaypoints.findIndex(
-        (op) => op.position >= positionOnPath
-      );
-
-      const formattedStep: OperationalPoint = {
-        id: step.id,
-        extensions: {
-          identifier: {
-            name: t('requestedPoint', { count: waypointCounter }),
-            uic: 0,
-          },
-        },
-        part: { track: step.track, position: step.offset },
-        position: positionOnPath,
-        weight: null,
-      };
-
-      waypointCounter += 1;
-
-      // If we can't find any op position greater than the current step position, we add it at the end
-      if (indexToInsert === -1) {
-        operationalPointsWithAllWaypoints.push(formattedStep);
-      } else {
-        operationalPointsWithAllWaypoints.splice(indexToInsert, 0, formattedStep);
+      if ('uic' in step) {
+        const matchedOP = operationalPoints.find(
+          (op) =>
+            'uic' in step &&
+            'secondary_code' in step &&
+            step.uic === op.extensions?.identifier?.uic &&
+            step.secondary_code === op.extensions?.sncf?.ch
+        );
+        if (matchedOP)
+          operationalPointsWithAllWaypoints.push({ ...matchedOP, weight: HIGHEST_PRIORITY_WEIGHT });
+        return operationalPointsWithAllWaypoints;
       }
 
+      if ('track' in step) {
+        const positionOnPath = pathItemsPositions[i];
+        const indexToInsert = operationalPointsWithAllWaypoints.findIndex(
+          (op) => op.position >= positionOnPath
+        );
+
+        const formattedStep: OperationalPoint = {
+          id: step.id,
+          extensions: {
+            identifier: {
+              name: t('requestedPoint', { count: waypointCounter }),
+              uic: 0,
+            },
+          },
+          part: { track: step.track, position: step.offset },
+          position: positionOnPath,
+          weight: HIGHEST_PRIORITY_WEIGHT,
+        };
+
+        waypointCounter += 1;
+
+        // If we can't find any op position greater than the current step position, we add it at the end
+        if (indexToInsert === -1) {
+          operationalPointsWithAllWaypoints.push(formattedStep);
+        } else {
+          operationalPointsWithAllWaypoints.splice(indexToInsert, 0, formattedStep);
+        }
+
+        return operationalPointsWithAllWaypoints;
+      }
       return operationalPointsWithAllWaypoints;
     },
     [...operationalPoints]

front/src/modules/pathfinding/utils.ts

@@ -179,7 +179,4 @@ export const isVia = (
   { withKP = false } = {}
 ) => vias.some((via) => pathStepMatchesOp(via, op, withKP));
 
-export const isStation = (chCode: string): boolean =>
-  chCode === 'BV' || chCode === '00' || chCode === '';
-
 export const isPathStepInvalid = (step: PathStep | null): boolean => step?.isInvalid || false;

front/src/modules/simulationResult/components/ManchetteWithSpaceTimeChart/ManchetteWithSpaceTimeChart.tsx

@@ -162,6 +162,7 @@ const ManchetteWithSpaceTimeChartWrapper = ({
       position: waypoint.position,
       name: waypoint.extensions?.identifier?.name,
       secondaryCode: waypoint.extensions?.sncf?.ch,
+      weight: waypoint.weight ?? 0,
     }));
   }, [waypointsPanelData, operationalPoints]);
 

front/src/modules/simulationResult/components/SpaceTimeChart/useGetProjectedTrainOperationalPoints.ts

@@ -13,7 +13,6 @@ import {
   type TrainScheduleResult,
 } from 'common/api/osrdEditoastApi';
 import { useOsrdConfSelectors } from 'common/osrdContext';
-import { isStation } from 'modules/pathfinding/utils';
 
 const useGetProjectedTrainOperationalPoints = (
   trainScheduleUsedForProjection?: TrainScheduleResult,
@@ -76,14 +75,8 @@ const useGetProjectedTrainOperationalPoints = (
           operationalPointsWithUniqueIds = JSON.parse(stringifiedSavedWaypoints) as NonNullable<
             PathProperties['operational_points']
           >;
-        } else {
-          // If the manchette hasn't been saved, we want to display by default only
-          // the waypoints with CH BV/00/'' and the ones added by map click
-          operationalPointsWithUniqueIds = operationalPointsWithUniqueIds.filter((op) =>
-            op.extensions?.sncf ? isStation(op.extensions.sncf.ch) : true
-          );
         }
-        setFilteredOperationalPoints(operationalPointsWithUniqueIds);
+        setFilteredOperationalPoints(operationalPointsWithAllWaypoints);
       }
     };
     getOperationalPoints();