Permalink
Browse files

MDL-29350 Prevent duplication of groupings when copying activities

This patch also includes a database upgrade to correct data produced as a
result of this bug.
  • Loading branch information...
bostelm authored and andrewnicols committed Sep 13, 2011
1 parent 6be90ce commit 6336bd914a7adb446dc48b88a0f58bbcec795a17
Showing with 24 additions and 2 deletions.
  1. +8 −1 backup/moodle2/restore_stepslib.php
  2. +15 −0 lib/db/upgrade.php
  3. +1 −1 version.php
@@ -802,7 +802,14 @@ public function process_grouping_group($data) {
$data->groupingid = $this->get_new_parentid('grouping'); // Use new parentid
$data->groupid = $this->get_mappingid('group', $data->groupid); // Get from mappings
- $DB->insert_record('groupings_groups', $data); // No need to set this mapping (no child info nor files)
+
+ $params = array();
+ $params['groupingid'] = $data->groupingid;
+ $params['groupid'] = $data->groupid;
+
+ if (!$DB->record_exists('groupings_groups', $params)) {
+ $DB->insert_record('groupings_groups', $data); // No need to set this mapping (no child info nor files)
+ }
}
protected function after_execute() {
View
@@ -6916,6 +6916,21 @@ function xmldb_main_upgrade($oldversion) {
upgrade_main_savepoint(true, 2011110200.02);
}
+ if ($oldversion < 2011111500.01) {
+ // Remove duplicate entries from groupings_groups table
+ $sql = 'SELECT MIN(id) AS firstid, groupingid, groupid FROM {groupings_groups} '.
+ 'GROUP BY groupingid, groupid HAVING COUNT(id)>1';
+ $badrecs = $DB->get_records_sql($sql);
+ foreach ($badrecs as $badrec) {
+ $where = 'groupingid = ? and groupid = ? and id > ?';
+ $params = array($badrec->groupingid, $badrec->groupid, $badrec->firstid);
+ $DB->delete_records_select('groupings_groups', $where, $params);
+ }
+
+ // Main savepoint reached
+ upgrade_main_savepoint(true, 2011111500.01);
+ }
+
return true;
}
View
@@ -30,7 +30,7 @@
defined('MOODLE_INTERNAL') || die();
-$version = 2011111500.00; // YYYYMMDD = weekly release date of this DEV branch
+$version = 2011111500.01; // YYYYMMDD = weekly release date of this DEV branch
// RR = release increments - 00 in DEV branches
// .XX = incremental changes

0 comments on commit 6336bd9

Please sign in to comment.