Skip to content

Commit bb0e642

Browse files
committed
perf: don't preemptively determine the backend in getSeenUsers
Signed-off-by: Robin Appelman <robin@icewind.nl>
1 parent ed114b7 commit bb0e642

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

lib/private/User/Manager.php

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -871,13 +871,7 @@ public function getSeenUsers(int $offset = 0, ?int $limit = null): \Iterator {
871871
$offset += $batchSize;
872872

873873
foreach ($userIds as $userId) {
874-
foreach ($this->backends as $backend) {
875-
if ($backend->userExists($userId)) {
876-
$user = new LazyUser($userId, $this, null, $backend);
877-
yield $userId => $user;
878-
break;
879-
}
880-
}
874+
yield $userId => $this->getExistingUser($userId);
881875
}
882876
} while (count($userIds) === $batchSize && $limit !== 0);
883877
}

0 commit comments

Comments
 (0)