Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-12296

Add option to disable unenrolment by external database enrolment plugin

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9, 2.0
    • Fix Version/s: 1.9, 2.0
    • Component/s: Enrolments
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      Add an option such that the plugin only adds users to courses but never removes them. This may be a requirement if (for example) courses need to be maintained for historical/archival purposes long after the students have ceased to be 'officially' enrolled in that course.

        Gliffy Diagrams

          Attachments

            Activity

            howardsmiller Howard Miller created issue -
            Hide
            howardsmiller Howard Miller added a comment -

            Option 'enrol_db_disableunenrol' has been added to external database configuration change. No check is made of course unenrollments if this is set to 'yes'. Default is 'no' - i.e., previous behavior.

            Show
            howardsmiller Howard Miller added a comment - Option 'enrol_db_disableunenrol' has been added to external database configuration change. No check is made of course unenrollments if this is set to 'yes'. Default is 'no' - i.e., previous behavior.
            howardsmiller Howard Miller made changes -
            Field Original Value New Value
            Status Open [ 1 ] Resolved [ 5 ]
            Fix Version/s 1.9 [ 10190 ]
            Fix Version/s 2.0 [ 10122 ]
            Resolution Fixed [ 1 ]
            Hide
            martinlanghoff Martín Langhoff added a comment -

            Hi Howard! Spotted 2 probs with the patch I see in cvs 19_STABLE...

            • enrol sync ignores the option (major)
            • there's a bogus "test()" method (minor, but untidy)

            reopening...

            Show
            martinlanghoff Martín Langhoff added a comment - Hi Howard! Spotted 2 probs with the patch I see in cvs 19_STABLE... enrol sync ignores the option (major) there's a bogus "test()" method (minor, but untidy) reopening...
            martinlanghoff Martín Langhoff made changes -
            Resolution Fixed [ 1 ]
            Status Resolved [ 5 ] Reopened [ 4 ]
            Hide
            howardsmiller Howard Miller added a comment -

            Thanks Martin.... on to it asap.

            Show
            howardsmiller Howard Miller added a comment - Thanks Martin.... on to it asap.
            Hide
            howardsmiller Howard Miller added a comment -

            Sorted..... this time..... I hope!

            Show
            howardsmiller Howard Miller added a comment - Sorted..... this time..... I hope!
            howardsmiller Howard Miller made changes -
            Status Reopened [ 4 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            dougiamas Martin Dougiamas made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            QA Assignee nobody
            dougiamas Martin Dougiamas made changes -
            Workflow jira [ 23537 ] MDL Workflow [ 57954 ]
            dougiamas Martin Dougiamas made changes -
            Workflow MDL Workflow [ 57954 ] MDL Full Workflow [ 87092 ]

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Mar/08