diff --git a/phpBB/adm/admin_prune_users.php b/phpBB/adm/admin_prune_users.php index 1681738aca..4c045c7bea 100644 --- a/phpBB/adm/admin_prune_users.php +++ b/phpBB/adm/admin_prune_users.php @@ -44,7 +44,7 @@ if (!$auth->acl_get('a_userdel')) } // Set mode -$mode = (isset($_REQUEST['mode'])) ? $_REQUEST['mode'] : ''; +$mode = (isset($_REQUEST['mode'])) ? htmlspecialchars($_REQUEST['mode']) : ''; // Do prune if (isset($_POST['prune'])) @@ -112,8 +112,8 @@ if (isset($_POST['prune'])) $where_sql .= ($active) ? " AND user_lastvisit " . $key_match[$active_select] . " " . gmmktime(0, 0, 0, $active[1], intval($active[2]), intval($active[0])) : ''; } - $sql = "SELECT username, user_id FROM " . USERS_TABLE . " - WHERE user_id <> " . ANONYMOUS . " + $sql = 'SELECT username, user_id FROM ' . USERS_TABLE . ' + WHERE user_id <> ' . ANONYMOUS . " $where_sql"; $result = $db->sql_query($sql); @@ -141,51 +141,50 @@ if (isset($_POST['prune'])) { if (!empty($_POST['deleteposts'])) { - $l_admin_log = 'log_prune_user_del_del'; - - // // Call unified post deletion routine? - // + + $l_log = 'LOG_PRUNE_USER_DEL_DEL'; } else { - $l_admin_log = 'log_prune_user_del_anon'; - for($i = 0; $i < sizeof($user_ids); $i++) { - $sql = "UPDATE " . POSTS_TABLE . " - SET poster_id = " . ANONYMOUS . ", post_username = '" . $usernames[$i] . "' + $sql = 'UPDATE ' . POSTS_TABLE . ' + SET poster_id = ' . ANONYMOUS . ", post_username = '" . $usernames[$i] . "' WHERE user_id = " . $userids[$i]; // $db->sql_query($sql); } + + $l_log = 'LOG_PRUNE_USER_DEL_ANON'; } - $sql = "DELETE FROM " . USERS_TABLE; + $sql = 'DELETE FROM ' . USERS_TABLE; } else if (!empty($_POST['deactivate'])) { - $l_admin_log = 'log_prune_user_deac'; + $sql = 'UPDATE ' . USERS_TABLE . " + SET user_active = 0"; - $sql = "UPDATE " . USERS_TABLE . " SET user_active = 0"; + $l_log = 'LOG_PRUNE_USER_DEAC'; } - $sql .= " WHERE user_id <> " . ANONYMOUS . " + + $sql .= ' WHERE user_id <> ' . ANONYMOUS . " $where_sql"; // $db->sql_query($sql); - add_log('admin', $l_admin_log, implode(', ', $usernames)); + add_log('admin', $l_log, implode(', ', $usernames)); unset($user_ids); unset($usernames); } - message_die(MESSAGE, $user->lang['Success_user_prune']); + trigger_error($user->lang['SUCCESS_USER_PRUNE']); } } -// -// -// -$find_count = array('lt' => $user->lang['Less_than'], 'eq' => $user->lang['Equal_to'], 'gt' => $user->lang['More_than']); + +// Front end +$find_count = array('lt' => $user->lang['LESS_THAN'], 'eq' => $user->lang['EQUAL_TO'], 'gt' => $user->lang['MORE_THAN']); $s_find_count = ''; foreach ($find_count as $key => $value) { @@ -193,7 +192,7 @@ foreach ($find_count as $key => $value) $s_find_count .= ''; } -$find_time = array('lt' => $user->lang['Before'], 'gt' => $user->lang['After']); +$find_time = array('lt' => $user->lang['BEFORE'], 'gt' => $user->lang['AFTER']); $s_find_join_time = ''; foreach ($find_time as $key => $value) { @@ -208,48 +207,52 @@ foreach ($find_time as $key => $value) // // // -adm_page_header($user->lang['Prune_users']); +adm_page_header($user->lang['PRUNE_USERS']); ?> -
lang['Prune_users_explain']; ?>
+lang['PRUNE_USERS_EXPLAIN']; ?>
-