Moodle
  1. Moodle
  2. MDL-26841

Import includes enrolment methods

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: 2.0.1, 2.1.1, 2.2
        • Fix Version/s: 2.0.5, 2.1.2
        • Component/s: Backup
        • Labels:
          None
        • Database:
          MySQL
        • Testing Instructions:
          Hide

          1) Create 2 courses: "source" and "target"
          2) In "source" add various enrolment plugins (manual, guest, self...) and add 1-2 manual enrolments (students). Add also 1 forum and ignite some discussions with those students.
          3) In "target" add manual enrolment only with 1 student (different student from the ones in point 2).
          4) Logged as admin, perform one import in "target", picking "source" as origin. Select the forum activity and import.
          5) TEST: The operations ends without error.
          6) TEST: The forum activity has been imported but without any discussion.
          7) TEST: Look at "target" enrolment methods. Only the original "manual" is there and none of the enrolment methods in "source" have been imported.

          Show
          1) Create 2 courses: "source" and "target" 2) In "source" add various enrolment plugins (manual, guest, self...) and add 1-2 manual enrolments (students). Add also 1 forum and ignite some discussions with those students. 3) In "target" add manual enrolment only with 1 student (different student from the ones in point 2). 4) Logged as admin, perform one import in "target", picking "source" as origin. Select the forum activity and import. 5) TEST: The operations ends without error. 6) TEST: The forum activity has been imported but without any discussion. 7) TEST: Look at "target" enrolment methods. Only the original "manual" is there and none of the enrolment methods in "source" have been imported.
        • Difficulty:
          Easy
        • 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:
        • Rank:
          16504

          Description

          When importing resources enrolment methods are imported as well causing duplicate and not wanted enrolment methods.

            Issue Links

            caused a regression

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-29618 Enrol methods and enrolments not being restored anymore

            • Critical - Crashes server, loss of data, severe memory leak
            • Closed
            has a non-specific relationship to

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-27061 Restoring backup with option "Merge the backup course into this course" causes duplicate section "Guest access" in "Edit course settings" form.

            • Minor - Minor loss of function where workaround is possible
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Adam Olley added a comment -

              Fix I've implemented so far for this is to simply not perform the enrolment restore step when in MODE_IMPORT:
              https://github.com/aolley/moodle/commit/99f451a9f884d4ee73d786be523b5f0ae3cece6d

              Probably beneficial to do similar to the backup steps as well to not back up the enrolments in the first place when in MODE_IMPORT.

              Show
              Adam Olley added a comment - Fix I've implemented so far for this is to simply not perform the enrolment restore step when in MODE_IMPORT: https://github.com/aolley/moodle/commit/99f451a9f884d4ee73d786be523b5f0ae3cece6d Probably beneficial to do similar to the backup steps as well to not back up the enrolments in the first place when in MODE_IMPORT.
              Hide
              Permalink
              Bryant Saxon added a comment -

              Do you know where the enrolment info is stored in the DB? I would like to try to manually fix some of my courses that are effected by this bug.

              Here's how I am noticing it – after we import some aspects of one course into another, the users/students are seeing double "Enrol Me" buttons after clicking a course from the main menu (for courses that they are not enrolled). Obviously, we are using self enrolment also.

              Show
              Bryant Saxon added a comment - Do you know where the enrolment info is stored in the DB? I would like to try to manually fix some of my courses that are effected by this bug. Here's how I am noticing it – after we import some aspects of one course into another, the users/students are seeing double "Enrol Me" buttons after clicking a course from the main menu (for courses that they are not enrolled). Obviously, we are using self enrolment also.
              Hide
              Permalink
              Bryant Saxon added a comment -

              Nevermind! I found it. It's in the mdl_enrol table. I just sorted by course ID and looked for duplicates of the same type and removed the latest entry.

              Show
              Bryant Saxon added a comment - Nevermind! I found it. It's in the mdl_enrol table. I just sorted by course ID and looked for duplicates of the same type and removed the latest entry.
              Hide
              Permalink
              Mark Ward added a comment -

              An additional note, if you import from a course which has guest enrolment installed but disabled (which is the default) you end up with multiple instances of Guest Enrolment. I have just had to delete 45 instances from 6 courses, this shouldn't be possible!

              Show
              Mark Ward added a comment - An additional note, if you import from a course which has guest enrolment installed but disabled (which is the default) you end up with multiple instances of Guest Enrolment. I have just had to delete 45 instances from 6 courses, this shouldn't be possible!
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              Note that there are really two issues here:

              1) Backup and restore handling IMPORT/HUB operations should not include enrolments information at all.

              This is going to be fixed by this, by radically avoiding enrol information to be processed under those modes.

              2) Some incorrect duplication checks lead to multiple instances of enrol plugins.

              This has been addressed @ MDL-29414 and will help fixing some issues like MDL-27061

              Said that, working on 1) right now.

              Show
              Eloy Lafuente (stronk7) added a comment - Note that there are really two issues here: 1) Backup and restore handling IMPORT/HUB operations should not include enrolments information at all. This is going to be fixed by this, by radically avoiding enrol information to be processed under those modes. 2) Some incorrect duplication checks lead to multiple instances of enrol plugins. This has been addressed @ MDL-29414 and will help fixing some issues like MDL-27061 Said that, working on 1) right now.
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              Sending to integration for 20, 21 and master.

              The fix prevents any enrolments information to be handled in any IMPORT and HUB operations + add one extra check to verify that the "enrolments.xml" is available before processing it on restore (this is necessary because that file, after this change, has become "optional").

              Ciao

              Show
              Eloy Lafuente (stronk7) added a comment - Sending to integration for 20, 21 and master. The fix prevents any enrolments information to be handled in any IMPORT and HUB operations + add one extra check to verify that the "enrolments.xml" is available before processing it on restore (this is necessary because that file, after this change, has become "optional"). Ciao
              Hide
              Permalink
              Sam Hemelryk added a comment -

              Thanks everyone, this has been integrated now and is up for testing.

              Cheers
              Sam

              Show
              Sam Hemelryk added a comment - Thanks everyone, this has been integrated now and is up for testing. Cheers Sam
              Hide
              Permalink
              Rossiani Wijaya added a comment -

              This works great.

              Test passed.

              ps: Thanks Aparup for testing this issue too but not fast enough to click the 'start testing' button

              Show
              Rossiani Wijaya added a comment - This works great. Test passed. ps: Thanks Aparup for testing this issue too but not fast enough to click the 'start testing' button
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              And this code has been spread to all Moodle git and cvs repositories. Many thanks! Closing.

              Ciao

              Show
              Eloy Lafuente (stronk7) added a comment - And this code has been spread to all Moodle git and cvs repositories. Many thanks! Closing. Ciao

                People

                • Votes:
                  4 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: