Moodle

Course page blocks are lost when restoring 1.3 and 1.4 backups

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.2
  • Fix Version/s: 1.9.3
  • Component/s: Backup
  • Labels:
    None
  • Environment:
    Moodle 1.9.x restoring 1.4 backup file
  • Database:
    Any
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

Uhm...

was working with some old (Moodle 1.4) backups.... trying to restore them to 1.8 and 1.9... and under 1.9, course blocks are lost.

Reviewing revisions I've arrived to MDL-8270.

It seems that the

if (!empty($CFG->showblocksonmodpages))

condition, is being applied when the course is an old one (containing course->blockinfo, instead of instances). I think that, in those backups, there are only PAGE_COURSE_VIEW blocks so, the check of $CFG->showblocksonmodpages isn't correct there at all.

Instead, it should be (and there it's) in the restore_create_block_instances() method, that is the one used when we have block instances from different pages.

So... I'm going to delete that condition when restoring old courses.

Ciao

Issue Links

Activity

Hide
Eloy Lafuente (stronk7) added a comment -

Resolving as fixed.

Show
Eloy Lafuente (stronk7) added a comment - Resolving as fixed.
Hide
Petr Škoda (skodak) added a comment -

makes sense, closing
thanks

Show
Petr Škoda (skodak) added a comment - makes sense, closing thanks

People

Dates

  • Created:
    Updated:
    Resolved: