MDL-50357 groups: remove pointless option of groups_delete_group_members
authorDan Poltawski <dan@moodle.com>
Wed, 30 Dec 2015 09:05:43 +0000 (09:05 +0000)
committerDan Poltawski <dan@moodle.com>
Wed, 30 Dec 2015 09:19:22 +0000 (09:19 +0000)
group/lib.php
lib/upgrade.txt

index e475318..3178e7b 100644 (file)
@@ -565,10 +565,10 @@ function groups_delete_grouping($groupingorid) {
  *
  * @param int $courseid
  * @param int $userid 0 means all users
  *
  * @param int $courseid
  * @param int $userid 0 means all users
- * @param bool $showfeedback
+ * @param bool $unused - formerly $showfeedback, is no longer used.
  * @return bool success
  */
  * @return bool success
  */
-function groups_delete_group_members($courseid, $userid=0, $showfeedback=false) {
+function groups_delete_group_members($courseid, $userid=0, $unused=false) {
     global $DB, $OUTPUT;
 
     // Get the users in the course which are in a group.
     global $DB, $OUTPUT;
 
     // Get the users in the course which are in a group.
@@ -598,10 +598,6 @@ function groups_delete_group_members($courseid, $userid=0, $showfeedback=false)
     $eventdata->userid   = $userid;
     events_trigger_legacy('groups_members_removed', $eventdata);
 
     $eventdata->userid   = $userid;
     events_trigger_legacy('groups_members_removed', $eventdata);
 
-    if ($showfeedback) {
-        echo $OUTPUT->notification(get_string('deleted').' - '.get_string('groupmembers', 'group'), 'notifysuccess');
-    }
-
     return true;
 }
 
     return true;
 }
 
@@ -1080,4 +1076,4 @@ function groups_sync_with_enrolment($enrolname, $courseid = 0, $gidfield = 'cust
     $rs->close();
 
     return $affectedusers;
     $rs->close();
 
     return $affectedusers;
-}
\ No newline at end of file
+}
index 8cedc33..c7d74ef 100644 (file)
@@ -31,6 +31,8 @@ information provided here is intended especially for developers.
     custom user filters. Similar deprecations in existing user_filter_* classes.
   * table_default_export_format_parent::table_default_export_format_parent() is
     deprecated, use parent::__construct() in extending classes.
     custom user filters. Similar deprecations in existing user_filter_* classes.
   * table_default_export_format_parent::table_default_export_format_parent() is
     deprecated, use parent::__construct() in extending classes.
+  * groups_delete_group_members() $showfeedback parameter has been removed and is no longer
+    respected. Users of this function should output their own feedback if required.
 
 === 3.0 ===
 
 
 === 3.0 ===