Merge pull request #55 from andil-elearning/master_missing_context

Prevent export crash due to missing user context
bostelm · Jul 25, 2024 · 858153b · 858153b
2 parents ec4aec8 + 89dc953
commit 858153b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/exportlib.php b/exportlib.php
@@ -659,7 +659,8 @@ public function get_header(scheduler $scheduler) {
      * @return string the value of this field for the given data
      */
     public function get_value(slot $slot, $appointment) {
-        if (!$appointment instanceof appointment || $appointment->studentid == 0) {
+        if (!$appointment instanceof appointment || $appointment->studentid == 0
+        || !context_user::instance($appointment->studentid, IGNORE_MISSING)) {
             return '';
         }
         $this->field->set_userid($appointment->studentid);