From 5a5682a0f61ee147143e09ce830447f265659179 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Tempez?= <frederic.tempez@outlook.com>
Date: Thu, 16 Jan 2025 10:50:06 +0100
Subject: [PATCH]  ajout d'un utilisateur formulaire avec erreur sans envoi
 d'email

---
 core/module/user/user.php | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/core/module/user/user.php b/core/module/user/user.php
index 110c8353..325db6f6 100644
--- a/core/module/user/user.php
+++ b/core/module/user/user.php
@@ -90,22 +90,23 @@ class user extends common
 			$this->getUser('permission', __CLASS__, __FUNCTION__) === true &&
 			$this->isPost()
 		) {
-			$check = true;
 			// L'identifiant d'utilisateur est indisponible
 			$userId = $this->getInput('userAddId', helper::FILTER_ID, true);
 			if ($this->getData(['user', $userId])) {
 				self::$inputNotices['userAddId'] = 'Identifiant déjà utilisé';
-				$check = false;
 			}
 			// Double vérification pour le mot de passe
 			if ($this->getInput('userAddPassword', helper::FILTER_STRING_SHORT, true) !== $this->getInput('userAddConfirmPassword', helper::FILTER_STRING_SHORT, true)) {
-				self::$inputNotices['userAddConfirmPassword'] = 'Incorrect';
-				$check = false;
+				self::$inputNotices['userAddConfirmPassword'] = 'Les mots de passe ne sont pas indentiques';
 			}
+
 			// Crée l'utilisateur
 			$userFirstname = $this->getInput('userAddFirstname', helper::FILTER_STRING_SHORT, true);
 			$userLastname = $this->getInput('userAddLastname', helper::FILTER_STRING_SHORT, true);
 			$userMail = $this->getInput('userAddMail', helper::FILTER_MAIL, true);
+			$pseudo = $this->getInput('userAddPseudo', helper::FILTER_STRING_SHORT, true);
+			$signature = $this->getInput('userAddSignature', helper::FILTER_INT, true);
+			$password = $this->getInput('userAddPassword', helper::FILTER_PASSWORD, true);
 
 			// Profil
 			$group = $this->getInput('userAddGroup', helper::FILTER_INT, true);
@@ -124,10 +125,10 @@ class user extends common
 					'group' => $group,
 					'profil' => $profil,
 					'lastname' => $userLastname,
-					'pseudo' => $this->getInput('userAddPseudo', helper::FILTER_STRING_SHORT, true),
-					'signature' => $this->getInput('userAddSignature', helper::FILTER_INT, true),
+					'pseudo' => $pseudo,
+					'signature' => $signature,
 					'mail' => $userMail,
-					'password' => $this->getInput('userAddPassword', helper::FILTER_PASSWORD, true),
+					'password' => $password,
 					'connectFail' => null,
 					'connectTimeout' => null,
 					'accessUrl' => null,
@@ -140,7 +141,10 @@ class user extends common
 
 			// Envoie le mail
 			$sent = true;
-			if ($this->getInput('userAddSendMail', helper::FILTER_BOOLEAN) && $check === true) {
+			if (
+				$this->getInput('userAddSendMail', helper::FILTER_BOOLEAN) &&
+				self::$inputNotices === []
+			) {
 				$sent = $this->sendMail(
 					$userMail,
 					'Compte créé sur ' . $this->getData(['config', 'title']),