front: use ocpType to correctly handle stop times

Author: RomainValls

front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx

@@ -173,8 +173,13 @@ const ImportTrainScheduleConfig = ({
       .map((ocpTT): Step | null => {
         const ocpRef = ocpTT.getAttribute('ocpRef');
         const times = ocpTT.getElementsByTagName('times')[0];
+        const isLastOcp = ocpTT === ocpTTs[ocpTTs.length - 1];
+        const ocpType = ocpTT.getAttribute('ocpType');
         let departureTime = times?.getAttribute('departure') || '';
-        let arrivalTime = times?.getAttribute('arrival') || '';
+        let arrivalTime =
+          ocpType === 'pass'
+            ? times?.getAttribute('departure') || ''
+            : times?.getAttribute('arrival') || '';
         arrivalTime = cleanTimeFormat(arrivalTime);
         departureTime = cleanTimeFormat(departureTime);
 
@@ -195,10 +200,29 @@ const ImportTrainScheduleConfig = ({
         const formattedDepartureTime = `${startDate} ${departureTime}`;
 
         let stopFor: number | undefined;
-        if (arrivalTime && departureTime) {
-          const arrivalDate = new Date(`${startDate}T${arrivalTime}`);
-          const departureDate = new Date(`${startDate}T${departureTime}`);
-          stopFor = Math.round((departureDate.getTime() - arrivalDate.getTime()) / 1000);
+
+        const arrivalDate = new Date(`${startDate}T${arrivalTime}`);
+        const departureDate = new Date(`${startDate}T${departureTime}`);
+        if (ocpType === 'stop') {
+          if (arrivalTime && departureTime) {
+            if (arrivalTime === departureTime) {
+              stopFor = 0;
+            } else {
+              stopFor = Math.round((departureDate.getTime() - arrivalDate.getTime()) / 1000);
+            }
+          } else if (departureTime) {
+            stopFor = 0;
+          }
+        } else if (ocpType === 'pass') {
+          if (arrivalTime && departureTime && arrivalTime !== departureTime) {
+            // Incorrect data, it shouldn't happen on a ocpType pass, skip step
+            return null;
+          }
+          if (isLastOcp && departureTime) {
+            stopFor = 0;
+          } else {
+            stopFor = undefined;
+          }
         }
 
         return {