Merge pull request #470 from ncstate-delta/null-auto-record

Set default for option_auto_record for older restores
ncstate-delta · Mar 16, 2023 · 869f5e0 · 869f5e0
2 parents 4400334 + 1b98f3f
commit 869f5e0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/backup/moodle2/restore_zoom_stepslib.php b/backup/moodle2/restore_zoom_stepslib.php
@@ -68,6 +68,7 @@ protected function process_zoom($data) {
             $data->join_url = '';
             $data->meeting_id = 0;
             $data->exists_on_zoom = ZOOM_MEETING_EXPIRED;
+            $data->option_auto_recording = 'none';
         }
 
         $data->course = $this->get_courseid();

diff --git a/lib.php b/lib.php
@@ -346,6 +346,9 @@ function populate_zoom_from_response(stdClass $zoom, stdClass $response) {
     if (isset($response->settings->auto_recording)) {
         $newzoom->option_auto_recording = $response->settings->auto_recording;
     }
+    if (!isset($newzoom->option_auto_recording)) {
+        $newzoom->option_auto_recording = 'none';
+    }
 
     return $newzoom;
 }