From 7e2d8ecbf90008ab5c1eec49e7cd001cc22acf60 Mon Sep 17 00:00:00 2001 From: Mariam05 Date: Mon, 9 Dec 2024 16:39:25 -0500 Subject: [PATCH] LTI-413: don't do unnecessary db updates --- app/jobs/room_meeting_watcher_job.rb | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/app/jobs/room_meeting_watcher_job.rb b/app/jobs/room_meeting_watcher_job.rb index 561e25a9..05139af4 100644 --- a/app/jobs/room_meeting_watcher_job.rb +++ b/app/jobs/room_meeting_watcher_job.rb @@ -11,7 +11,7 @@ def perform(room, data) info = fetch_meeting_info(data) if info[:meeting_in_progress] - @chosen_room.update(watcher_job_active: true) + @chosen_room.update(watcher_job_active: true) unless @chosen_room.watcher_job_active # no need to update if it's already true logger.info("Meeting in progress. Sending broadcast to room '#{room.name}'") # Broadcast updates to this room’s channel @@ -24,7 +24,7 @@ def perform(room, data) # Broadcast that the meeting ended MeetingInfoChannel.broadcast_to(room, { meeting_in_progress: false, action: 'end' }) - @chosen_room.update(watcher_job_active: false) + @chosen_room.update(watcher_job_active: false) if @chosen_room.watcher_job_active # no need to update if it's already false # Do not re-enqueue, job ends here end end @@ -42,8 +42,4 @@ def fetch_meeting_info(data) data[:participant_count] = info[:participantCount] data end - - def meeting_running?(info) - info[:returncode] == 'SUCCESS' && info[:running] == true - end end