[MDL-61576] The lti_build_custom_parameters function does not contain all necessary params - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.3.4, 3.4, 3.5
Fix Version/s: 3.3.6, 3.4.3
Component/s: External Tool (IMS-LTI)
Labels:
- ci
- triaged

Testing Instructions:
1. No testing instructions required. Sanity check the diff. No additional code is being executed, we just removed an if statement that was always returning true.
Affected Branches:
MOODLE_33_STABLE, MOODLE_34_STABLE, MOODLE_35_STABLE
Fixed Branches:
MOODLE_33_STABLE, MOODLE_34_STABLE
Pull from Repository:
git://github.com/cescobedo/moodle.git
Pull Master Branch:
~~MDL-61576~~-master
Pull Master Diff URL:
https://github.com/cescobedo/moodle/compare/b63a3b04b1...MDL-61576-master

Description

The variable $typeconfig is never passed, but is used in the function. The only reason it doesn't break is because we use !isset($typeconfig['allowinstructorcustom']) and isset() won't complain. This was discovered reviewing ~~MDL-60416~~ which will address this issue in 3.5 (assuming it gets integrated), but this still needs fixing in prior supported branches.

Attachments

Issue Links

blocks

MDL-60416 Support LTI Advantage 1.1

Closed

Discovered while testing

MDL-60416 Support LTI Advantage 1.1

Closed

Activity

People

Assignee:

Carlos Escobedo

Reporter:

Mark Nelson

Peer reviewer:

Mark Nelson

Integrator:

Eloy Lafuente (stronk7)

Tester:

Nobody

Participants:

Andrew Nicols, Carlos Escobedo, chrislawson, CiBoT, Eloy Lafuente (stronk7), Mark Nelson

Component watchers:

Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias

Votes:

0 Vote for this issue

Watchers:

5 Start watching this issue

Dates

Created:

02/Mar/18 8:05 PM

Updated:

30/Mar/18 1:38 AM

Resolved:

30/Mar/18 1:38 AM

Fix Release Date:

17/May/18