Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
These are some minor tweaks needed to obtain Tool Provider Certification via the official IMS Global testing harness. A Patch will be included here to achieve this. They are very minor changes involving correct handling of error reporting and data validation.
Comment | File | Size | Author |
---|---|---|---|
#2 | imscompliance-2389303-2.patch | 10.62 KB | btopro |
#1 | imscompliance-2389303-1.patch | 4.56 KB | btopro |
Comments
Comment #1
btopro CreditAttribution: btopro commentedPatch checks for:
if its a good message
if its the correctly reported LTI version (not just if it has one)
adds roles for "Else" and "Alumni" which the harness has in its testing capabilities
report back to the launch_presentation_return_url if there is an error reported based on ill-formed message
context_id is not actually required for a launch based on the way the spec is formed through LTI Tool provider was enforcing it (commented out)
Comment #2
btopro CreditAttribution: btopro commentedfixed some notices, passed testing for 1.0 and 1.2 (draft) using this. I couldn't figure out the outcomes part of this module so didn't pass outcomes / 1.1.
Comment #3
darrenwh CreditAttribution: darrenwh as a volunteer and at Investis Digital commentedJust a few coding standard things here...
Newline missing at end of file
This block of code should be just deleted not commented out
Missing period at end of comment, Capital at beginning.
Missing newline at end of file.
Comment #4
darrenwh CreditAttribution: darrenwh as a volunteer and at Investis Digital commentedComment #5
kenianbei CreditAttribution: kenianbei commented