Fixed remove_teacher, so that a removed teacher is also removed from any

groups they belong to.
2004-09-14 23:41:27 +00:00
parent 908b8b9a4f
commit 5ec9237d2b
1 changed files with 11 additions and 0 deletions
@@ -965,6 +965,17 @@ function remove_teacher($userid, $courseid=0) {
                delete_records("forum_subscriptions", "forum", $forum->id, "userid", $userid);
            }
        }
+
+        /// Next if the teacher is not registered as a student, but is
+        /// a member of a group, remove them from the group.
+        if (!isstudent($courseid, $userid)) {
+            if ($groups = get_groups($courseid, $userid)) {
+                foreach ($groups as $group) {
+                    delete_records("groups_members", "groupid", $group->id, "userid", $userid);
+                }
+            }
+        }
+
        return delete_records("user_teachers", "userid", $userid, "course", $courseid);
    } else {
        delete_records("forum_subscriptions", "userid", $userid);