X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=user_delete.php;h=38d4f5424bc8f30efe151ad34da43ca62204a2ae;hb=HEAD;hp=b8ec5e9add3fafd9d02eab22684bc414455d1711;hpb=9339284bfdaba5a0c2a4e3c9fa000194a508d0bf;p=timetracker.git diff --git a/user_delete.php b/user_delete.php index b8ec5e9a..38d4f542 100644 --- a/user_delete.php +++ b/user_delete.php @@ -28,22 +28,16 @@ require_once('initialize.php'); import('form.Form'); -import('ttUserHelper'); +import('ttUser'); // Access checks. if (!ttAccessAllowed('manage_users')) { header('Location: access_denied.php'); exit(); } -// Get user id we are deleting. -// A cast to int is for safety against manipulation of request parameter (sql injection). -$user_id = (int) $request->getParameter('id'); -// We need user name and login to display. -$user_details = ttUserHelper::getUserDetails($user_id); -if (!$user_details || // No details. - $user_details['rank'] > $user->rank || // User has a bigger rank. - ($user_details['rank'] == $user->rank && $user_details['id'] <> $user->id) // Same rank but not us. - ) { +$user_id = (int)$request->getParameter('id'); +$user_details = $user->getUserDetails($user_id); +if (!$user_details) { header('Location: access_denied.php'); exit(); } @@ -59,7 +53,7 @@ $form->addInput(array('type'=>'submit','name'=>'btn_cancel','value'=>$i18n->get( if ($request->isPost()) { if ($request->getParameter('btn_delete')) { - if (ttUserHelper::markDeleted($user_id)) { + if ($user->markUserDeleted($user_id)) { // If we deleted the "on behalf" user reset its info in session. if ($user_id == $user->behalf_id) { unset($_SESSION['behalf_id']);