diff options
| author | Richard McGirr <rmcgirr83@gmail.com> | 2016-01-22 06:03:42 -0500 |
|---|---|---|
| committer | Richard McGirr <rmcgirr83@gmail.com> | 2016-01-22 06:03:42 -0500 |
| commit | b74dc1fa958d9c326068a63ff43290e1bb2065ad (patch) | |
| tree | 633a740141712da63616bd15b4820570312c593b /phpBB/memberlist.php | |
| parent | 91fbdd9ff83c88cc52649964518eca1f1220372f (diff) | |
| download | forums-b74dc1fa958d9c326068a63ff43290e1bb2065ad.tar forums-b74dc1fa958d9c326068a63ff43290e1bb2065ad.tar.gz forums-b74dc1fa958d9c326068a63ff43290e1bb2065ad.tar.bz2 forums-b74dc1fa958d9c326068a63ff43290e1bb2065ad.tar.xz forums-b74dc1fa958d9c326068a63ff43290e1bb2065ad.zip | |
[ticket/14427] Memberlist displays inaccruate information
PHPBB3-14427
Diffstat (limited to 'phpBB/memberlist.php')
| -rw-r--r-- | phpBB/memberlist.php | 32 |
1 files changed, 12 insertions, 20 deletions
diff --git a/phpBB/memberlist.php b/phpBB/memberlist.php index 6be44587f5..7cf39f4eb7 100644 --- a/phpBB/memberlist.php +++ b/phpBB/memberlist.php @@ -1229,21 +1229,19 @@ switch ($mode) ); extract($phpbb_dispatcher->trigger_event('core.memberlist_modify_sql_query_data', compact($vars))); - // Count the users ... - if ($sql_where) - { - $sql = 'SELECT COUNT(u.user_id) AS total_users - FROM ' . USERS_TABLE . " u$sql_from - WHERE u.user_type IN (" . USER_NORMAL . ', ' . USER_FOUNDER . ") - $sql_where"; - $result = $db->sql_query($sql); - $total_users = (int) $db->sql_fetchfield('total_users'); - $db->sql_freeresult($result); - } - else + $user_types = array(USER_NORMAL, USER_FOUNDER); + if ($auth->acl_get('a_user')) { - $total_users = $config['num_users']; + $user_types[] = USER_INACTIVE; } + // Count the users ... + $sql = 'SELECT COUNT(u.user_id) AS total_users + FROM ' . USERS_TABLE . " u$sql_from + WHERE " . $db->sql_in_set('u.user_type', $user_types) . " + $sql_where"; + $result = $db->sql_query($sql); + $total_users = (int) $db->sql_fetchfield('total_users'); + $db->sql_freeresult($result); // Build a relevant pagination_url $params = $sort_params = array(); @@ -1409,13 +1407,7 @@ switch ($mode) ); } - $user_types = array(USER_NORMAL, USER_FOUNDER); - if ($auth->acl_get('a_user')) - { - $user_types[] = USER_INACTIVE; - } - - $start = $pagination->validate_start($start, $config['topics_per_page'], $config['num_users']); + $start = $pagination->validate_start($start, $config['topics_per_page'], $total_users); // Get us some users :D $sql = "SELECT u.user_id |