From 58feda32b75233a489bf5bdb11461b13ff68a4fa Mon Sep 17 00:00:00 2001
From: Clara Ni <clara.ni@outlook.fr>
Date: Thu, 19 Dec 2024 15:06:51 +0100
Subject: [PATCH 1/5] front: support any type of files in importTrainSchedules

Signed-off-by: Clara Ni <clara.ni@outlook.fr>
---
 .../importTrainSchedule.json                  |  4 +-
 .../importTrainSchedule.json                  |  4 +-
 front/src/common/uploadFileModal.tsx          | 58 ++-----------------
 .../ImportTrainScheduleConfig.tsx             | 33 +++++++----
 .../helpers/handleParseFiles.ts               | 56 +++++-------------
 5 files changed, 42 insertions(+), 113 deletions(-)

diff --git a/front/public/locales/en/operationalStudies/importTrainSchedule.json b/front/public/locales/en/operationalStudies/importTrainSchedule.json
index ee826430b13..ce9c24adbbf 100644
--- a/front/public/locales/en/operationalStudies/importTrainSchedule.json
+++ b/front/public/locales/en/operationalStudies/importTrainSchedule.json
@@ -9,13 +9,11 @@
     "error": "An error has occurred",
     "errorEmptyFile": "Empty file",
     "errorImport": "Unable to convert data to TrainSchedule",
-    "errorInvalidJSONFormat": "Invalid JSON format",
-    "errorInvalidXMLFormat": "Invalid XML format",
     "errorNoDate": "You must enter a date.",
     "errorNoFrom": "You must fill in an origin.",
     "errorNoTo": "You must fill in a destination.",
     "errorSameFromTo": "Origin and destination must be different",
-    "errorUnsupportedFileType": "Unsupported file type"
+    "errorInvalidFile": "Invalid file"
   },
   "failure": "Operation failed",
   "from": "Origin",
diff --git a/front/public/locales/fr/operationalStudies/importTrainSchedule.json b/front/public/locales/fr/operationalStudies/importTrainSchedule.json
index 25048276f3b..a53558a1bda 100644
--- a/front/public/locales/fr/operationalStudies/importTrainSchedule.json
+++ b/front/public/locales/fr/operationalStudies/importTrainSchedule.json
@@ -9,13 +9,11 @@
     "error": "Une erreur est survenue",
     "errorEmptyFile": "Fichier vide",
     "errorImport": "Impossible de convertir les données en TrainSchedule",
-    "errorInvalidJSONFormat": "Format JSON invalide",
-    "errorInvalidXMLFormat": "Format XML invalide",
     "errorNoDate": "Vous devez renseigner une date.",
     "errorNoFrom": "Vous devez renseigner une origine.",
     "errorNoTo": "Vous devez renseigner une destination.",
     "errorSameFromTo": "L'origine et la destination doivent être différentes.",
-    "errorUnsupportedFileType": "Type de fichier non supporté"
+    "errorInvalidFile": "Fichier invalide"
   },
   "failure": "Opération échouée",
   "from": "Origine",
diff --git a/front/src/common/uploadFileModal.tsx b/front/src/common/uploadFileModal.tsx
index f465e29186b..0b0e7146da7 100644
--- a/front/src/common/uploadFileModal.tsx
+++ b/front/src/common/uploadFileModal.tsx
@@ -1,4 +1,4 @@
-import { useCallback, useContext, useState } from 'react';
+import { useContext, useState } from 'react';
 
 import { Download } from '@osrd-project/ui-icons';
 import { isNil } from 'lodash';
