From c23d1598b37422a3435b71c0bc6b60cc7219f16b Mon Sep 17 00:00:00 2001
From: Matthew Hilton <matthewhilton@catalyst-au.net>
Date: Thu, 14 Sep 2023 08:12:22 +1000
Subject: [PATCH] [#107] Add notification formatting to check statuses

---
 classes/admin_setting_check.php | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/classes/admin_setting_check.php b/classes/admin_setting_check.php
index 33de049..25ab873 100644
--- a/classes/admin_setting_check.php
+++ b/classes/admin_setting_check.php
@@ -18,6 +18,8 @@
 
 use admin_setting;
 use core\check\check;
+use core\check\result;
+use core\output\notification;
 
 /**
  * Admin setting for check api.
@@ -76,8 +78,18 @@ public function output_html($data, $query = '') {
 
         $resulthtml = $OUTPUT->check_result($checkresult);
         $resultinfo = $checkresult->get_summary();
-
         $out = $resulthtml . ' ' . $resultinfo;
+
+        switch($checkresult->get_status()){
+            case result::CRITICAL:
+            case result::ERROR:
+                $out = $OUTPUT->notification($out, notification::NOTIFY_ERROR, false);
+                break;
+
+            case result::OK:
+                $out = $OUTPUT->notification($out, notification::NOTIFY_SUCCESS, false);
+                break;
+        }
         return format_admin_setting($this, $this->visiblename, '', $out);
     }
 }