Skip to content

Commit

Permalink
get_config() callers switched to expect a single value returned
Browse files Browse the repository at this point in the history
This simplifies the logic in the callers -- a good hint that
that we are doing the right thing.
  • Loading branch information
martinlanghoff committed Jan 16, 2007
1 parent 9220fba commit 1fd471a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions grade/lib.php
Expand Up @@ -3106,12 +3106,12 @@ function grade_get_course_students($courseid) {
}

$configvar = get_config('', 'gradebookroles');
if (empty($configvar->value)) {
if (empty($configvar)) {
notify ('no roles defined in admin->appearance->graderoles');
return false; // no roles to displayreturn false;
}

if ($rolestoget = explode(',', $configvar->value)) {
if ($rolestoget = explode(',', $configvar)) {
foreach ($rolestoget as $crole) {
if ($tempstudents = get_role_users($crole, $context, true)) {
foreach ($tempstudents as $tempuserid=>$tempstudent) {
Expand Down

0 comments on commit 1fd471a

Please sign in to comment.