diff options
| author | Máté Bartus <mate.bartus@gmail.com> | 2015-08-08 19:53:34 +0200 |
|---|---|---|
| committer | Máté Bartus <mate.bartus@gmail.com> | 2015-08-08 19:53:34 +0200 |
| commit | 962fb6b19b01b1ad33b4f194443075dd759d1dc8 (patch) | |
| tree | 0aa476cb131f265abc88dfa1d5b194c348284b7c /phpBB/phpbb/notification/manager.php | |
| parent | ec270713d0f33531e9fd4b38ca0d5cc1f902aef3 (diff) | |
| parent | 05d583bba7faaf82487305e892f7758d4161f303 (diff) | |
| download | forums-962fb6b19b01b1ad33b4f194443075dd759d1dc8.tar forums-962fb6b19b01b1ad33b4f194443075dd759d1dc8.tar.gz forums-962fb6b19b01b1ad33b4f194443075dd759d1dc8.tar.bz2 forums-962fb6b19b01b1ad33b4f194443075dd759d1dc8.tar.xz forums-962fb6b19b01b1ad33b4f194443075dd759d1dc8.zip | |
Merge pull request #3819 from Nicofuma/ticket/14079
[ticket/14079] Correctly mark notifications as read
Diffstat (limited to 'phpBB/phpbb/notification/manager.php')
| -rw-r--r-- | phpBB/phpbb/notification/manager.php | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/phpBB/phpbb/notification/manager.php b/phpBB/phpbb/notification/manager.php index eb9a93be3d..4be678ac91 100644 --- a/phpBB/phpbb/notification/manager.php +++ b/phpBB/phpbb/notification/manager.php @@ -166,6 +166,7 @@ class manager $notification_type_id = false; } + /** @var method_interface $method */ foreach ($this->get_available_subscription_methods() as $method) { $method->mark_notifications($notification_type_id, $item_id, $user_id, $time, $mark_read); @@ -597,26 +598,32 @@ class manager { foreach ($types as $id => $type) { - $subscriptions[$id] = array(); + $type_subscriptions = $default_methods; if (!empty($user_notifications[$id])) { foreach ($user_notifications[$id] as $user_notification) { + $key = array_search($user_notification['method'], $type_subscriptions, true); if (!$user_notification['notify']) { + if ($key !== false) + { + unset($type_subscriptions[$key]); + } + continue; } - - if (!isset($subscriptions[$id])) + else if ($key === false) { - $subscriptions[$id] = array(); + $type_subscriptions[] = $user_notification['method']; } - - $subscriptions[$id][] = $user_notification['method']; } } - $subscriptions[$id] = array_merge($subscriptions[$id], $default_methods); + if (!empty($type_subscriptions)) + { + $subscriptions[$id] = $type_subscriptions; + } } } |