Moodle
  1. Moodle
  2. MDL-12296

Add option to disable unenrolment by external database enrolment plugin

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor 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

          Activity

          Howard Miller created issue -
          Hide
          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
          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.
          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
          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
          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...
          Martín Langhoff made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Hide
          Howard Miller added a comment -

          Thanks Martin.... on to it asap.

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

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

          Show
          Howard Miller added a comment - Sorted..... this time..... I hope!
          Howard Miller made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Martin Dougiamas made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          QA Assignee nobody
          Martin Dougiamas made changes -
          Workflow jira [ 23537 ] MDL Workflow [ 57954 ]
          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: