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

Allow Non-Enrolled "Other Users" Role Assignments to be set by "Enrol" Database Plugin

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.13, 2.2, 2.3.3, 2.6.2, 2.7, 2.8
    • Fix Version/s: 2.8
    • Component/s: Enrolments
    • Labels:
    • Testing Instructions:
      Hide
      1. Add moodle/course:view permission to roles you plan to test with
      2. Add column to external database table
      3. Set configuration for database plugin
      4. Add rows to external database table with flag set
      5. Login as a user to test sync_user_enrolments()
      6. Run database plugin CLI to test sync_enrolments()
      7. Verify that "Other Users" are not listed as participants, but do have access to the course
      Show
      Add moodle/course:view permission to roles you plan to test with Add column to external database table Set configuration for database plugin Add rows to external database table with flag set Login as a user to test sync_user_enrolments() Run database plugin CLI to test sync_enrolments() Verify that "Other Users" are not listed as participants, but do have access to the course
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-28420_master

      Description

      It would be very useful if the remote table that provides role assignment information to Moodle could have an optional column which Moodle uses to determine if the user should not be enrolled. Otherwise, automatically "enrolled" users incorrectly show up in Participants when they are intended to be "Other Users". This is an important lack of functionality, for which there is no automated workaround.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              jrchamp Jonathan Champ
              Reporter:
              jrchamp Jonathan Champ
              Peer reviewer:
              Damyon Wiese
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Jetha Chan
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Nov/14