Moodle
  1. Moodle
  2. MDL-30211

IMS Enterprise created courses not getting default enrolment methods

    Details

    • Rank:
      26465

      Description

      When courses are created by IMS Enterprise, the default enrolment methods (like manual enrolment) are not enabled in the course.

      Replication steps:

      1. Set Manual Enrolments to be added to all new courses.
      2. Site Administration --> Plugins --> Enrolments --> Manual Enrolments (Check Add instance to new courses and save)
      3. Configure IMS Enterprise, use attached file - IMS_Course.xml to create a course.
      4. Check enrolment methods, note, manual enrolments not there.

      Apply patch, repeat above process. Note manual enrolments now available

      1. IMS_Course.xml
        0.5 kB
        Adam Barbary
      2. IMS_enrol_defaults2_1.patch
        0.5 kB
        Adam Barbary

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          Thanks for reporting that and providing a patch.

          Show
          Michael de Raadt added a comment - Thanks for reporting that and providing a patch.
          Hide
          Michael de Raadt added a comment -

          On the surface this appears to be the same issue, but looking at your patch it is quite different to the one currently in integration. It might be worth checking of the linked duplicate resolves your problem.

          Show
          Michael de Raadt added a comment - On the surface this appears to be the same issue, but looking at your patch it is quite different to the one currently in integration. It might be worth checking of the linked duplicate resolves your problem.
          Hide
          Adam Barbary added a comment -

          Hi Michael,

          This is actually a separate issue. The other patch, which I also submitted, contains fixes for the default course setup. Number of topics, format, availability, upload size etc. This patch addresses enrolment methods. I concede that they apply to the same plugin, and could be merged into the one fix, but they are separate issues.

          Cheers,
          Adam

          Show
          Adam Barbary added a comment - Hi Michael, This is actually a separate issue. The other patch, which I also submitted, contains fixes for the default course setup. Number of topics, format, availability, upload size etc. This patch addresses enrolment methods. I concede that they apply to the same plugin, and could be merged into the one fix, but they are separate issues. Cheers, Adam
          Hide
          Adam Barbary added a comment -

          Hi Michael, could you please reopen this bug, as it is different to the other? It involves enrolment methods, which are assigned differently to the standard course configuration defaults.
          Cheers,
          Adam

          Show
          Adam Barbary added a comment - Hi Michael, could you please reopen this bug, as it is different to the other? It involves enrolment methods, which are assigned differently to the standard course configuration defaults. Cheers, Adam
          Hide
          Sam Hemelryk added a comment -

          Thanks Dan - this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Dan - this has been integrated now
          Hide
          Ankit Agarwal added a comment -

          Hi Dan,
          Am not sure how to follow the step 3
          "Configure IMS Enterprise, use attached file - IMS_Course.xml to create a course."

          Can you please elaborate the testing instructions.
          Thanks

          Show
          Ankit Agarwal added a comment - Hi Dan, Am not sure how to follow the step 3 "Configure IMS Enterprise, use attached file - IMS_Course.xml to create a course." Can you please elaborate the testing instructions. Thanks
          Hide
          Ankit Agarwal added a comment -

          Working great!
          Thanks

          Show
          Ankit Agarwal added a comment - Working great! Thanks
          Hide
          Adam Barbary added a comment -

          Sorry Ankit, timezones are a real nuisance. Looks like you have it figured now though. Am excited that the IMS Enterprise module is getting some well deserved attention.

          Show
          Adam Barbary added a comment - Sorry Ankit, timezones are a real nuisance. Looks like you have it figured now though. Am excited that the IMS Enterprise module is getting some well deserved attention.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Yes, you did it!

          Now your code is part of the best weeklies released ever, many thanks!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Yes, you did it! Now your code is part of the best weeklies released ever, many thanks! Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: