MDL-55022 mod_lti: Fix undefined variable

diff --git a/mod/lti/classes/external.php b/mod/lti/classes/external.php
index fca7c77..e6d5181 100644 (file)
--- a/mod/lti/classes/external.php
+++ b/mod/lti/classes/external.php
@@ -523,7 +523,7 @@ class mod_lti_external extends external_api {
         }
 
         if (!empty($capabilityoffered)) {
-            $config->lti_capabilities = $capabilitiesoffered;
+            $config->lti_capabilities = $capabilityoffered;
         }
 
         if (!empty($serviceoffered)) {

diff --git a/mod/lti/tests/externallib_test.php b/mod/lti/tests/externallib_test.php
index b3ba4ea..a7f0323 100644 (file)
--- a/mod/lti/tests/externallib_test.php
+++ b/mod/lti/tests/externallib_test.php
@@ -285,10 +285,12 @@ class mod_lti_external_testcase extends externallib_advanced_testcase {
      * Test create tool proxy
      */
     public function test_mod_lti_create_tool_proxy() {
-        $proxy = mod_lti_external::create_tool_proxy('Test proxy', $this->getExternalTestFileUrl('/test.html'), array(), array());
+        $capabilities = ['AA', 'BB'];
+        $proxy = mod_lti_external::create_tool_proxy('Test proxy', $this->getExternalTestFileUrl('/test.html'), $capabilities, []);
         $this->assertEquals('Test proxy', $proxy->name);
         $this->assertEquals($this->getExternalTestFileUrl('/test.html'), $proxy->regurl);
         $this->assertEquals(LTI_TOOL_PROXY_STATE_PENDING, $proxy->state);
+        $this->assertEquals(implode("\n", $capabilities), $proxy->capabilityoffered);
     }
 
     /*