Fixed case for multiple grops

safatshahin · Mar 17, 2006 · 884dc78 · 884dc78
1 parent 36bd0b2
commit 884dc78
Showing 1 changed file with 3 additions and 11 deletions.
diff --git a/admin/uploaduser.php b/admin/uploaduser.php
@@ -334,18 +334,10 @@
                         if ($course[$i] && $groupid[$i]) {
                             if (record_exists('user_students','userid',$user->id,'course',$course[$i]->id) ||
                                     record_exists('user_teachers','userid',$user->id,'course',$course[$i]->id)) {
-                                $usergroup = user_group($course[$i]->id,$user->id);
-                                if ($usergroup) {
-                                    notify('-->' . get_string('groupalready','error',$usergroup->name));
+                                if (add_user_to_group($groupid[$i], $user->id)) {
+                                    notify('-->' . get_string('addedtogroup','',$addgroup[$i]));
                                 } else {
-                                    $group_member->groupid = $groupid[$i];
-                                    $group_member->userid = $user->id;
-                                    $group_member->timeadded = time();
-                                    if (insert_record("groups_members",$group_member)) {
-                                        notify('-->' . get_string('addedtogroup','',$addgroup[$i]));
-                                    } else {
-                                        notify('-->' . get_string('addedtogroupnot','',$addgroup[$i]));
-                                    }
+                                    notify('-->' . get_string('addedtogroupnot','',$addgroup[$i]));
                                 }
                             } else {
                                 notify('-->' . get_string('addedtogroupnotenrolled','',$addgroup[$i]));