From a211a40c75394096b132b697f631621e09c8cc53 Mon Sep 17 00:00:00 2001 From: Dan Marsden Date: Tue, 2 Nov 2010 01:38:56 +0000 Subject: [PATCH] Assignment MDL-24948 fix depreceated calls to get group info --- mod/assignment/type/online/assignment.class.php | 12 +++--------- mod/assignment/type/upload/assignment.class.php | 12 +++--------- .../type/uploadsingle/assignment.class.php | 12 +++--------- 3 files changed, 9 insertions(+), 27 deletions(-) diff --git a/mod/assignment/type/online/assignment.class.php b/mod/assignment/type/online/assignment.class.php index 7317f3e0c8b..8a225494de3 100644 --- a/mod/assignment/type/online/assignment.class.php +++ b/mod/assignment/type/online/assignment.class.php @@ -398,18 +398,12 @@ class assignment_online extends assignment_base { //online assignment can use html $filextn=".html"; - if (isset($this->cm->groupmode) && empty($this->course->groupmodeforce)) { - $groupmode = $this->cm->groupmode; - } else { - $groupmode = $this->course->groupmode; - } - + $groupmode = groups_get_activity_groupmode($this->cm); $groupid = 0; // All users $groupname = ''; if ($groupmode) { - $group = get_current_group($this->course->id, true); - $groupid = $group->id; - $groupname = $group->name.'-'; + $groupid = groups_get_activity_group($this->cm, true); + $groupname = groups_get_group_name($groupid).'-'; } $filename = str_replace(' ', '_', clean_filename($this->course->shortname.'-'.$this->assignment->name.'-'.$groupname.$this->assignment->id.".zip")); //name of new zip file. foreach ($submissions as $submission) { diff --git a/mod/assignment/type/upload/assignment.class.php b/mod/assignment/type/upload/assignment.class.php index 47319827f3d..367af5e0232 100644 --- a/mod/assignment/type/upload/assignment.class.php +++ b/mod/assignment/type/upload/assignment.class.php @@ -1097,18 +1097,12 @@ class assignment_upload extends assignment_base { $filesforzipping = array(); $fs = get_file_storage(); - if (isset($this->cm->groupmode) && empty($this->course->groupmodeforce)) { - $groupmode = $this->cm->groupmode; - } else { - $groupmode = $this->course->groupmode; - } - + $groupmode = groups_get_activity_groupmode($this->cm); $groupid = 0; // All users $groupname = ''; if ($groupmode) { - $group = get_current_group($this->course->id, true); - $groupid = $group->id; - $groupname = $group->name.'-'; + $groupid = groups_get_activity_group($this->cm, true); + $groupname = groups_get_group_name($groupid).'-'; } $filename = str_replace(' ', '_', clean_filename($this->course->shortname.'-'.$this->assignment->name.'-'.$groupname.$this->assignment->id.".zip")); //name of new zip file. foreach ($submissions as $submission) { diff --git a/mod/assignment/type/uploadsingle/assignment.class.php b/mod/assignment/type/uploadsingle/assignment.class.php index bd75c085166..d0278acb5a7 100644 --- a/mod/assignment/type/uploadsingle/assignment.class.php +++ b/mod/assignment/type/uploadsingle/assignment.class.php @@ -364,18 +364,12 @@ class assignment_uploadsingle extends assignment_base { $filesforzipping = array(); $fs = get_file_storage(); - if (isset($this->cm->groupmode) && empty($this->course->groupmodeforce)) { - $groupmode = $this->cm->groupmode; - } else { - $groupmode = $this->course->groupmode; - } - + $groupmode = groups_get_activity_groupmode($this->cm); $groupid = 0; // All users $groupname = ''; if ($groupmode) { - $group = get_current_group($this->course->id, true); - $groupid = $group->id; - $groupname = $group->name.'-'; + $groupid = groups_get_activity_group($this->cm, true); + $groupname = groups_get_group_name($groupid).'-'; } $filename = str_replace(' ', '_', clean_filename($this->course->shortname.'-'.$this->assignment->name.'-'.$groupname.$this->assignment->id.".zip")); //name of new zip file. foreach ($submissions as $submission) {