Fix case where not visible module was not hide

CrymSonir · CrymSonir · commit f9ca6776a412 · 2024-10-16T11:04:39.000+02:00
diff --git a/block_course_modulenavigation.php b/block_course_modulenavigation.php
@@ -280,7 +280,7 @@ public function get_content() {
                         }
                         $thissection->modules[] = $thismod;
                     } else {
-                        $thissection->modules[] = $this->checkmodule(
+                        $modulechecked = $this->checkmodule(
                             $module,
                             $context,
                             $completioninfo,
@@ -289,6 +289,10 @@ public function get_content() {
                             $myactivityid,
                             $thissection,
                         );
+
+                        if($modulechecked !== null) {
+                            $thissection->modules[] = $modulechecked;
+                        }
                     }
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -280,7 +280,7 @@ public function get_content() {`
`280`	`280`	`}`
`281`	`281`	`$thissection->modules[] = $thismod;`
`282`	`282`	`} else {`
`283`		`- $thissection->modules[] = $this->checkmodule(`
	`283`	`+ $modulechecked = $this->checkmodule(`
`284`	`284`	`$module,`
`285`	`285`	`$context,`
`286`	`286`	`$completioninfo,`
`@@ -289,6 +289,10 @@ public function get_content() {`
`289`	`289`	`$myactivityid,`
`290`	`290`	`$thissection,`
`291`	`291`	`);`
	`292`	`+`
	`293`	`+ if($modulechecked !== null) {`
	`294`	`+ $thissection->modules[] = $modulechecked;`
	`295`	`+ }`
`292`	`296`	`}`
`293`	`297`	`}`
`294`	`298`