From 6caedd35636b8157df3000d19b55962fc3b252a0 Mon Sep 17 00:00:00 2001 From: xdrm-brackets Date: Tue, 16 Feb 2016 14:51:19 +0100 Subject: [PATCH] - [x] [users.js][userDefault][userRepo] Gestion de la nouvelle modification (avec recherche) --- manager/module/userDefault.php | 24 ++++++++------------ manager/repo/user.php | 2 -- todo.md | 7 +++--- view/js/users.js | 41 +++++++++++++++++++--------------- 4 files changed, 36 insertions(+), 38 deletions(-) diff --git a/manager/module/userDefault.php b/manager/module/userDefault.php index 3891008..9b04027 100755 --- a/manager/module/userDefault.php +++ b/manager/module/userDefault.php @@ -297,6 +297,7 @@ /* MODIFIE UN UTILISATEUR DONNEE * + * @id_user UID de l'utilisateur * @code Code RFID de l'utilisateur * @username Identifiant l'utilisateur * @firstname Prenom de l'utilisateur @@ -308,12 +309,13 @@ * @return status Retourne si oui ou non tout s'est bien deroule * */ - public static function edit($code=null, $username=null, $firstname=null, $lastname=null, $mail=null, $password=null, $status=null){ + public static function edit($id_user=null, $code=null, $username=null, $firstname=null, $lastname=null, $mail=null, $password=null, $status=null){ /* [1] Normalisation + verification des donnees =========================================================*/ $password_hash = sessionManager::secure_sha1($password); - $correct_param = Database::check('user.code', $code); + $correct_param = Database::check('auto_increment_id', $id_user); + $correct_param = $correct_param && Database::check('user.code', $code); $correct_param = $correct_param && Database::check('user.username', $username); $correct_param = $correct_param && Database::check('user.firstname', $firstname); $correct_param = $correct_param && Database::check('user.lastname', $lastname); @@ -326,27 +328,19 @@ return array('ModuleError' => ManagerError::ParamError); - /* [2] Recuperation de l'UID de l'utilisateur + /* [2] On verifie l'existence de l'utilisateur =========================================================*/ - $getByCode = new Repo('user/getByCode', array($code)); - $getByUsername = new Repo('user/getByUsername', array($username)); - - // On recupere les resultats - $uidbycode = $getByCode->answer(); - $uidbyusername = $getByUsername->answer(); - - // On recupere l'utilisateur (si recupere avec code OU username) - $user = (is_array($uidbyusername)) ? $uidbyusername : null; - $user = (is_array($uidbycode)) ? $uidbycode : $user; + $user_exists = new Repo('user/getById', array($id_user)); + $user_exists = $user_exists->answer(); // Si on a recupere aucun utilisateur, on retourne une erreur - if( !is_array($user) ) + if( !is_array($user_exists) ) return array('ModuleError' => ManagerError::ModuleError); /* [3] Modification de l'utilisateur =========================================================*/ - $request = new Repo('user/edit', array($user['id_user'], $code, $username, $firstname, $lastname, $mail, $password_hash, $status)); + $request = new Repo('user/edit', array($id_user, $code, $username, $firstname, $lastname, $mail, $password_hash, $status)); $answer = $request->answer(); return array( diff --git a/manager/repo/user.php b/manager/repo/user.php index 7820dc8..16c320e 100755 --- a/manager/repo/user.php +++ b/manager/repo/user.php @@ -340,8 +340,6 @@ $checker = $check_user['id_user'] == $id_user; $checker = $checker && $check_user['code'] == $code; $checker = $checker && $check_user['username'] == $username; - var_dump($check_user['username']); - var_dump($username); $checker = $checker && $check_user['firstname'] == $firstname; $checker = $checker && $check_user['lastname'] == $lastname; $checker = $checker && $check_user['mail'] == $mail; diff --git a/todo.md b/todo.md index c0a4718..ec31d77 100755 --- a/todo.md +++ b/todo.md @@ -7,18 +7,19 @@ ############ # EN COURS # ############ -- [ ] [users.js][userDefault][userRepo] Gestion de la nouvelle modification (avec recherche) + +- [ ] [machineDefault] Implementer la modification - [ ] [view/js/machines] Recherche instantanee -- [ ] [view/js/users] Recherche instantanee +- [ ] [view/js/users] Recherche instantanee ######## # FAIT # ######## +- [x] [users.js][userDefault][userRepo] Gestion de la nouvelle modification (avec recherche) - [x] [view/js/machines] Modification de machine -- [x] [machineDefault] Implementer la modification - [x] [userDefault] Implementer la modification - [x] [userRepo] Gestion des parametres optionnels - [x] [view/users] Gestion des parametres optionnels physiques -> separer la selection et modification diff --git a/view/js/users.js b/view/js/users.js index 5911a7c..12ab599 100644 --- a/view/js/users.js +++ b/view/js/users.js @@ -342,26 +342,31 @@ if( section.edit.element != null ){ // On annule l'envoi de base (PHP) e.preventDefault(); - var request = { - path: 'userDefault/edit', // On veut creer un utilisateur - data: [ - section.edit.input.code.value, - section.edit.input.username.value, - section.edit.input.firstname.value, - section.edit.input.lastname.value, - section.edit.input.mail.value, - section.edit.input.password.value, - section.edit.input.status.value - ] - }; + // Si l'id est defini + if( section.edit.input.search.id.value != '' ){ - api.send(request, function(answer){ - if( answer.ModuleError == 0 && answer.status == true ) // Tout s'est bien deroule - console.log('Utilisateur modifie!'); - else // Erreur - console.error('ModuleError::'+answer.ModuleError); + var request = { + path: 'userDefault/edit', // On veut creer un utilisateur + data: [ + section.edit.input.search.id.value, + section.edit.input.code.value, + section.edit.input.username.value, + section.edit.input.firstname.value, + section.edit.input.lastname.value, + section.edit.input.mail.value, + section.edit.input.password.value, + section.edit.input.status.value + ] + }; - }); + api.send(request, function(answer){ + if( answer.ModuleError == 0 && answer.status == true ) // Tout s'est bien deroule + console.log('Utilisateur modifie!'); + else // Erreur + console.error('ModuleError::'+answer.ModuleError); + }); + + } }, false);