Skip to content


Merge branch 'MDL-57742_34' of git:// into M…
Browse files Browse the repository at this point in the history
  • Loading branch information
David Monllao committed May 28, 2018
2 parents 8359822 + 6c4ffec commit 3753139
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
4 changes: 3 additions & 1 deletion lib/ltiprovider/readme_moodle.txt
Expand Up @@ -5,9 +5,11 @@ Some changes from the upstream version have been made:
* Define consumer profile member variable for ToolConsumer class
* Added context type property for Context class
* Set context type if 'context_type' parameter was submitted through POST
These changes can be reverted once the following pull requests have been integrated upstream:
* Do not require tool_consumer_instance_guid
These changes can be reverted once the following pull requests/issues have been integrated upstream:

It is recommended by upstream to install depdencies via composer - but the composer installation is bundled
with an autoloader so it's better to do it manually.
Expand Down
5 changes: 0 additions & 5 deletions lib/ltiprovider/src/ToolProvider/ToolProvider.php
Expand Up @@ -808,11 +808,6 @@ private function authenticate()
if (!$this->ok) {
$this->reason = 'Request is from an invalid tool consumer.';
} else {
$this->ok = isset($_POST['tool_consumer_instance_guid']);
if (!$this->ok) {
$this->reason = 'A tool consumer GUID must be included in the launch request.';
if ($this->ok) {
Expand Down

0 comments on commit 3753139

Please sign in to comment.