fix: handle null values in array_reverse call

catalyst · Oct 24, 2024 · 6eeef26 · 6eeef26
1 parent 95c7dcf
commit 6eeef26
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib.php b/lib.php
@@ -371,9 +371,10 @@ protected function load_definition() {
         $options = $this->get_options();
         if (!$options['sortlevelsasc']) {
             if ($this->definition) {
-                foreach (array_keys($this->definition->rubric_criteria) as $rcid) {
-                    $this->definition->rubric_criteria[$rcid]['levels'] =
-                    array_reverse($this->definition->rubric_criteria[$rcid]['levels'], true);
+                foreach ($this->definition->rubric_criteria as $rcid => $criteria) {
+                    if (!empty($criteria['levels']) && is_array($criteria['levels'])) {
+                        $this->definition->rubric_criteria[$rcid]['levels'] = array_reverse($criteria['levels'], true);
+                    }
                 }
             }
         }