mirror/php-phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-02-22 19:07:27 +01:00
Code Issues Releases Wiki Activity

[ticket/12742] Add sort options for notification types requiring sorting

Browse Source 
PHPBB3-12742
This commit is contained in:
Marc Alexander 2014-08-09 17:43:28 +02:00
parent 0de1d65804
commit 57d94b7bb4
3 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
phpBB/phpbb/notification/type/base.php
View File

@ -540,9 +540,10 @@ abstract class base implements \phpbb\notification\type\type_interface
* @param array $users Array of users that have subscribed to a notification
* @param int $forum_id Forum ID of the forum
* @param array $options Array of notification options
* @param bool $sort Whether the users array should be sorted. Default: false
* @return array Array of users that are authorised recipients
*/
protected function get_authorised_recipients($users, $forum_id, $options)
protected function get_authorised_recipients($users, $forum_id, $options, $sort = false)
{
if (empty($users))
{
@ -551,6 +552,11 @@ abstract class base implements \phpbb\notification\type\type_interface
$users = array_unique($users);
if ($sort)
{
sort($users);
}
$auth_read = $this->auth->acl_get_list($users, 'f_read', $forum_id);
if (empty($auth_read))

2
phpBB/phpbb/notification/type/post.php
View File

@ -123,7 +123,7 @@ class post extends \phpbb\notification\type\base
}
$this->db->sql_freeresult($result);
$notify_users = $this->get_authorised_recipients($users, $post['forum_id'], $options);
$notify_users = $this->get_authorised_recipients($users, $post['forum_id'], $options, true);
if (empty($notify_users))
{

2
phpBB/phpbb/notification/type/quote.php
View File

@ -102,7 +102,7 @@ class quote extends \phpbb\notification\type\post
}
$this->db->sql_freeresult($result);
return $this->get_authorised_recipients($users, $post['forum_id'], $options);
return $this->get_authorised_recipients($users, $post['forum_id'], $options, true);
}
/**