Skip to content

Commit

Permalink
Merge pull request phpbb#3699 from nickvergessen/ticket/13823
Browse files Browse the repository at this point in the history
[ticket/13823] Move the part of the query, so the update will get it …
  • Loading branch information
nickvergessen committed Jun 10, 2015
2 parents dfb7bb1 + 3d18d6d commit 2c65b9b
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions phpBB/includes/acp/acp_prune.php
Original file line number Diff line number Diff line change
Expand Up @@ -506,9 +506,9 @@ function get_prune_users(&$user_ids, &$usernames)
WHERE ug.group_id = ' . (int) $group_id . '
AND ug.user_id <> ' . ANONYMOUS . '
AND u.user_type <> ' . USER_FOUNDER . '
AND ug.user_pending = 0 ' .
(!empty($user_ids) ? ' AND ' . $db->sql_in_set('ug.user_id', $user_ids) : '') . '
AND u.user_id = ug.user_id';
AND ug.user_pending = 0
AND u.user_id = ug.user_id
' . (!empty($user_ids) ? ' AND ' . $db->sql_in_set('ug.user_id', $user_ids) : '');
$result = $db->sql_query($sql);

// we're performing an intersection operation, so all the relevant users
Expand All @@ -532,10 +532,10 @@ function get_prune_users(&$user_ids, &$usernames)
$sql = 'SELECT u.user_id, u.username, COUNT(p.post_id) AS queue_posts
FROM ' . POSTS_TABLE . ' p, ' . USERS_TABLE . ' u
WHERE u.user_id <> ' . ANONYMOUS . '
AND u.user_type <> ' . USER_FOUNDER .
(!empty($user_ids) ? ' AND ' . $db->sql_in_set('p.poster_id', $user_ids) : '') . '
AND u.user_type <> ' . USER_FOUNDER . '
AND ' . $db->sql_in_set('p.post_visibility', array(ITEM_UNAPPROVED, ITEM_REAPPROVE)) . '
AND u.user_id = p.poster_id
' . (!empty($user_ids) ? ' AND ' . $db->sql_in_set('p.poster_id', $user_ids) : '') . '
GROUP BY p.poster_id
HAVING queue_posts ' . $key_match[$queue_select] . ' ' . $posts_on_queue;
$result = $db->sql_query($sql);
Expand Down

0 comments on commit 2c65b9b

Please sign in to comment.