X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=user_delete.php;h=647f21728b8df17c03be161edcb981e8f2bf229a;hb=d5f3d27d84a808f8333822de5662b2cd3a2e322c;hp=b8ec5e9add3fafd9d02eab22684bc414455d1711;hpb=9339284bfdaba5a0c2a4e3c9fa000194a508d0bf;p=timetracker.git diff --git a/user_delete.php b/user_delete.php index b8ec5e9a..647f2172 100644 --- a/user_delete.php +++ b/user_delete.php @@ -35,15 +35,9 @@ 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->getUser($user_id); +if (!$user_details) { header('Location: access_denied.php'); exit(); }