Fix #145 - hiding variable status items no longer working

diff --git a/locallib.php b/locallib.php
index 5bcdb49..0f542e5 100644 (file)
--- a/locallib.php
+++ b/locallib.php
@@ -1472,7 +1472,10 @@ class attendance {
     public function update_status($status, $acronym, $description, $grade, $visible) {
         global $DB;
 
-        if (empty($acronym) || empty($description)) {
+        if (isset($visible)) {
+            $status->visible = $visible;
+            $updated[] = $visible ? get_string('show') : get_string('hide');
+        } else if (empty($acronym) || empty($description)) {
             return array('acronym' => $acronym, 'description' => $description);
         }
 
@@ -1490,10 +1493,6 @@ class attendance {
             $status->grade = $grade;
             $updated[] = $grade;
         }
-        if (isset($visible)) {
-            $status->visible = $visible;
-            $updated[] = $visible ? get_string('show') : get_string('hide');
-        }
         $DB->update_record('attendance_statuses', $status);
 
         $event = \mod_attendance\event\status_updated::create(array(