From e23f8db3ff7a533b06e005f39f4c5bcb67ed908a Mon Sep 17 00:00:00 2001
From: "kelly.nguyen" <kelly.nguyen@etu.hesge.ch>
Date: Wed, 21 Aug 2024 21:40:57 +0200
Subject: [PATCH] fixed error message

---
 ExpressAPI/src/routes/UserRoutes.ts | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/ExpressAPI/src/routes/UserRoutes.ts b/ExpressAPI/src/routes/UserRoutes.ts
index 0426ce5..d234d74 100644
--- a/ExpressAPI/src/routes/UserRoutes.ts
+++ b/ExpressAPI/src/routes/UserRoutes.ts
@@ -104,14 +104,14 @@ class UserRoutes implements RoutesManager {
                fs.mkdirSync(folderPath);
           } catch (err) {
                logger.error('Folder creation error');
-               return req.session.sendResponse(res, StatusCodes.OK, false); 
+               return req.session.sendResponse(res, StatusCodes.BAD_REQUEST, false); 
           }
           
           try {
                fs.mkdirSync(pathResume);
           } catch (err) {
                logger.error('Folder creation error');
-               return req.session.sendResponse(res, StatusCodes.OK, false); 
+               return req.session.sendResponse(res, StatusCodes.BAD_REQUEST, false); 
           }
           
           if (dataUser) {
@@ -135,7 +135,7 @@ class UserRoutes implements RoutesManager {
                
                fs.readFile(templatePath, 'utf-8', (err, template) => {
                     if (err) {
-                         res.status(500).send('Error exporting assignment');
+                         res.status(500).send('Error while exporting the assignment');
                          return;
                     }
                     
@@ -181,7 +181,7 @@ class UserRoutes implements RoutesManager {
                     fs.mkdirSync(pathPortfolio + '/exercises');
                } catch (err) {
                     logger.error('Folder creation error');
-                    return req.session.sendResponse(res, StatusCodes.OK, false); 
+                    return req.session.sendResponse(res, StatusCodes.BAD_REQUEST, false); 
                }
                if (dataUser) {
                     dataUser.assignments.forEach(async a => {
@@ -192,7 +192,7 @@ class UserRoutes implements RoutesManager {
                          
                          await fs.writeFile(zipPath, buffer, (err) => {
                               if (err) {
-                                   console.error('Error saving archive:', err);
+                                   console.error('Error while saving the archive:', err);
                               } else {
                                    console.log(`Archive ${zipName} saved successfully!`);
                               }
@@ -207,7 +207,7 @@ class UserRoutes implements RoutesManager {
                          
                          await fs.writeFile(zipPath, buffer, (err) => {
                               if (err) {
-                                   console.error('Error saving archive:', err);
+                                   console.error('Error while saving the archive:', err);
                               } else {
                                    console.log(`Archive ${zipName} saved successfully!`);
                               }
@@ -234,7 +234,7 @@ class UserRoutes implements RoutesManager {
                const zipPath = path.join('/tmp', `${dataUser.gitlabUsername}.zip`);
                const output = fs.createWriteStream(zipPath);
                const archiveZip = archiver('zip', {
-                    zlib: { level: 9 } // Compression maximale
+                    zlib: { level: 9 }
                });
                
                try {
@@ -242,11 +242,11 @@ class UserRoutes implements RoutesManager {
                     archiveZip.directory(folderPath, false);
                     await archiveZip.finalize();
                } catch (error) {
-                    console.error('Error exporting assignment:', error);
-                    res.status(500).send('Error exporting assignment');
+                    console.error('Error while exporting the assignment:', error);
+                    res.status(500).send('Error while exporting the assignment');
                }
                
-               // Attendre la fin de l'écriture dans le fichier zip
+               // Waiting the end of the writing process of the zip file
                output.on('close', function() {
                     const options = {
                          root: path.join('/tmp')
@@ -255,7 +255,7 @@ class UserRoutes implements RoutesManager {
                     const fileName = `${dataUser.gitlabUsername}.zip`;
                     res.sendFile(fileName, options, function (err) {
                          if (err) {
-                              console.error('Error sending file:', err);
+                              console.error('Error while sending the file:', err);
                          } else {
                               console.log('Sent:', fileName);
                               fs.rmSync(folderPath, { recursive: true, force: true });
-- 
GitLab