minor: Complete comment about why we need to specify a user with permissions.

This commit is contained in:
Justin Clark-Casey
2014-07-31 02:30:11 +01:00
parent fc1f8365da
commit ca65d9dbd5
@@ -31,6 +31,9 @@ $charter = $params->args['charter'];
// and feed back the fields we don't want to change!
$existingGroupE = GetGroupByName($GROUPS_SERVICE_URI, $groupId)->RESULT;
// The update change can only be performed by a user with sufficient permissions, which the founder should have.
// Can't get around this at the moment, even though this should be a voluntary perms check for simulator purposes,
// not mandatory.
$existingFounderId = (string)$existingGroupE->FounderID;
$existingGroupId = (string)$existingGroupE->GroupID;
$existingGroupPictureId = (string)$existingGroupE->InsigniaID;
@@ -40,9 +43,9 @@ $existingOpenEnrollment = ToBool((string)$existingGroupE->OpenEnrollment);
$existingMembershipFee = (int)$existingGroupE->MembershipFee;
$existingShownInList = ToBool((string)$existingGroupE->ShownInList);
echo "existingGroupId '$existingGroupId'\n";
echo "existingGroupPictureId '$existingGroupPictureId'\n";
echo "existingAllowPublish '$existingAllowPublish'\n";
//echo "existingGroupId '$existingGroupId'\n";
//echo "existingGroupPictureId '$existingGroupPictureId'\n";
//echo "existingAllowPublish '$existingAllowPublish'\n";
UpdateGroup(
$GROUPS_SERVICE_URI, $existingGroupId, $existingFounderId, $charter,