From fdad47e84ec66f251da3a03657c385d849d38e4f Mon Sep 17 00:00:00 2001 From: Alexis <35.alexis.pele@gmail.com> Date: Wed, 23 Dec 2020 18:10:02 +0100 Subject: [PATCH] - Fixed an error with 200 codes and translated API errors --- api/repositories/user-repository.js | 32 ++++++++++++++--------------- 1 file changed, 15 insertions(+), 17 deletions(-) diff --git a/api/repositories/user-repository.js b/api/repositories/user-repository.js index a8cbe55..bbdeb0f 100644 --- a/api/repositories/user-repository.js +++ b/api/repositories/user-repository.js @@ -31,7 +31,7 @@ class UserRepository { }) .catch(() => { reject({ - "message": "Database error, couldn't get all users.", + "message": "Erreur de base de données, les utilisateurs n'ont pas pu être récupérés.", "code": 500 }) }) @@ -48,7 +48,7 @@ class UserRepository { }) .catch(err => { reject({ - "message": "User with this UUID was not found.", + "message": "L'utilisateur avec cet UUID n'a pas été trouvé.", "code": 404 }) }) @@ -65,7 +65,7 @@ class UserRepository { }) .catch(() => { reject({ - "message": "User with this email was not found.", + "message": "L'utilisateur avec cet email n'a pas été trouvé.", "code": 404 }) }) @@ -77,17 +77,17 @@ class UserRepository { // Checks if body exists and if the model fits, and throws errors if it doesn't if (isEmptyObject(u)) { reject({ - "message": "Request body cannot be empty.", + "message": "Le corps de requête ne peut être vide.", "code": 403 }) } else if (!v.validate(u, UserValidation).valid) { reject({ - "message": "Schema is not valid - " + v.validate(u, UserValidation).errors, + "message": "Structure de la requête invalide - " + v.validate(u, UserValidation).errors, "code": 403 }) } else if (isXSSAttempt(u.name) || isXSSAttempt(u.password) || isXSSAttempt(u.mail)) { reject({ - "message": "Injection attempt detected, aborting the request.", + "message": "Essai d'injection détecté, avortement de la requête.", "code": 403 }) } else { @@ -115,14 +115,14 @@ class UserRepository { }) .then(newUser => { resolve({ - "message": "Account successfully created !", + "message": `Compte utilisateur #${newUser.id} créé avec succès.`, "code": 201, "user": newUser, }) }) .catch(err => { resolve({ - "message": "An error has occured while creating your account.", + "message": "Une erreur s'est produite en créant votre compte. Veuillez réessayer ultérieurement ou contactez l'administrateur.", "code": 500, }) }) @@ -146,13 +146,13 @@ class UserRepository { if (match) { resolve({ - "message": "User successfully logged in !", + "message": `L'utilisateur #${fetchedUser.id} s'est connecté.`, "code": 200, "user": fetchedUser, }) } else { reject({ - "message": "Authentification attempt failed ; credentials are incorrect.", + "message": "Les informations de connexions sont erronées.", "code": 400, }) } @@ -169,23 +169,21 @@ class UserRepository { if (!this.validateEmail(mail)) { reject({ - "message": "The request isn't a correctly formed email.", + "message": "La requête n'est pas un email valide.", "code": 400, }) } this.getOneByEmail(mail, false) .then(() => { - resolve({ - "message": "This email is already linked to an account.", - "available": false, - "code": 200 + reject({ + "message": "Cet email est déjà lié à un compte.", + "code": 409 }) }) .catch(() => { resolve({ - "message": "This email is available.", - "available": true, + "message": "Cet email est disponible.", "code": 200 }) })