Skip to content

Yodo - You only debug once #6935

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
nickvergessen wants to merge 13 commits into from
Closed

Yodo - You only debug once #6935

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
f004976
Handle all user status changes in case of "End meeting for all" in 1 …
nickvergessen Feb 10, 2022
904236d
Bump NC api lib
nickvergessen Feb 25, 2022
4fd2566
Add an integration test for the company call case
nickvergessen Feb 25, 2022
b30d0ea
Reset all in_call flags in one go
nickvergessen Feb 25, 2022
8c12796
Need to be combined with setLastMessage and delayed
nickvergessen Feb 25, 2022
114a9dd
Only calculate the room name once instead of loading all participants
nickvergessen Feb 25, 2022
d32e490
fixup! Need to be combined with setLastMessage and delayed
nickvergessen Feb 26, 2022
3925520
Combine last_activity and last_message update where possible
nickvergessen Feb 26, 2022
222dd2a
Emit a event so the HPB can send the update to the participants
nickvergessen Mar 2, 2022
dc54521
Deduplicate getting the room and participant in case some annotations…
nickvergessen Mar 4, 2022
6f3f404
Inject the room instead of querying it over and over again
nickvergessen Mar 4, 2022
f22cf58
Inject the attendee as well when loaded already
nickvergessen Mar 4, 2022
4733de6
Add chat case of company call
nickvergessen Mar 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Only calculate the room name once instead of loading all participants 
Signed-off-by: Joas Schilling <[email protected]>
  • Loading branch information
@nickvergessen
nickvergessen committed Feb 26, 2022
commit 114a9dda2fa18323162ce3bdbd9a929398726351
10 changes: 9 additions & 1 deletion lib/Activity/Listener.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -232,6 +232,11 @@ public function generateInvitationActivity(Room $room, array $participants): voi
return;
}

// We know the new participant is in the room,
// so skip loading them just to make sure they can read it.
// Must be overwritten later on for one-to-one chats.
$roomName = $room->getDisplayName($actorId);

foreach ($participants as $participant) {
if ($participant['actorType'] !== Attendee::ACTOR_USERS) {
// No user => no activity
Expand All @@ -244,7 +249,10 @@ public function generateInvitationActivity(Room $room, array $participants): voi
}

try {
$roomName = $room->getDisplayName($participant['actorId']);
if ($room->getType() === Room::TYPE_ONE_TO_ONE) {
// Overwrite the room name with the other participant
$roomName = $room->getDisplayName($participant['actorId']);
}
$event
->setObject('room', $room->getId(), $roomName)
->setSubject('invitation', [
Expand Down