Skip to content

Commit 767dc3e

Browse files
committed
fix(activity): do not loop affected userIDs twice
Signed-off-by: Anna Larch <[email protected]>
1 parent 6911a33 commit 767dc3e

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

lib/private/Activity/Manager.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -124,10 +124,8 @@ public function bulkPublish(IEvent $event, array $affectedUserIds, ISetting $set
124124
throw new IncompleteActivityException('The given event is invalid');
125125
}
126126

127-
if ($event->getAuthor() === '') {
128-
if ($this->session->getUser() instanceof IUser) {
129-
$event->setAuthor($this->session->getUser()->getUID());
130-
}
127+
if (($event->getAuthor() === '') && $this->session->getUser() instanceof IUser) {
128+
$event->setAuthor($this->session->getUser()->getUID());
131129
}
132130

133131
if (!$event->getTimestamp()) {
@@ -141,6 +139,7 @@ public function bulkPublish(IEvent $event, array $affectedUserIds, ISetting $set
141139
foreach ($this->getConsumers() as $c) {
142140
if ($c instanceof IBulkConsumer) {
143141
$c->bulkReceive($event, $affectedUserIds, $setting);
142+
continue;
144143
}
145144
foreach ($affectedUserIds as $affectedUserId) {
146145
$event->setAffectedUser($affectedUserId);

0 commit comments

Comments
 (0)