Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-53832 Upgrade enrol_lti to support LTI v2.0
  3. MDL-55718

Support user images in new enrol_lti

    XMLWordPrintable

Details

    • Sub-task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.2
    • None
    • Enrolments, LTI provider
    • MOODLE_32_STABLE
    • MDL-55718-master
    • Hide
      1. Apply the patch from MDL-37041 to a new moodle instance (site C)
      2. Go to site A and copy the cartridge url and secret
      3. On site C, create a new external tool instance and paste the cartridge url and secret
      4. Make sure the user image is visible
      Show
      Apply the patch from MDL-37041 to a new moodle instance (site C) Go to site A and copy the cartridge url and secret On site C, create a new external tool instance and paste the cartridge url and secret Make sure the user image is visible

    Description

      Previously the lti provider supported user_images, but it has not been implemented yet using the new library. We need to get and set the user's image to maintain feature continuity between 3.1 and 3.2

      We will need to pull request library and get ToolProvider to use $CUSTOM_SUBSTITUTION_VARIABLES. At the moment it ignores them. Adding a simple for loop going through $CUSTOM_SUBSTITUTION_VARIABLES and adding them to settings will be enough

      Attachments

        Issue Links

          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-37041 The LTI consumer does not pass the user's profile picture in the user_image launch parameter

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              johno John Okely
              johno John Okely
              Jun Pataleta Jun Pataleta
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: