Fixes #808. Allow taking of attendance for non-participation groups.

micaherne · danmarsden · commit 201c5355ae68 · 2025-03-14T20:04:53.000+13:00
diff --git a/take.php b/take.php
@@ -54,7 +54,8 @@
 $att = new mod_attendance_structure($att, $cm, $course, $PAGE->context, $pageparams);
 
 $allowedgroups = groups_get_activity_allowed_groups($cm);
-if (!empty($pageparams->grouptype) && !array_key_exists($pageparams->grouptype, $allowedgroups)) {
+$accessallgroups = has_capability('moodle/site:accessallgroups', $context);
+if (!$accessallgroups && !empty($pageparams->grouptype) && !array_key_exists($pageparams->grouptype, $allowedgroups)) {
      $group = groups_get_group($pageparams->grouptype);
      throw new moodle_exception('cannottakeforgroup', 'attendance', '', $group->name);
 }

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,8 @@`
`54`	`54`	`$att = new mod_attendance_structure($att, $cm, $course, $PAGE->context, $pageparams);`
`55`	`55`
`56`	`56`	`$allowedgroups = groups_get_activity_allowed_groups($cm);`
`57`		`-if (!empty($pageparams->grouptype) && !array_key_exists($pageparams->grouptype, $allowedgroups)) {`
	`57`	`+$accessallgroups = has_capability('moodle/site:accessallgroups', $context);`
	`58`	`+if (!$accessallgroups && !empty($pageparams->grouptype) && !array_key_exists($pageparams->grouptype, $allowedgroups)) {`
`58`	`59`	`$group = groups_get_group($pageparams->grouptype);`
`59`	`60`	`throw new moodle_exception('cannottakeforgroup', 'attendance', '', $group->name);`
`60`	`61`	`}`