Merge pull request ncstate-delta#503 from ncstate-delta/fix/zoom-erro…

…r-message-handling settings: get error response from webservice_exception
learnweb · Aug 3, 2023 · cb97951 · cb97951
2 parents e66f829 + d5e5441
commit cb97951
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/settings.php b/settings.php
@@ -36,14 +36,16 @@
 
     // Test whether connection works and display result to user.
     if (!CLI_SCRIPT && $PAGE->url == $CFG->wwwroot . '/' . $CFG->admin . '/settings.php?section=modsettingzoom') {
-        $status = 'connectionok';
-        $notifyclass = 'notifysuccess';
+        $status = 'connectionfailed';
+        $notifyclass = 'notifyproblem';
         $errormessage = '';
         try {
             zoom_get_user(zoom_get_api_identifier($USER));
+            $status = 'connectionok';
+            $notifyclass = 'notifysuccess';
+        } catch (\mod_zoom\webservice_exception $error) {
+            $errormessage = $error->response;
         } catch (moodle_exception $error) {
-            $notifyclass = 'notifyproblem';
-            $status = 'connectionfailed';
             $errormessage = $error->a;
         }