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

The find users autocomplete in the Enrol user pop-up should respect get_extra_user_fields

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Setup

      1. Login as an admin.
      2. Navigate to "Users > Permissions > User policies " in site administration.
      3. Go to "Show user identity" and check, at least, City/town and Country checkboxes.
      4. Navigate to " Users > Accounts > Browse list of users" in site administration.
      5. Edit one user profile to make sure the City/town and Country fields are not empty.

      Testing scenario

      1. Login as a teacher.
      2. Go to the Course home page.
      3. Go to the Participants page.
      4. Click the 'Enrol users' button.
      5. Fill 'Select users' and search for the user with the filled City/town and Country fields.
      6. CHECK the City/town and Country information is displayed for the searched user (the one edited during the setup).
      Show
      Setup Login as an admin. Navigate to "Users > Permissions > User policies " in site administration. Go to "Show user identity" and check, at least, City/town and Country checkboxes. Navigate to " Users > Accounts > Browse list of users" in site administration. Edit one user profile to make sure the City/town and Country fields are not empty. Testing scenario Login as a teacher. Go to the Course home page. Go to the Participants page. Click the 'Enrol users' button. Fill 'Select users' and search for the user with the filled City/town and Country fields. CHECK the City/town and Country information is displayed for the searched user (the one edited during the setup).
    • Affected Branches:
      MOODLE_34_STABLE, MOODLE_35_STABLE, MOODLE_36_STABLE
    • Fixed Branches:
      MOODLE_35_STABLE, MOODLE_36_STABLE
    • Pull from Repository:
    • Pull 3.5 Branch:
    • Pull 3.6 Branch:
    • Pull Master Branch:
      MDL-64031-master

      Description

      The user fields shown in the Enrol users pop-up should respect the $CFG->showuseridentity setting, even if that includes columns that cannot be chosen in the admin UI. E.g. if you have $CFG->showuseridentity = 'idnumber,username'; in config.php.

      However, in https://github.com/moodle/moodle/blob/master/enrol/manual/amd/src/form-potential-user-selector.js#L81, there is a hard-coded list of user fields. That should not be.

      The way to access this list of fields to show in addtion to fullname($user) is get_extra_user_fields() in PHP code. That needs to be passed through to the JS somehow.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                thếvinhlê Vinh.Le
                Reporter:
                timhunt Tim Hunt
                Peer reviewer:
                Tim Hunt
                Integrator:
                Sara Arjona (@sarjona)
                Tester:
                Anna Carissa Sadia
                Participants:
                Component watchers:
                Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón, Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón, Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Mar/19

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 20 minutes
                  20m