mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-08-25 17:31:55 +02:00
Code Issues Releases Wiki Activity

Remove use of event priorities

Browse Source 
Event priorities are no longer in Laravel - see dbbfc62bef

Now we can't force this event listener to be run before the discussion
metadata is updated; instead we just subtract one off of the new
last post number.
This commit is contained in:
Toby Zerner
2018-01-11 14:06:43 +10:30
parent 14bc69d6bd
commit c7aeef19ea
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
extensions/subscriptions/src/Listener/SendNotificationWhenReplyIsPosted.php
View File

@@ -44,10 +44,7 @@ class SendNotificationWhenReplyIsPosted
{ {
$events->listen(ConfigureNotificationTypes::class, [$this, 'addNotificationType']); $events->listen(ConfigureNotificationTypes::class, [$this, 'addNotificationType']);
// Register with '1' as priority so this runs before discussion metadata $events->listen(Posted::class, [$this, 'whenPosted']);
// is updated, as we need to compare the user's last read number to that
// of the previous post.
$events->listen(Posted::class, [$this, 'whenPosted'], 1);
$events->listen(Hidden::class, [$this, 'whenHidden']); $events->listen(Hidden::class, [$this, 'whenHidden']);
$events->listen(Restored::class, [$this, 'whenRestored']); $events->listen(Restored::class, [$this, 'whenRestored']);
$events->listen(Deleted::class, [$this, 'whenDeleted']); $events->listen(Deleted::class, [$this, 'whenDeleted']);
@@ -72,7 +69,7 @@ class SendNotificationWhenReplyIsPosted
$notify = $discussion->readers() $notify = $discussion->readers()
->where('users.id', '!=', $post->user_id) ->where('users.id', '!=', $post->user_id)
->where('users_discussions.subscription', 'follow') ->where('users_discussions.subscription', 'follow')
->where('users_discussions.read_number', $discussion->last_post_number) ->where('users_discussions.read_number', $discussion->last_post_number - 1)
->get(); ->get();
$this->notifications->sync( $this->notifications->sync(