@@ -16,50 +16,6 @@ const UploadFileModal = ({ handleSubmit }: UploadFileModalProps) => {
   const { t } = useTranslation(['operationalStudies/importTrainSchedule']);
   const { closeModal } = useContext(ModalContext);
   const [selectedFile, setSelectedFile] = useState<File | undefined>(undefined);
-  const [isValid, setIsValid] = useState<string | undefined>(undefined);
-
-  const parseXML = (xmlString: string) => {
-    try {
-      const parser = new DOMParser();
-      const xmlDoc = parser.parseFromString(xmlString, 'application/xml');
-      const parserError = xmlDoc.getElementsByTagName('parsererror');
-      if (parserError.length > 0) {
-        throw new Error('Invalid XML');
-      }
-      return undefined;
-    } catch (error) {
-      return t('errorMessages.errorInvalidXMLFormat').toString();
-    }
-  };
-  // TODO : create the translation keys
-  const validateFile = useCallback(
-    async (fileToValidate: File): Promise<string | undefined> => {
-      if (fileToValidate.size === 0) {
-        return t('errorMessages.errorEmptyFile').toString();
-      }
-      if (fileToValidate.type === 'application/json') {
-        try {
-          JSON.parse(await fileToValidate.text());
-        } catch (e) {
-          return t('errorMessages.errorInvalidJSONFormat').toString();
-        }
-      } else if (
-        fileToValidate.type === 'application/railml' ||
-        fileToValidate.name.endsWith('.railml')
-      ) {
-        const fileContent = await fileToValidate.text();
-        const xmlError = parseXML(fileContent);
-        if (xmlError) {
-          return xmlError;
-        }
-      } else {
-        return t('errorMessages.errorUnsupportedFileType').toString();
-      }
-
-      return undefined;
-    },
-    [t]
-  );
 
   return (
     <>
@@ -73,21 +29,15 @@ const UploadFileModal = ({ handleSubmit }: UploadFileModalProps) => {
           <input
             type="file"
             name="file"
-            accept=".json,.xml,.railml"
+            accept=".json,.txt,.xml,.railml"
             onChange={async (e) => {
               if (e.target.files && e.target.files.length > 0) {
-                const error = await validateFile(e.target.files[0]);
-                setIsValid(error);
-                if (isNil(error)) {
-                  setSelectedFile(e.target.files[0]);
-                }
+                setSelectedFile(e.target.files[0]);
               } else {
                 setSelectedFile(undefined);
-                setIsValid(undefined);
               }
             }}
           />
-          {!isNil(isValid) && <div className="text-danger">{isValid}</div>}
         </>
       </ModalBodySNCF>
       <ModalFooterSNCF>
@@ -105,7 +55,7 @@ const UploadFileModal = ({ handleSubmit }: UploadFileModalProps) => {
             <div className="col-6">
               <button
                 type="button"
-                disabled={isNil(selectedFile) || !isNil(isValid)}
+                disabled={isNil(selectedFile)}
                 className="btn btn-block btn-sm btn-primary"
                 onClick={() => {
                   if (selectedFile) handleSubmit(selectedFile);
diff --git a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
index 6b205d0a8e4..08df7b0bc64 100644
--- a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
+++ b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
@@ -320,22 +320,35 @@ const ImportTrainScheduleConfig = ({
     closeModal();
     setTrainsList([]);
 
-    const fileName = file.name.toLowerCase();
-    const fileExtension = fileName.split('.').pop();
+    let fileContent: string;
+    try {
+      fileContent = await file.text();
+    } catch (error) {
+      handleFileReadingError(error as Error);
+      return;
+    }
 
     try {
-      const fileContent = await file.text();
+      processJsonFile(fileContent, setTrainsJsonData);
+    } catch {
+      // if file was json, display error message immidiately and return
+      if (file.type === 'application/json') {
+        dispatch(
+          setFailure({
+            name: t('errorMessages.error'),
+            message: t('errorMessages.errorInvalidFile'),
+          })
+        );
+        return;
+      }
 
-      if (fileExtension === 'json') {
-        processJsonFile(fileContent, setTrainsJsonData, dispatch);
-      } else if (fileExtension === 'xml' || fileExtension === 'railml') {
-        processXmlFile(fileContent, parseRailML, updateTrainSchedules, dispatch);
-      } else {
+      try {
+        await processXmlFile(fileContent, parseRailML, updateTrainSchedules);
+      } catch {
         handleUnsupportedFileType(dispatch);
       }
-    } catch (error) {
-      handleFileReadingError(error as Error);
     }
+
   };
   return (
     <>
diff --git a/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts b/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
index 08902a97b78..dd3396fe26c 100644
--- a/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
+++ b/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
@@ -9,38 +9,13 @@ export const handleFileReadingError = (error: Error) => {
   console.error('File reading error:', error);
 };
 
-export const handleJsonParsingError = (error: Error, dispatch: Dispatch) => {
-  console.error('Error parsing JSON:', error);
-  dispatch(
-    setFailure({
-      name: t('errorMessages.error'),
-      message: t('errorMessages.errorInvalidJSONFormat'),
-    })
-  );
-};
-
-export const handleXmlParsingError = (error: Error, dispatch: Dispatch) => {
-  console.error('Error parsing XML/RailML:', error);
-  dispatch(
-    setFailure({
-      name: t('errorMessages.error'),
-      message: t('errorMessages.errorInvalidXMLFormat'),
-    })
-  );
-};
-
 export const processJsonFile = (
   fileContent: string,
-  setTrainsJsonData: (data: TrainScheduleBase[]) => void,
-  dispatch: Dispatch
+  setTrainsJsonData: (data: TrainScheduleBase[]) => void
 ) => {
-  try {
-    const importedTrainSchedules: TrainScheduleBase[] = JSON.parse(fileContent);
-    if (importedTrainSchedules && importedTrainSchedules.length > 0) {
-      setTrainsJsonData(importedTrainSchedules);
-    }
-  } catch (error) {
-    handleJsonParsingError(error as Error, dispatch);
+  const importedTrainSchedules: TrainScheduleBase[] = JSON.parse(fileContent);
+  if (importedTrainSchedules && importedTrainSchedules.length > 0) {
+    setTrainsJsonData(importedTrainSchedules);
   }
 };
 
@@ -48,23 +23,18 @@ export const processXmlFile = async (
   fileContent: string,
   parseRailML: (xmlDoc: Document) => Promise<ImportedTrainSchedule[]>,
   updateTrainSchedules: (schedules: ImportedTrainSchedule[]) => void,
-  dispatch: Dispatch
 ) => {
-  try {
-    const parser = new DOMParser();
-    const xmlDoc = parser.parseFromString(fileContent, 'application/xml');
-    const parserError = xmlDoc.getElementsByTagName('parsererror');
+  const parser = new DOMParser();
+  const xmlDoc = parser.parseFromString(fileContent, 'application/xml');
+  const parserError = xmlDoc.getElementsByTagName('parsererror');
 
-    if (parserError.length > 0) {
-      throw new Error('Invalid XML');
-    }
+  if (parserError.length > 0) {
+    throw new Error('Invalid XML');
+  }
 
-    const importedTrainSchedules = await parseRailML(xmlDoc);
-    if (importedTrainSchedules && importedTrainSchedules.length > 0) {
-      updateTrainSchedules(importedTrainSchedules);
-    }
-  } catch (error) {
-    handleXmlParsingError(error as Error, dispatch);
+  const importedTrainSchedules = await parseRailML(xmlDoc);
+  if (importedTrainSchedules && importedTrainSchedules.length > 0) {
+    updateTrainSchedules(importedTrainSchedules);
   }
 };
 

From bce1af377de20ffb6fb2f37b00c58a2663457830 Mon Sep 17 00:00:00 2001
From: Clara Ni <clara.ni@outlook.fr>
Date: Thu, 19 Dec 2024 15:25:42 +0100
Subject: [PATCH 2/5] front: display error if imported json is empty

Signed-off-by: Clara Ni <clara.ni@outlook.fr>
---
 .../ImportTrainScheduleConfig.tsx             | 11 +++++---
 .../helpers/handleParseFiles.ts               | 25 +++++++++----------
 2 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
index 08df7b0bc64..f5a80458ee0 100644
--- a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
+++ b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
@@ -25,7 +25,6 @@ import { formatIsoDate } from 'utils/date';
 
 import {
   handleFileReadingError,
-  handleUnsupportedFileType,
   processJsonFile,
   processXmlFile,
 } from '../ManageTrainSchedule/helpers/handleParseFiles';
@@ -329,7 +328,7 @@ const ImportTrainScheduleConfig = ({
     }
 
     try {
-      processJsonFile(fileContent, setTrainsJsonData);
+      processJsonFile(fileContent, setTrainsJsonData, dispatch, t);
     } catch {
       // if file was json, display error message immidiately and return
       if (file.type === 'application/json') {
@@ -345,10 +344,14 @@ const ImportTrainScheduleConfig = ({
       try {
         await processXmlFile(fileContent, parseRailML, updateTrainSchedules);
       } catch {
-        handleUnsupportedFileType(dispatch);
+        dispatch(
+          setFailure({
+            name: t('errorMessages.error'),
+            message: t('errorMessages.errorUnsupportedFileType'),
+          })
+        );
       }
     }
-
   };
   return (
     <>
diff --git a/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts b/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
index dd3396fe26c..030e2ebe9ba 100644
--- a/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
+++ b/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
@@ -1,4 +1,4 @@
-import { t } from 'i18next';
+import type { TFunction } from 'i18next';
 import type { Dispatch } from 'redux';
 
 import type { ImportedTrainSchedule } from 'applications/operationalStudies/types';
@@ -11,18 +11,27 @@ export const handleFileReadingError = (error: Error) => {
 
 export const processJsonFile = (
   fileContent: string,
-  setTrainsJsonData: (data: TrainScheduleBase[]) => void
+  setTrainsJsonData: (data: TrainScheduleBase[]) => void,
+  dispatch: Dispatch,
+  t: TFunction
 ) => {
   const importedTrainSchedules: TrainScheduleBase[] = JSON.parse(fileContent);
   if (importedTrainSchedules && importedTrainSchedules.length > 0) {
     setTrainsJsonData(importedTrainSchedules);
+  } else {
+    dispatch(
+      setFailure({
+        name: t('errorMessages.error'),
+        message: t('errorMessages.errorEmptyFile'),
+      })
+    );
   }
 };
 
 export const processXmlFile = async (
   fileContent: string,
   parseRailML: (xmlDoc: Document) => Promise<ImportedTrainSchedule[]>,
-  updateTrainSchedules: (schedules: ImportedTrainSchedule[]) => void,
+  updateTrainSchedules: (schedules: ImportedTrainSchedule[]) => void
 ) => {
   const parser = new DOMParser();
   const xmlDoc = parser.parseFromString(fileContent, 'application/xml');
@@ -37,13 +46,3 @@ export const processXmlFile = async (
     updateTrainSchedules(importedTrainSchedules);
   }
 };
-
-export const handleUnsupportedFileType = (dispatch: Dispatch) => {
-  console.error('Unsupported file type');
-  dispatch(
-    setFailure({
-      name: t('errorMessages.error'),
-      message: t('errorMessages.errorUnsupportedFileType'),
-    })
-  );
-};

From 2bf094c9f634249dd7c9ba6605ef02adff771aad Mon Sep 17 00:00:00 2001
From: Clara Ni <clara.ni@outlook.fr>
Date: Thu, 19 Dec 2024 15:29:02 +0100
Subject: [PATCH 3/5] front: improve import error message

Signed-off-by: Clara Ni <clara.ni@outlook.fr>
---
 .../locales/en/operationalStudies/importTrainSchedule.json     | 3 ++-
 .../locales/fr/operationalStudies/importTrainSchedule.json     | 3 ++-
 .../ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx      | 2 +-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/front/public/locales/en/operationalStudies/importTrainSchedule.json b/front/public/locales/en/operationalStudies/importTrainSchedule.json
index ce9c24adbbf..2bf6d745c48 100644
--- a/front/public/locales/en/operationalStudies/importTrainSchedule.json
+++ b/front/public/locales/en/operationalStudies/importTrainSchedule.json
@@ -28,7 +28,8 @@
   "status": {
     "calculatingTrainScheduleCompleteAllSuccess_one": "A train is imported.",
     "calculatingTrainScheduleCompleteAllSuccess": "All trains are imported.",
-    "calculatingTrainScheduleCompleteAllFailure": "All running time calculations have failed."
+    "invalidTrainSchedules_one": "Invalid train schedule.",
+    "invalidTrainSchedules": "Invalid train schedules."
   },
   "success": "Operation successful",
   "to": "Destination",
diff --git a/front/public/locales/fr/operationalStudies/importTrainSchedule.json b/front/public/locales/fr/operationalStudies/importTrainSchedule.json
index a53558a1bda..cffbc1ffb3b 100644
--- a/front/public/locales/fr/operationalStudies/importTrainSchedule.json
+++ b/front/public/locales/fr/operationalStudies/importTrainSchedule.json
@@ -28,7 +28,8 @@
   "status": {
     "calculatingTrainScheduleCompleteAllSuccess_one": "Le train a été importé.",
     "calculatingTrainScheduleCompleteAllSuccess": "Tous les trains ont été importés.",
-    "calculatingTrainScheduleCompleteAllFailure": "Tous les calculs de marche ont échoué."
+    "invalidTrainSchedules_one": "La circulation à importer n'est pas au bon format.",
+    "invalidTrainSchedules": "Les circulations à importer ne sont pas au bon format."
   },
   "success": "Opération réussie",
   "to": "Destination",
diff --git a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx
index 6a2e957f977..65307f11906 100644
--- a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx
+++ b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx
@@ -104,7 +104,7 @@ const ImportTrainScheduleTrainsList = ({
       dispatch(
         setFailure({
           name: t('failure'),
-          message: t('status.calculatingTrainScheduleCompleteAllFailure', {
+          message: t('status.invalidTrainSchedules', {
             trainsList,
             count: trainsList.length,
           }),

From 64fade60994fa126eb83ccd513638fc3e7a19c08 Mon Sep 17 00:00:00 2001
From: Clara Ni <clara.ni@outlook.fr>
Date: Thu, 19 Dec 2024 15:32:22 +0100
Subject: [PATCH 4/5] front: clean and improve import translations

Signed-off-by: Clara Ni <clara.ni@outlook.fr>
---
 .../en/operationalStudies/importTrainSchedule.json        | 6 +++---
 .../fr/operationalStudies/importTrainSchedule.json        | 8 ++++----
 .../ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx | 6 +++---
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/front/public/locales/en/operationalStudies/importTrainSchedule.json b/front/public/locales/en/operationalStudies/importTrainSchedule.json
index 2bf6d745c48..b31e39f13c3 100644
--- a/front/public/locales/en/operationalStudies/importTrainSchedule.json
+++ b/front/public/locales/en/operationalStudies/importTrainSchedule.json
@@ -26,10 +26,10 @@
   "selectStation": "Select this station",
   "startTime": "START",
   "status": {
-    "calculatingTrainScheduleCompleteAllSuccess_one": "A train is imported.",
-    "calculatingTrainScheduleCompleteAllSuccess": "All trains are imported.",
     "invalidTrainSchedules_one": "Invalid train schedule.",
-    "invalidTrainSchedules": "Invalid train schedules."
+    "invalidTrainSchedules": "Invalid train schedules.",
+    "successfulImport_one": "A train is imported.",
+    "successfulImport": "All trains are imported."
   },
   "success": "Operation successful",
   "to": "Destination",
diff --git a/front/public/locales/fr/operationalStudies/importTrainSchedule.json b/front/public/locales/fr/operationalStudies/importTrainSchedule.json
index cffbc1ffb3b..42bcab0ef1c 100644
--- a/front/public/locales/fr/operationalStudies/importTrainSchedule.json
+++ b/front/public/locales/fr/operationalStudies/importTrainSchedule.json
@@ -26,12 +26,12 @@
   "selectStation": "Sélectionner cette station",
   "startTime": "DÉBUT",
   "status": {
-    "calculatingTrainScheduleCompleteAllSuccess_one": "Le train a été importé.",
-    "calculatingTrainScheduleCompleteAllSuccess": "Tous les trains ont été importés.",
     "invalidTrainSchedules_one": "La circulation à importer n'est pas au bon format.",
-    "invalidTrainSchedules": "Les circulations à importer ne sont pas au bon format."
+    "invalidTrainSchedules": "Les circulations à importer ne sont pas au bon format.",
+    "successfulImport_one": "Le train a été importé.",
+    "successfulImport": "Tous les trains ont été importés."
   },
   "success": "Opération réussie",
   "to": "Destination",
-  "trainsFound": "trains trouvé(s)"
+  "trainsFound": "train(s) trouvé(s)"
 }
diff --git a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx
index 65307f11906..4c46c9cae07 100644
--- a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx
+++ b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleTrainsList.tsx
@@ -94,9 +94,9 @@ const ImportTrainScheduleTrainsList = ({
       dispatch(
         setSuccess({
           title: t('success'),
-          text: t('status.calculatingTrainScheduleCompleteAllSuccess', {
+          text: t('status.successfulImport', {
             trainsList,
-            count: trainsList.length,
+            count: trainsList.length || trainsJsonData.length,
           }),
         })
       );
@@ -106,7 +106,7 @@ const ImportTrainScheduleTrainsList = ({
           name: t('failure'),
           message: t('status.invalidTrainSchedules', {
             trainsList,
-            count: trainsList.length,
+            count: trainsList.length || trainsJsonData.length,
           }),
         })
       );

From 3f4ed077d57b629da7f2b934d50a789c9db29092 Mon Sep 17 00:00:00 2001
From: Clara Ni <clara.ni@outlook.fr>
Date: Fri, 20 Dec 2024 17:10:01 +0100
Subject: [PATCH 5/5] front: add validation when importing trains from json
 file

Signed-off-by: Clara Ni <clara.ni@outlook.fr>
---
 .../ImportTrainScheduleConfig.tsx             | 44 ++++++------
 .../helpers/handleParseFiles.ts               | 70 +++++++++++++++++--
 2 files changed, 87 insertions(+), 27 deletions(-)

diff --git a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
index f5a80458ee0..fa3b5267a46 100644
--- a/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
+++ b/front/src/modules/trainschedule/components/ImportTrainSchedule/ImportTrainScheduleConfig.tsx
@@ -327,30 +327,30 @@ const ImportTrainScheduleConfig = ({
       return;
     }
 
+    const fileHasBeenParsed = processJsonFile(
+      fileContent,
+      file.type,
+      setTrainsJsonData,
+      dispatch,
+      t
+    );
+
+    // the file has been processed, return
+    if (fileHasBeenParsed) {
+      return;
+    }
+
+    // try to parse the file as an XML file
     try {
-      processJsonFile(fileContent, setTrainsJsonData, dispatch, t);
+      await processXmlFile(fileContent, parseRailML, updateTrainSchedules);
     } catch {
-      // if file was json, display error message immidiately and return
-      if (file.type === 'application/json') {
-        dispatch(
-          setFailure({
-            name: t('errorMessages.error'),
-            message: t('errorMessages.errorInvalidFile'),
-          })
-        );
-        return;
-      }
-
-      try {
-        await processXmlFile(fileContent, parseRailML, updateTrainSchedules);
-      } catch {
-        dispatch(
-          setFailure({
-            name: t('errorMessages.error'),
-            message: t('errorMessages.errorUnsupportedFileType'),
-          })
-        );
-      }
+      // the file is not supported or is an invalid XML file
+      dispatch(
+        setFailure({
+          name: t('errorMessages.error'),
+          message: t('errorMessages.errorInvalidFile'),
+        })
+      );
     }
   };
   return (
diff --git a/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts b/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
index 030e2ebe9ba..d869e488a14 100644
--- a/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
+++ b/front/src/modules/trainschedule/components/ManageTrainSchedule/helpers/handleParseFiles.ts
@@ -9,23 +9,83 @@ export const handleFileReadingError = (error: Error) => {
   console.error('File reading error:', error);
 };
 
+const TRAIN_SCHEDULE_COMPULSORY_KEYS: (keyof TrainScheduleBase)[] = [
+  'constraint_distribution',
+  'path',
+  'rolling_stock_name',
+  'start_time',
+  'train_name',
+];
+
+const validateTrainSchedules = (
+  importedTrainSchedules: Partial<TrainScheduleBase>[]
+): TrainScheduleBase[] => {
+  const isInvalidTrainSchedules = importedTrainSchedules.some((trainSchedule) => {
+    if (
+      TRAIN_SCHEDULE_COMPULSORY_KEYS.some((key) => !(key in trainSchedule)) ||
+      !Array.isArray(trainSchedule.path)
+    ) {
+      return true;
+    }
+    const hasInvalidSteps = trainSchedule.path.some((step) => !('id' in step));
+    return hasInvalidSteps;
+  });
+
+  if (isInvalidTrainSchedules) {
+    throw new Error('Invalid train schedules: some compulsory keys are missing');
+  }
+  return importedTrainSchedules as TrainScheduleBase[];
+};
+
 export const processJsonFile = (
   fileContent: string,
+  fileExtension: string,
   setTrainsJsonData: (data: TrainScheduleBase[]) => void,
   dispatch: Dispatch,
   t: TFunction
 ) => {
-  const importedTrainSchedules: TrainScheduleBase[] = JSON.parse(fileContent);
-  if (importedTrainSchedules && importedTrainSchedules.length > 0) {
-    setTrainsJsonData(importedTrainSchedules);
-  } else {
+  const isJsonFile = fileExtension === 'application/json';
+
+  // try to parse the file content
+  let rawContent;
+  try {
+    rawContent = JSON.parse(fileContent);
+  } catch {
+    if (isJsonFile) {
+      dispatch(
+        setFailure({
+          name: t('errorMessages.error'),
+          message: t('errorMessages.errorInvalidFile'),
+        })
+      );
+    }
+    return isJsonFile;
+  }
+
+  // validate the trainSchedules
+  try {
+    const importedTrainSchedules = validateTrainSchedules(rawContent);
+    if (importedTrainSchedules.length > 0) {
+      setTrainsJsonData(importedTrainSchedules);
+    } else {
+      dispatch(
+        setFailure({
+          name: t('errorMessages.error'),
+          message: t('errorMessages.errorEmptyFile'),
+        })
+      );
+    }
+  } catch {
     dispatch(
       setFailure({
         name: t('errorMessages.error'),
-        message: t('errorMessages.errorEmptyFile'),
+        message: t('errorMessages.errorInvalidFile'),
       })
     );
   }
+
+  // file has been parsed successfully
+  return true;
 };
 
 export const processXmlFile = async (