From: moodler <moodler>
Date: Thu, 31 Aug 2006 08:30:28 +0000 (+0000)
Subject: Clean up for role_assign()
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=86cbff7abd128a260c04ee09c195b34936cd3852;p=moodle.git

Clean up for role_assign()
---

diff --git a/lib/accesslib.php b/lib/accesslib.php
index d806ce08b7..954a4c5967 100755
--- a/lib/accesslib.php
+++ b/lib/accesslib.php
@@ -1170,12 +1170,10 @@ function role_assign($roleid, $userid, $groupid, $contextid, $timestart=0, $time
  * @return boolean - success or failure
  */
 function role_unassign($roleid, $userid, $groupid, $contextid) {
-    if ($groupid) {
-        // do nothing yet as this is not implemented
-    } 
-    else {
-          return delete_records('role_assignments', 'userid', $userid, 
-                  'roleid', $roleid, 'contextid', $contextid); 
+    if ($groupid && empty($userid)) {
+        return delete_records('role_assignments', 'groupid', $groupid, 'roleid', $roleid, 'contextid', $contextid); 
+    } else {
+        return delete_records('role_assignments', 'userid', $userid, 'roleid', $roleid, 'contextid', $contextid); 
     } 
 }