Extend Role Mapping for LTI Launches, NRPS Data, and Providers

Description

A robust LTI role mapping feature was added for outbound LTI Launches and NRPS calls in

There is no default mapping - it needs to be added to each tool in the Admin UI. Here is the LTI documentation on role mapping:

This Jira will add a default role mapping if no mapping is specified in the tool that will work in most cases.

This also completely updates the role configuration for the !site.template.lti which has not kept up with !site.template.course since it was created in ( inserted in 2012-02-16) - The new approach is to copy role information from !site.template.course and tweak it. As a side note, should we add a Guest role into the !site.template.course ?

Some of the work in is included in this Jira. If I found some portfolioesque cruft in the files being adjusted - I just took it out. As they say in “Office Space” - “we just fixed the glitch - the rest will take care of itself.”

This fix was designed to be back-portable to Sakai-22/21 - but it will require a conversion script - so the back port effort may not be worth it.

Earle - This needs a conversion before you back port it to 23. The lines fro here:

To the end of the file - need to be run before this is back-ported to 23.

This in effect is the first of two modifications. This modification ignores Plus but fixes the core LTI role mapping. A follow on Jira will plug all this mapping nicely into SakaiPlus. I kept the changes separate to allow for back porting of the mapping without pulling all of SakaiPlus into a branch.

Activity

Show:

Sam Ottenhoff March 7, 2024 at 9:38 PM

does it make sense to get this into 22.x?

Andrea Schmidt April 18, 2023 at 10:45 PM

Automation for Jira November 27, 2022 at 6:30 PM

A pull request has been created, "SAK-48085 - Rework role mapping for LTI and !site.template.lti", you can see it at https://github.com/sakaiproject/sakai/pull/11051

Fixed

Details

Priority

23 Status

QA Verification Pass

Components

Assignee

Reporter

Conversion Script Required

Yes

Created November 21, 2022 at 12:23 PM
Updated January 27, 2025 at 4:00 PM
Resolved November 29, 2022 at 4:28 PM