Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
3.0.5, 3.1, 3.2
-
MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
-
MOODLE_30_STABLE, MOODLE_31_STABLE
-
MDL-55457-master -
Hide
Make your site publicly accessible
- Download ngrok https://ngrok.com/download
- Run ngrok ngrok http 80 --region=au
- Edit your config.php and set the wwwroot to use the ngrok provided url
New functionality test master and 3.1
- Go back to Site administration ▶︎ Plugins ▶︎ Activity modules ▶︎ External tool ▶︎ Manage tools
- Log in to http://prototype.moodle.net/lti (details on login page)
- Go to "Workplace induction" course
- Click Shared as LTI tool in Course administration
- Copy the url in the Proxy URL column for one of the shared tools
- Paste the URL in the box and click add
- There should be a loading animation
- Click register
- There should be a loading animation
- Make sure you are given a list of capabilities
- Confirm the capabilites
- Make sure the tool type was added successfully. It should have a broken icon.
- Edit the type (click the cog) ensure the icon url is present. It will be a broken/nonexistant url. That is expected. It's a work in progress branch not part of this issue.
Regression test master and 3.1
- Go back to Site administration ▶︎ Plugins ▶︎ Activity modules ▶︎ External tool ▶︎ Manage tools
- Paste http://lti.tools/test/tp.php in the box and click add
- There should be a loading animation
- Click on registration
- Select capabilities and press register
- There should be a loading animation
- Make sure you are given a list of capabilities
- Confirm the capabilites
- Make sure the tool type was added successfully. It should have an icon
- Edit the type (click the cog) ensure the icon url is present.
New functionality 3.0
- Log in to http://prototype.moodle.net/lti (details on login page)
- Go to "Workplace induction" course
- Click Shared as LTI tool in Course administration
- Copy the url in the Proxy URL column for one of the shared tools
- On your moodle 3.0 instance go to Site administration / ▶︎ Plugins / ▶︎ Activity modules / ▶︎ LTI / ▶︎ Manage external tool registrations
- Click "Configure a new external tool registration"
- Paste the URL in the registration URL box
- Select all capabilities and services
- On the Manage external tool registrations click the tick next to the tool you just added
- Make sure you are given a list of capabilities
- Confirm the capabilites
- Go to Site administration / ▶︎ Plugins / ▶︎ Activity modules / ▶︎ LTI / ▶︎ Manage external tool types
- Go to pending
- Click the tick next to the tool
- Make sure the tool type was added successfully. It should have a broken icon.
- Edit the type (click the cog) ensure the icon url is present. It will be a broken/nonexistant url. That is expected. It's a work in progress branch not part of this issue.
Regression test 3.0
- Go to Site administration / ▶︎ Plugins / ▶︎ Activity modules / ▶︎ LTI / ▶︎ Manage external tool registrations
- Click "Configure a new external tool registration"
- Paste http://lti.tools/test/tp.php in the registration URL box
- Select all capabilities and services
- On the Manage external tool registrations click the tick next to the tool you just added
- Select capabilities and press register
- There should be a loading animation
- Make sure you are given a list of capabilities
- Confirm the capabilites
- Make sure the tool type was added successfully. It should have an icon
- Edit the type (click the cog) ensure the icon url is present.
ShowMake your site publicly accessible Download ngrok https://ngrok.com/download Run ngrok ngrok http 80 --region=au Edit your config.php and set the wwwroot to use the ngrok provided url New functionality test master and 3.1 Go back to Site administration ▶︎ Plugins ▶︎ Activity modules ▶︎ External tool ▶︎ Manage tools Log in to http://prototype.moodle.net/lti (details on login page) Go to "Workplace induction" course Click Shared as LTI tool in Course administration Copy the url in the Proxy URL column for one of the shared tools Paste the URL in the box and click add There should be a loading animation Click register There should be a loading animation Make sure you are given a list of capabilities Confirm the capabilites Make sure the tool type was added successfully. It should have a broken icon. Edit the type (click the cog) ensure the icon url is present. It will be a broken/nonexistant url. That is expected. It's a work in progress branch not part of this issue. Regression test master and 3.1 Go back to Site administration ▶︎ Plugins ▶︎ Activity modules ▶︎ External tool ▶︎ Manage tools Paste http://lti.tools/test/tp.php in the box and click add There should be a loading animation Click on registration Select capabilities and press register There should be a loading animation Make sure you are given a list of capabilities Confirm the capabilites Make sure the tool type was added successfully. It should have an icon Edit the type (click the cog) ensure the icon url is present. New functionality 3.0 Log in to http://prototype.moodle.net/lti (details on login page) Go to "Workplace induction" course Click Shared as LTI tool in Course administration Copy the url in the Proxy URL column for one of the shared tools On your moodle 3.0 instance go to Site administration / ▶︎ Plugins / ▶︎ Activity modules / ▶︎ LTI / ▶︎ Manage external tool registrations Click "Configure a new external tool registration" Paste the URL in the registration URL box Select all capabilities and services On the Manage external tool registrations click the tick next to the tool you just added Make sure you are given a list of capabilities Confirm the capabilites Go to Site administration / ▶︎ Plugins / ▶︎ Activity modules / ▶︎ LTI / ▶︎ Manage external tool types Go to pending Click the tick next to the tool Make sure the tool type was added successfully. It should have a broken icon. Edit the type (click the cog) ensure the icon url is present. It will be a broken/nonexistant url. That is expected. It's a work in progress branch not part of this issue. Regression test 3.0 Go to Site administration / ▶︎ Plugins / ▶︎ Activity modules / ▶︎ LTI / ▶︎ Manage external tool registrations Click "Configure a new external tool registration" Paste http://lti.tools/test/tp.php in the registration URL box Select all capabilities and services On the Manage external tool registrations click the tick next to the tool you just added Select capabilities and press register There should be a loading animation Make sure you are given a list of capabilities Confirm the capabilites Make sure the tool type was added successfully. It should have an icon Edit the type (click the cog) ensure the icon url is present.
Description
According to the specification icon_info should be an object not an array.
https://www.imsglobal.org/lti/model/mediatype/application/vnd/ims/lti/v2/toolproxy%2Bjson/index.html
However since we've supported having it as array for so long and people may be using it that way, a simple check to see if it's an array or not should be enough
See service/toolproxy/classes/local/resource/toolproxy.php execute()