Permalink
Browse files

MDL-37078: Restore - section restore fails with multiple availability…

… conditions
  • Loading branch information...
1 parent 0b7c52d commit 9789d825c2dafd39cee2f194ba241f173a4a16a1 @sammarshallou sammarshallou committed Dec 12, 2012
Showing with 4 additions and 4 deletions.
  1. +3 −3 backup/moodle2/restore_stepslib.php
  2. +1 −1 lib/conditionlib.php
@@ -1259,11 +1259,11 @@ public function after_restore() {
$sectionid = $this->get_task()->get_sectionid();
// Get data object for current section availability (if any).
- $data = $DB->get_record('course_sections_availability',
- array('coursesectionid' => $sectionid), 'id, sourcecmid, gradeitemid', IGNORE_MISSING);
+ $records = $DB->get_records('course_sections_availability',
+ array('coursesectionid' => $sectionid), 'id, sourcecmid, gradeitemid');
// If it exists, update mappings.
- if ($data) {
+ foreach ($records as $data) {
// Only update mappings for entries which are created by this restore.
// Otherwise, when you restore to an existing course, it will mess up
// existing section availability entries.
View
@@ -942,7 +942,7 @@ public function is_available(&$information, $grabthelot=false, $userid=0, $modin
if (empty($modinfo->cms[$cmid])) {
global $PAGE;
if (isset($PAGE) && strpos($PAGE->pagetype, 'course-view-')===0) {
- debugging("Warning: activity {$this->cm->id} '{$this->cm->name}' has condition " .
+ debugging("Warning: activity {$this->item->id} '{$this->item->name}' has condition " .
"on deleted activity $cmid (to get rid of this message, edit the named activity)");
}
continue;

0 comments on commit 9789d82

Please sign in to comment.