MDL-15372 - Links in course summary and section summaries not recoded on restore. Petr thinks it is a bad idea to recode links in course summaries, becuase he thinks there should not be links there at all, but I disagree, and as long as links are there, and we have the code to stop them breaking on backup and restore, we should fix the bug.
This commit is contained in:
@@ -93,6 +93,42 @@ define('RESTORE_GROUPS_GROUPINGS', 3);
|
||||
echo "<ul>";
|
||||
}
|
||||
|
||||
// Recode links in the course summary.
|
||||
if (!defined('RESTORE_SILENTLY')) {
|
||||
echo '<li>' . get_string('from') . ' ' . get_string('course');
|
||||
}
|
||||
$course = get_record('course', 'id', $restore->course_id, '', '', '', '', 'id,summary');
|
||||
$coursesummary = restore_decode_content_links_worker($course->summary, $restore);
|
||||
if ($coursesummary != $course->summary) {
|
||||
$course->summary = addslashes($coursesummary);
|
||||
if (!update_record('course', $course)) {
|
||||
$status = false;
|
||||
}
|
||||
}
|
||||
if (!defined('RESTORE_SILENTLY')) {
|
||||
echo '</li>';
|
||||
}
|
||||
|
||||
// Recode links in section summaries.
|
||||
$sections = get_records('course_sections', 'course', $restore->course_id, 'id', 'id,summary');
|
||||
if ($sections) {
|
||||
if (!defined('RESTORE_SILENTLY')) {
|
||||
echo '<li>' . get_string('from') . ' ' . get_string('sections');
|
||||
}
|
||||
foreach ($sections as $section) {
|
||||
$sectionsummary = restore_decode_content_links_worker($section->summary, $restore);
|
||||
if ($sectionsummary != $section->summary) {
|
||||
$section->summary = addslashes($sectionsummary);
|
||||
if (!update_record('course_sections', $section)) {
|
||||
$status = false;
|
||||
}
|
||||
}
|
||||
}
|
||||
if (!defined('RESTORE_SILENTLY')) {
|
||||
echo '</li>';
|
||||
}
|
||||
}
|
||||
|
||||
// Restore links in modules.
|
||||
foreach ($restore->mods as $name => $info) {
|
||||
//If the module is being restored
|
||||
|
||||
Reference in New Issue
Block a user