diff options
| author | Tristan Darricau <tristan.darricau@sensiolabs.com> | 2015-10-12 10:20:16 +0200 |
|---|---|---|
| committer | Tristan Darricau <tristan.darricau@sensiolabs.com> | 2015-10-12 10:20:16 +0200 |
| commit | 2668fa42ad977078883724e546e67c3439f28d58 (patch) | |
| tree | ff5d4e34cfd7b20330c6d8afe7e64d7f847582f5 /phpBB/includes/functions.php | |
| parent | 5f8fb38b529ef0b04fcfbaaee940b29e7a5e92b1 (diff) | |
| parent | 8a52ccc9389dd54e42e357f09808203dcbf65e91 (diff) | |
| download | forums-2668fa42ad977078883724e546e67c3439f28d58.tar forums-2668fa42ad977078883724e546e67c3439f28d58.tar.gz forums-2668fa42ad977078883724e546e67c3439f28d58.tar.bz2 forums-2668fa42ad977078883724e546e67c3439f28d58.tar.xz forums-2668fa42ad977078883724e546e67c3439f28d58.zip | |
Merge pull request #3946 from RMcGirr83/ticket_13591
[ticket/13591] Change SQL query into array to allow
* RMcGirr83/ticket_13591:
[ticket/13591] Change SQL query into array to allow
Diffstat (limited to 'phpBB/includes/functions.php')
| -rw-r--r-- | phpBB/includes/functions.php | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/phpBB/includes/functions.php b/phpBB/includes/functions.php index 4fb2da7a72..b2b891be66 100644 --- a/phpBB/includes/functions.php +++ b/phpBB/includes/functions.php @@ -4264,10 +4264,14 @@ function obtain_users_online_string($online_users, $item_id = 0, $item = 'forum' if (sizeof($online_users['online_users'])) { - $sql = 'SELECT username, username_clean, user_id, user_type, user_allow_viewonline, user_colour - FROM ' . USERS_TABLE . ' - WHERE ' . $db->sql_in_set('user_id', $online_users['online_users']) . ' - ORDER BY username_clean ASC'; + $sql_ary = array( + 'SELECT' => 'u.username, u.username_clean, u.user_id, u.user_type, u.user_allow_viewonline, u.user_colour', + 'FROM' => array( + USERS_TABLE => 'u', + ), + 'WHERE' => $db->sql_in_set('u.user_id', $online_users['online_users']), + 'ORDER BY' => 'u.username_clean ASC', + ); /** * Modify SQL query to obtain online users data @@ -4279,13 +4283,14 @@ function obtain_users_online_string($online_users, $item_id = 0, $item = 'forum' * @var string item Restrict online users to a certain * session item, e.g. forum for * session_forum_id - * @var string sql SQL query to obtain users online data + * @var string sql_ary SQL query to obtain users online data * @since 3.1.4-RC1 + * @changed 3.1.7-RC1 Change sql query into array and adjust var accordingly. Allows extension authors the ability to adjust the sql_ary. */ - $vars = array('online_users', 'item_id', 'item', 'sql'); + $vars = array('online_users', 'item_id', 'item', 'sql_ary'); extract($phpbb_dispatcher->trigger_event('core.obtain_users_online_string_sql', compact($vars))); - $result = $db->sql_query($sql); + $result = $db->sql_query($db->sql_build_query('SELECT', $sql_ary)); $rowset = $db->sql_fetchrowset($result); $db->sql_freeresult($result); |