diff --git a/admin/controllers/add-user.php b/admin/controllers/add-user.php index 9ec0e558..b6c6859e 100644 --- a/admin/controllers/add-user.php +++ b/admin/controllers/add-user.php @@ -19,16 +19,22 @@ function addUser($args) global $Language; // Check if the username already exist in db. - if( $dbUsers->userExists($args['username']) || Text::isEmpty($args['username']) ) + if( Text::isEmpty($args['username']) ) { - Alert::set($Language->g('username-already-exists-or-is-empty')); + Alert::set($Language->g('username-field-is-empty')); + return false; + } + + if( $dbUsers->userExists($args['username']) ) + { + Alert::set($Language->g('username-already-exists')); return false; } // Validate password. if( ($args['password'] != $args['confirm-password'] ) || Text::isEmpty($args['password']) ) { - Alert::set($Language->g('password-does-not-match-the-confirm-password')); + Alert::set($Language->g('The password and confirmation password do not match')); return false; } @@ -40,7 +46,7 @@ function addUser($args) } else { - Alert::set($Language->g('an-error-occurred-while-trying-to-create-the-user-account')); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to create the account.'); return false; } } diff --git a/admin/controllers/edit-page.php b/admin/controllers/edit-page.php index 8c4ad2a2..73684983 100644 --- a/admin/controllers/edit-page.php +++ b/admin/controllers/edit-page.php @@ -26,12 +26,12 @@ function editPage($args) { $dbPages->regenerate(); - Alert::set($Language->g('the-changes-have-been-saved')); + Alert::set($Language->g('The changes have been saved')); Redirect::page('admin', 'edit-page/'.$args['key']); } else { - Alert::set($Language->g('an-error-occurred-while-trying-to-edit-the-page')); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to edit the page.'); } } @@ -42,12 +42,12 @@ function deletePage($key) if( $dbPages->delete($key) ) { - Alert::set('The page has been deleted successfully'); + Alert::set($Language->g('The page has been deleted successfully')); Redirect::page('admin', 'manage-pages'); } else { - Alert::set('an-error-occurred-while-trying-to-delete-the-page'); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to delete the page.'); } } diff --git a/admin/controllers/edit-post.php b/admin/controllers/edit-post.php index fb565644..a2473f24 100644 --- a/admin/controllers/edit-post.php +++ b/admin/controllers/edit-post.php @@ -20,27 +20,28 @@ function editPost($args) // Edit the post. if( $dbPosts->edit($args) ) { - Alert::set($Language->g('the-changes-have-been-saved')); + Alert::set($Language->g('The changes have been saved')); Redirect::page('admin', 'edit-post/'.$args['key']); } else { - Alert::set($Language->g('an-error-occurred-while-trying-to-edit-the-post')); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to edit the post.'); } } function deletePost($key) { global $dbPosts; + global $Language; if( $dbPosts->delete($key) ) { - Alert::set('The post has been deleted successfull'); + Alert::set($Language->g('The post has been deleted successfully')); Redirect::page('admin', 'manage-posts'); } else { - Alert::set('an-error-occurred-while-trying-to-delete-the-post'); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to delete the post.'); } } diff --git a/admin/controllers/edit-user.php b/admin/controllers/edit-user.php index f7c39687..3dc6b563 100644 --- a/admin/controllers/edit-user.php +++ b/admin/controllers/edit-user.php @@ -9,19 +9,52 @@ function editUser($args) global $dbUsers; global $Language; - if(isset($args['password'])) + if( $dbUsers->set($args) ) { + Alert::set($Language->g('The changes have been saved')); + } + else { + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to edit the user.'); + } +} + +function setPassword($args) +{ + global $dbUsers; + global $Language; + + if( ($args['password']===$args['confirm-password']) && !Text::isEmpty($args['password']) ) { - if( ($args['password']===$args['confirm-password']) && !Text::isEmpty($args['password']) ) { - return $dbUsers->setPassword($args); + if( $dbUsers->setPassword($args) ) { + Alert::set($Language->g('The changes have been saved')); } else { - Alert::set($Language->g('password-does-not-match-the-confirm-password')); - return false; + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to change the user password.'); } } - else - { - return $dbUsers->set($args); + else { + Alert::set($Language->g('The password and confirmation password do not match')); + return false; + } +} + +function deleteUser($args, $deleteContent=false) +{ + global $dbUsers; + global $dbPosts; + global $Language; + + if($deleteContent) { + $dbPosts->deletePostsByUser($args['username']); + } + else { + $dbPosts->linkPostsToUser($args['username'], 'admin'); + } + + if( $dbUsers->delete($args['username']) ) { + Alert::set($Language->g('User deleted')); + } + else { + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to delete the user.'); } } @@ -31,17 +64,25 @@ function editUser($args) if( $_SERVER['REQUEST_METHOD'] == 'POST' ) { - + // Prevent editors users to administrate other users. if($Login->role()!=='admin') { $_POST['username'] = $Login->username(); unset($_POST['role']); } - if( editUser($_POST) ) { - Alert::set($Language->g('the-changes-have-been-saved')); + if(isset($_POST['delete-user-all'])) { + deleteUser($_POST, true); + } + elseif(isset($_POST['delete-user-associate'])) { + deleteUser($_POST, false); + } + elseif(isset($_POST['change-password'])) { + setPassword($_POST); + } + elseif(isset($_POST['edit-user'])) { + editUser($_POST); } - } // ============================================================================ diff --git a/admin/controllers/login.php b/admin/controllers/login.php index bbde272e..cb373aec 100644 --- a/admin/controllers/login.php +++ b/admin/controllers/login.php @@ -11,6 +11,6 @@ if( $_SERVER['REQUEST_METHOD'] == 'POST' ) } else { - Alert::set('Username or password incorrect'); + Alert::set($Language->g('Username or password incorrect')); } } \ No newline at end of file diff --git a/admin/controllers/logout.php b/admin/controllers/logout.php index 53702e42..fcc3df16 100644 --- a/admin/controllers/logout.php +++ b/admin/controllers/logout.php @@ -3,4 +3,4 @@ if( $Login->logout()) { Redirect::home(); -} +} \ No newline at end of file diff --git a/admin/controllers/new-page.php b/admin/controllers/new-page.php index 46c77507..a44599e7 100644 --- a/admin/controllers/new-page.php +++ b/admin/controllers/new-page.php @@ -20,12 +20,12 @@ function addPage($args) // Add the page. if( $dbPages->add($args) ) { - Alert::set('Page added successfuly'); + Alert::set($Language->g('Page added successfully')); Redirect::page('admin', 'manage-pages'); } else { - Alert::set($Language->g('an-error-occurred-while-trying-to-create-the-page')); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to create the page.'); } } diff --git a/admin/controllers/new-post.php b/admin/controllers/new-post.php index b69ca85c..b1729ca4 100644 --- a/admin/controllers/new-post.php +++ b/admin/controllers/new-post.php @@ -20,12 +20,12 @@ function addPost($args) // Add the page. if( $dbPosts->add($args) ) { - Alert::set('Post added successfuly'); + Alert::set($Language->g('Post added successfully')); Redirect::page('admin', 'manage-posts'); } else { - Alert::set($Language->g('an-error-occurred-while-trying-to-create-the-post')); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to create the post.'); } } diff --git a/admin/controllers/settings.php b/admin/controllers/settings.php index 455b4136..2610e8a4 100644 --- a/admin/controllers/settings.php +++ b/admin/controllers/settings.php @@ -35,7 +35,7 @@ function setSettings($args) Alert::set($Language->g('the-changes-have-been-saved')); } else { - Alert::set('Error occurred when trying to saved the settings'); + Log::set(__METHOD__.LOG_SEP.'Error occurred when trying to save the settings.'); } } diff --git a/admin/views/edit-user.php b/admin/views/edit-user.php index fb3d305f..99511c7b 100644 --- a/admin/views/edit-user.php +++ b/admin/views/edit-user.php @@ -5,6 +5,7 @@
  • p('Profile') ?>
  • p('Email') ?>
  • p('Password') ?>
  • +
  • p('Delete') ?>
  • @@ -14,6 +15,7 @@
    +