Issue Details (XML | Word | Printable)

Key: CONTRIB-1026
Type: Improvement Improvement
Status: Open Open
Priority: Critical Critical
Assignee: Chardelle Busch
Reporter: John T. Macklin
Votes: 1
Watchers: 1
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Non-core contributed modules

Certificate Module Reports by Seperate Group

Created: 11/Feb/09 05:07 AM   Updated: 11/Feb/09 05:17 AM
Component/s: Module: Certificate
Affects Version/s: 1.9.4
Fix Version/s: None

Environment: This improvement is not affected by Operating System.

Participants: Chardelle Busch, Helen Foster and John T. Macklin
Security Level: None
Difficulty: Moderate
Affected Branches: MOODLE_19_STABLE


 Description  « Hide
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]); +}



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Helen Foster added a comment - 11/Feb/09 05:17 AM
John, thanks for your report. Assigning to the certificate module component lead.