github-mirror/framework
2
0
Fork 0
mirror of https://github.com/flarum/framework.git synced 2024-12-13 07:03:35 +08:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

Notification one-per-user limit should work between instances

Browse Source
This commit is contained in:
Toby Zerner 2015-07-23 14:33:58 +09:30
parent 3533dfffc1
commit 4817275f4e
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
framework/core/src/Core/Notifications/NotificationSyncer.php
View File

@ -17,14 +17,14 @@ class NotificationSyncer
*
* @var bool
*/
protected $onePerUser = false;
protected static $onePerUser = false;
/**
* An internal list of user IDs that notifications have been sent to.
*
* @var int[]
*/
protected $sentTo = [];
protected static $sentTo = [];
/**
* @var NotificationRepository
@ -80,9 +80,9 @@ class NotificationSyncer
if ($existing) {
$toUndelete[] = $existing->id;
$toDelete->forget($toDelete->search($existing));
} elseif (! $this->onePerUser || ! in_array($user->id, $this->sentTo)) {
} elseif (! static::$onePerUser || ! in_array($user->id, static::$sentTo)) {
$newRecipients[] = $user;
$this->sentTo[] = $user->id;
static::$sentTo[] = $user->id;
}
}
@ -136,12 +136,12 @@ class NotificationSyncer
*/
public function onePerUser(callable $callback)
{
$this->sentTo = [];
$this->onePerUser = true;
static::$sentTo = [];
static::$onePerUser = true;
$callback();
$this->onePerUser = false;
static::$onePerUser = false;
}
/**
@ -169,7 +169,7 @@ class NotificationSyncer
);
if ($blueprint instanceof MailableBlueprint) {
$this->mailNotifications($blueprint);
$this->mailNotifications($blueprint, $recipients);
}
}