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
Need to be combined with setLastMessage and delayed 
Currently there are 142 queries when adding a group with 70 users
instead of 1 query updating both values at the end.

Signed-off-by: Joas Schilling <[email protected]>
  • Loading branch information
@nickvergessen
nickvergessen committed Feb 25, 2022
commit 8c12796b0d276bce3c7fbddc1416dc803925094b
1 change: 1 addition & 0 deletions lib/Room.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -768,6 +768,7 @@ public function setPassword(string $password): bool {
* @return bool
*/
public function setLastActivity(\DateTime $now): bool {
// FIXME combine with setLastMessage and delay in case of multiple system messages
$update = $this->db->getQueryBuilder();
$update->update('talk_rooms')
->set('last_activity', $update->createNamedParameter($now, 'datetime'))
Expand Down