Course ID should be checked and retained during backup and restore / copy course

[cjbsdigitallearning]

Hi

Great plugin. Also great that it supports the display of a section form a different course. However I think it can be improved by the following.

We plan to use the section block to show frequently accesses item on every course page, so for us the block will show the same or very similar content in each course. Course ID will always be the current course not a different one. We use a template course which is then used to create each new course, of which there are hundreds. We've encountered an issue in that the course ID is not updated to reflect the new course ID once restored, meaning that we'd need to manually update the Course ID in every course.

I can understand this behaviour wouldn't be desirable if displaying a section form a different course, so here what I propose:

During backup and restore / copy the Course ID in the Section block should be checked, if it matches the course being processed, then it should be updated to reflect the new course. If not it should retain the current behaviour on remain unchanged.

Hope this makes sense, happy to discuss further.

Best wishes
Neil

[ak4t0sh]

Hi,

Thanks for the feedback :) and sorry for the late answer.
You are right current course id should be updated during the backup/restore process.
Currently I do not have a lot of time to work on it but if you are able to do the developments do not hesitate to do a PR and I'll review it :)

[SimonThornett]

Morning!

I have created a pull request that resolves the issue Nail mentioned by adding the restore_section_block_task and updating the courseid in the block config.

Kind regards,
Simon T

[SimonThornett]

Just to update and sat that the block restore has been updated to only process config records where the original courseid and config->course match.

Kind regards,
Simon T