mirror/php-phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-07-30 21:40:43 +02:00
Code Issues Releases Wiki Activity

Merge pull request #6484 from rxu/ticket/17107

Browse Source 
[ticket/17107] Fix viewonline page locations for posting modes - 3.3.x
This commit is contained in:
Marc Alexander
2023-06-18 23:12:59 +02:00
parent 65fb35223c 4f9bd296a7
commit c11f16fda0
5 changed files with 181 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
phpBB/config/default/container/services_content.yml
View File

@@ -71,3 +71,4 @@ services:
class: phpbb\viewonline_helper
arguments:
- '@filesystem'
- '@dbal.conn'

67
phpBB/phpbb/viewonline_helper.php
View File

@@ -14,27 +14,74 @@
namespace phpbb;
/**
* Class to handle viewonline related tasks
*/
* Class to handle viewonline related tasks
*/
class viewonline_helper
{
/** @var \phpbb\filesystem\filesystem_interface */
protected $filesystem;
/** @var \phpbb\db\driver\driver_interface */
protected $db;
/**
* @param \phpbb\filesystem\filesystem_interface $filesystem phpBB's filesystem service
*/
public function __construct(\phpbb\filesystem\filesystem_interface $filesystem)
* @param \phpbb\filesystem\filesystem_interface $filesystem phpBB's filesystem service
* @param \phpbb\db\driver\driver_interface $db
*/
public function __construct(\phpbb\filesystem\filesystem_interface $filesystem, \phpbb\db\driver\driver_interface $db)
{
$this->filesystem = $filesystem;
$this->db = $db;
}
/**
* Get user page
*
* @param string $session_page User's session page
* @return array Match array filled by preg_match()
*/
* Get forum IDs for topics
*
* Retrieve forum IDs and add the data into the session data array
* Array structure matches sql_fethrowset() result array
*
* @param array $session_data_rowset Users' session data array
* @return void
*/
public function get_forum_ids(array &$session_data_rowset): void
{
$topic_ids = $match = [];
foreach ($session_data_rowset as $number => $row)
{
if ($row['session_forum_id'] == 0 && preg_match('#t=([0-9]+)#', $row['session_page'], $match))
{
$topic_ids[$number] = (int) $match[1];
}
}
if (count($topic_ids = array_unique($topic_ids)))
{
$sql_ary = [
'SELECT' => 't.topic_id, t.forum_id',
'FROM' => [
TOPICS_TABLE => 't',
],
'WHERE' => $this->db->sql_in_set('t.topic_id', $topic_ids),
'ORDER_BY' => 't.topic_id',
];
$result = $this->db->sql_query($this->db->sql_build_query('SELECT', $sql_ary));
$forum_ids_rowset = $this->db->sql_fetchrowset($result);
$this->db->sql_freeresult($result);
foreach ($forum_ids_rowset as $forum_ids_row)
{
$session_data_row_number = array_search((int) $forum_ids_row['topic_id'], $topic_ids);
$session_data_rowset[$session_data_row_number]['session_forum_id'] = (int) $forum_ids_row['forum_id'];
}
}
}
/**
* Get user page
*
* @param string $session_page User's session page
* @return array Match array filled by preg_match()
*/
public function get_user_page($session_page)
{
$session_page = $this->filesystem->clean_path($session_page);

8
phpBB/viewonline.php
View File

@@ -180,6 +180,8 @@ $vars = array('sql_ary', 'show_guests', 'guest_counter', 'forum_data');
extract($phpbb_dispatcher->trigger_event('core.viewonline_modify_sql', compact($vars)));
$result = $db->sql_query($db->sql_build_query('SELECT', $sql_ary));
$session_data_rowset = $db->sql_fetchrowset($result);
$db->sql_freeresult($result);
$prev_id = $prev_ip = $user_list = array();
$logged_visible_online = $logged_hidden_online = $counter = 0;
@@ -190,7 +192,10 @@ $controller_helper = $phpbb_container->get('controller.helper');
/** @var \phpbb\group\helper $group_helper */
$group_helper = $phpbb_container->get('group_helper');
while ($row = $db->sql_fetchrow($result))
// Get forum IDs for session pages which have only 't' parameter
$viewonline_helper->get_forum_ids($session_data_rowset);
foreach ($session_data_rowset as $row)
{
if ($row['user_id'] != ANONYMOUS && !isset($prev_id[$row['user_id']]))
{
@@ -438,7 +443,6 @@ while ($row = $db->sql_fetchrow($result))
$template->assign_block_vars('user_row', $template_row);
}
$db->sql_freeresult($result);
unset($prev_id, $prev_ip);
$order_legend = ($config['legend_sort_groupname']) ? 'group_name' : 'group_legend';