[MDL-35730] enrol_manual user selector doesn't use current context - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 2.1.8, 2.2.5, 2.3.2
Fix Version/s: 2.2.6, 2.3.3
Component/s: Enrolments
Labels:
- netspot
- partner
- patch
- triaged

Affected Branches:
MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
Fixed Branches:
MOODLE_22_STABLE, MOODLE_23_STABLE
Pull from Repository:
https://github.com/aolley/moodle.git
Pull Master Branch:
~~MDL-35730~~
Pull Master Diff URL:
https://github.com/aolley/moodle/compare/moodle:master...aolley:MDL-35730
Difficulty:
Easy
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.

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:: Adam Olley

Reporter:: Adam Olley

Peer reviewer:: Petr Skoda

Integrator:: Sam Hemelryk

Tester:: Tim Barker

Participants:: Adam Olley, Eloy Lafuente (stronk7), Petr Skoda, Sam Hemelryk, Tim Barker

Component watchers:: David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo

Votes:: 0 Vote for this issue

Watchers:: 0 Start watching this issue

Dates

Created:: 02/Oct/12 9:19 PM

Updated:: 15/Oct/12 3:08 PM

Resolved:: 15/Oct/12 3:08 PM

Fix Release Date:: 12/Nov/12