-
Improvement
-
Resolution: Duplicate
-
Minor
-
None
-
2.3.8, 2.4.5, 2.5, 2.6, 2.7
-
MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE
Current implementation makes bad assumptions to try to determine roles that are sent to LTI Tool Provider, like this one:
if (has_capability('moodle/course:manageactivities', $coursecontext)) {
|
array_push($roles, 'Instructor');
|
} else {
|
array_push($roles, 'Learner');
|
}
|
A better way to deal with this would use "role archetypes" and hardcode a relation between "archetype" and the corresponding "lti role".
Also I believe that "every role" should be passed, but prepending a Moodle's specific namespace. LTI specification doesn't prohibit us from using custom roles.
- has a non-specific relationship to
-
-
- Closed
-
- has been marked as being related by
-
MDL-47525 IMS roles cannot be retrieved without a current user
-
- Closed
-
- is duplicated by
-
-
- Closed
-