Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Critical
-
Resolution: Fixed
-
Affects Version/s: 1.9.4
-
Fix Version/s: None
-
Component/s: Module: Certificate
-
Labels:None
-
Environment:This improvement is not affected by Operating System.
-
Difficulty:Moderate
-
Affected Branches:MOODLE_19_STABLE
Description
Certificate Modlule lib.php,v 1.21.2.20 2008/09/04 (2008080904)
Fix Certificate Completion Reports list/sort By Course Group
Issue:
There is currently an issue with the Certificate Module which prevents reporting
Certificate Issues by group. Cannot filter Certificate issues by group in
report correctly.
Suggested Code Modification:
Line 545 of mod/certificate/lib.php
- foreach($groupusers as $id => $gpuser) {
- if (!isset($users[$id])) { - //remove this user as it isn't in the group! - unset($users[$id]); - }
- }
+ foreach($users as $user){ //remove this user as it isn't in the group! + if(!array_key_exists($user->id, $groupusers)) + unset($users[$user->id]); +}
John, thanks for your report. Assigning to the certificate module component lead.