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

enrol_manual user selector doesn't use current context

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      1) Have a non-admin user assigned at the course or category level of the course you're testing with.
      2) Have the role they're assigned to at that course/cat level have the moodle/site:viewuseridentity capability (editingteacher for example has this at course level).
      3) Make sure site config is set to show extra fields in the user selector ($CFG->showuseridentity)
      4) When logged in as this user with this patch applied, the extra fields should be visible.

      Show
      1) Have a non-admin user assigned at the course or category level of the course you're testing with. 2) Have the role they're assigned to at that course/cat level have the moodle/site:viewuseridentity capability (editingteacher for example has this at course level). 3) Make sure site config is set to show extra fields in the user selector ($CFG->showuseridentity) 4) When logged in as this user with this patch applied, the extra fields should be visible.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      The manual enrolment user selectors base the viewing of identities off of the site context.

      This fix has the selectors informed of the current (course) context so that the useridentity check can be checked for users that only have that ability at a category (or course) level.

        Attachments

          Activity

            People

            Assignee:
            aolley Adam Olley
            Reporter:
            aolley Adam Olley
            Peer reviewer:
            Petr Skoda
            Integrator:
            Sam Hemelryk
            Tester:
            Tim Barker
            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:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              12/Nov/12