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

LTI Consumer should send more user roles

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Reopened
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.3.8, 2.4.5, 2.5, 2.6, 2.7
    • Fix Version/s: None
    • Labels:
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE

      Description

      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.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              brodock Gabriel Mazetto
              Peer reviewer:
              Adrian Greeve
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              5 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated: