MBS-7802: Changes due to failing core unit tests

mebis-lp · May 31, 2024 · 644c2af · 644c2af
1 parent d4f9100
commit 644c2af
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 8 deletions.
diff --git a/classes/frontend.php b/classes/frontend.php
@@ -67,10 +67,15 @@ protected function get_javascript_init_params($course, \cm_info $cm = null, \sec
             ];
         }
 
-        $roles = explode(',', get_config('availability_role', 'coursecatroles'));
-        foreach ($roles as $rec) {
-            $rec = $DB->get_record('role', ['id' => $rec]);
-            if (empty($rec->id))  continue;
+        $roleids = explode(',', get_config('availability_role', 'coursecatroles'));
+        foreach ($roleids as $roleid) {
+            if(empty($roleid)){
+                continue;
+            }
+            $rec = $DB->get_record('role', ['id' => $roleid]);
+            if (empty($rec->id)) {
+                continue;
+            }
             $jsarray[] = (object)[
                 'id' => $rec->id,
                 'name' => (!empty($rec->name) ? $rec->name : $rec->shortname),
@@ -79,10 +84,15 @@ protected function get_javascript_init_params($course, \cm_info $cm = null, \sec
             ];
         }
 
-        $roles = explode(',', get_config('availability_role', 'globalroles'));
-        foreach ($roles as $rec) {
-            $rec = $DB->get_record('role', ['id' => $rec]);
-            if (empty($rec->id))  continue;
+        $roleids = explode(',', get_config('availability_role', 'globalroles'));
+        foreach ($roleids as $roleid) {
+            if (empty($roleid)) {
+                continue;
+            }
+            $rec = $DB->get_record('role', ['id' => $roleid]);
+            if (empty($rec->id)) {
+                continue;
+            }
             $jsarray[] = (object)[
                 'id' => $rec->id,
                 'name' => (!empty($rec->name) ? $rec->name : $rec->shortname),

diff --git a/tests/condition_test.php b/tests/condition_test.php
@@ -86,6 +86,7 @@ public function test_save() {
         $structure = (object)['id' => 123];
         $cond = new condition($structure);
         $structure->type = 'role';
+        $structure->typeid = 0;
         $this->assertEquals($structure, $cond->save());
     }
 }