GNUsocial/gnu-social
4
0
Fork 1
Code Issues Releases Wiki Activity

[Notification][ENTITY][Activity] Avoid including sender in notification targets

Browse Source
This commit is contained in:
Diogo Peralta Cordeiro 2021-12-12 06:38:07 +00:00
parent 93fa7eb0b5
commit 57beb178cc
Signed by: diogo
GPG Key ID: 18D2D35001FBFAB0
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
components/Notification/Notification.php
View File

@ -36,7 +36,7 @@ class Notification extends Component
*/ */
public function onNewNotification(Actor $sender, Activity $activity, array $ids_already_known = [], ?string $reason = null): bool public function onNewNotification(Actor $sender, Activity $activity, array $ids_already_known = [], ?string $reason = null): bool
{ {
$targets = $activity->getNotificationTargets($ids_already_known); $targets = $activity->getNotificationTargets($ids_already_known, sender_id: $sender->getId());
$this->notify($sender, $activity, $targets, $reason); $this->notify($sender, $activity, $targets, $reason);
return Event::next; return Event::next;

5
src/Entity/Activity.php
View File

@ -157,7 +157,7 @@ class Activity extends Entity
* *
* @return array of Actors * @return array of Actors
*/ */
public function getNotificationTargets(array $ids_already_known = []): array public function getNotificationTargets(array $ids_already_known = [], ?int $sender_id = null): array
{ {
$target_ids = []; $target_ids = [];
@ -179,6 +179,9 @@ class Activity extends Entity
if (array_key_exists('object', $ids_already_known)) { if (array_key_exists('object', $ids_already_known)) {
array_push($target_ids, ...$ids_already_known['object']); array_push($target_ids, ...$ids_already_known['object']);
} else { } else {
if (!is_null($author = $this->getObject()?->getActorId()) && $author !== $sender_id) {
$target_ids[] = $this->getObject()->getActorId();
}
array_push($target_ids, ...$this->getObject()->getNotificationTargets($ids_already_known)); array_push($target_ids, ...$this->getObject()->getNotificationTargets($ids_already_known));
} }