X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=mobile%2Fuser_delete.php;h=e44483ecc9e4b6653cc13fcf92bda53fa1bad2dd;hb=42fb0e1b78c91f0aacbc3d9d707181445e33caf9;hp=0e045d018cce25297bea52e67cb449b5332c5e70;hpb=9339284bfdaba5a0c2a4e3c9fa000194a508d0bf;p=timetracker.git

diff --git a/mobile/user_delete.php b/mobile/user_delete.php
index 0e045d01..e44483ec 100644
--- a/mobile/user_delete.php
+++ b/mobile/user_delete.php
@@ -28,29 +28,20 @@
 
 require_once('../initialize.php');
 import('form.Form');
-import('ttUserHelper');
+import('ttUser');
 
-// Access check.
+// Access checks.
 if (!ttAccessAllowed('manage_users')) {
   header('Location: access_denied.php');
   exit();
 }
-
-// Get user id we are deleting from the request.
-// 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);
-
-// Security checks.
-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();
 }
+// End of access checks.
 
 $smarty->assign('user_to_delete', $user_details['name']." (".$user_details['login'].")");
 
@@ -62,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']);