Drupal
/
drupal
mirror of https://git.drupalcode.org/project/drupal.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

#604304 by JeremyFrench: Fixed bad query and broken paging on admin/people.

merge-requests/26/head
Angie Byron 2010-01-12 06:18:58 +00:00
parent e1812b3353
commit 100ec14f14
2 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
modules/user/user.admin.inc
View File

@ -151,12 +151,11 @@ function user_admin_account() {
);
$query = db_select('users', 'u');
$query->leftJoin('users_roles', 'ur', 'u.uid = ur.uid');
$query->condition('u.uid', 0, '<>');
user_build_filter_query($query);
$count_query = clone $query;
$count_query->addExpression('COUNT(DISTINCT u.uid)');
$count_query->addExpression('COUNT(u.uid)');
$query = $query->extend('PagerDefault')->extend('TableSort');
$query

18
modules/user/user.module
View File

@ -2829,7 +2829,8 @@ function user_filters() {
*/
function user_build_filter_query(SelectQuery $query) {
$filters = user_filters();
$role_count = 0;
$permission_count = 0;
// Extend Query with filter conditions.
foreach (isset($_SESSION['user_overview_filter']) ? $_SESSION['user_overview_filter'] : array() as $filter) {
list($key, $value) = $filter;
@ -2843,8 +2844,19 @@ function user_build_filter_query(SelectQuery $query) {
if (user_access($value, $account)) {
continue;
}
$query->leftJoin('role_permission', 'p', 'ur.rid = p.rid');
$query->condition(db_or()->condition('u.uid', 1)->condition('p.permission', $value));
$user_role_alias = 'ur' . $role_count;
$permission_alias = 'p' . $permission_count;
$query->innerJoin('users_roles', $user_role_alias, $user_role_alias . '.uid = u.uid');
$query->innerJoin('role_permission', $permission_alias , $user_role_alias . '.rid = ' . $permission_alias . '.rid');
$query->condition($permission_alias . '.permission', $value);
$role_count++;
$permission_count++;
}
else if ($key == 'role') {
$user_role_alias = 'ur' . $role_count;
$query->innerJoin('users_roles', $user_role_alias, $user_role_alias . '.uid = u.uid');
$query->condition($user_role_alias . '.rid' , $value);
$role_count++;
}
else {
$query->condition($filters[$key]['field'], $value);