FIX #393 Error when adding or modifying a user

pluxml · Apr 26, 2020 · 139d58a · 139d58a
1 parent cb24fb0
commit 139d58a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/core/lib/class.plx.admin.php b/core/lib/class.plx.admin.php
@@ -460,25 +460,25 @@ public function editUsers($content, $action=false) {
 
 			foreach($this->aUsers as $user_id => $user) {
 				# controle de l'unicité du nom de l'utilisateur
-				if(in_array($user['name'], $users_name)) {
+			    if(in_array($user['name'], $users_name)) {
 					$this->aUsers = $save;
 					return plxMsg::Error(L_ERR_USERNAME_ALREADY_EXISTS.' : '.plxUtils::strCheck($user['name']));
 				}
-				else {
+				else if ($user['delete'] == 0) {
 					$users_name[] = $user['name'];
 				}
 				# controle de l'unicité du login de l'utilisateur
 				if(in_array($user['login'], $users_login)) {
 					return plxMsg::Error(L_ERR_LOGIN_ALREADY_EXISTS.' : '.plxUtils::strCheck($user['login']));
 				}
-				else {
+				else if ($user['delete'] == 0) {
 					$users_login[] = $user['login'];
 				}
 				# controle de l'unicité de l'adresse e-mail
 				if(in_array($user['email'], $users_email)) {
 					return plxMsg::Error(L_ERR_EMAIL_ALREADY_EXISTS.' : '.plxUtils::strCheck($user['email']));
 				}
-				else {
+				else if ($user['delete'] == 0) {
 					$users_email[] = $user['email'];
 				}
 				$xml .= "\t".'<user number="'.$user_id.'" active="'.$user['active'].'" profil="'.$user['profil'].'" delete="'.$user['delete'].'">'."\n";

diff --git a/readme/CHANGELOG b/readme/CHANGELOG
@@ -1,6 +1,7 @@
 ## PLUXML 5.8.3 (yyyy/mm/dd) ##
 [+] New "rename" and "copy to clipboard" icons in medias administration page #387 (bazooka07)
 FIX Headline and Tags toggles always closed in article edition #382 (sudwebdesign)
+FIX Error when adding or modifying a user #393 (P3ter)
 
 ## PLUXML 5.8.2 (2020/02/09) ##
 [+] #371 Check PluXml update using javascript if "curl" or "file_get_contents" can not be used (bazooka07)