Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.0.4, 2.1.1, 2.2
-
Component/s: Backup
-
Labels:
-
Testing Instructions:
-
Workaround:
-
Affected Branches:MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
-
Fixed Branches:MOODLE_20_STABLE, MOODLE_21_STABLE
-
Pull from Repository:
-
Pull Master Branch:
-
Pull Master Diff URL:
Description
At the moment, a user cannot import course contents from course A into course B unless he has the capability moodle/restore:restorecourse in the context of the source course.
However, according to MDL-24518, this should be controlled by the capability moodle/backup:backuptargetimport .
To reproduce, on a Moodle installation in standard setup:
- Create two courses, called "source" and "target". Create some labels in "source" (as example content).
- On "source", allow the following capabilities for the non-editing teacher role:
- moodle/backup:backupcourse
- moodle/backup:configure
- moodle/backup:backuptargetimport
- Create a user "U" and enrol the user as follows:
- into "source" as non-editing teacher,
- into "target" as editing teacher.
- Log in as "U".
- Go to "target" and click on "Import".
EXPECTED: The list of courses displayed contains the course "source".
ACTUAL: The list of courses displayed contains only the course "target".
The underlying problem is that the list is filtered by courses on which the user "U" has the capability moodle/restore:restorecourse . This is done in the constructor of class import_course_search, which is inherited without modification from class restore_course_search.
Patch will follow.