L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-53918 tool_cohortroles: Assign role on non-empty user and cohort IDs

Browse Source
This commit is contained in:
Jun Pataleta
2016-04-29 15:43:56 +08:00
parent 6bd3ab0717
commit d238016e0d
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
admin/tool/cohortroles/index.php
+9 -9
View File
@@ -57,15 +57,15 @@ if ($removeid) {
require_sesskey();
// We must create them all or none.
$saved = 0;
foreach ($data->userids as $userid) {
if (empty($data->cohortids)) {
$data->cohortids = array();
}
foreach ($data->cohortids as $cohortid) {
$params = (object) array('userid' => $userid, 'cohortid' => $cohortid, 'roleid' => $data->roleid);
$result = \tool_cohortroles\api::create_cohort_role_assignment($params);
if ($result) {
$saved++;
// Loop through userids and cohortids only if both of them are not empty.
if (!empty($data->userids) && !empty($data->cohortids)) {
foreach ($data->userids as $userid) {
foreach ($data->cohortids as $cohortid) {
$params = (object) array('userid' => $userid, 'cohortid' => $cohortid, 'roleid' => $data->roleid);
$result = \tool_cohortroles\api::create_cohort_role_assignment($params);
if ($result) {
$saved++;
}
}
}
}