MDL-53832 enrol_lti: Update codes to use the new DB field names

mvaraujo · Oct 19, 2016 · b75d0f0 · b75d0f0
1 parent fa015ab
commit b75d0f0
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/enrol/lti/classes/tool_provider.php b/enrol/lti/classes/tool_provider.php
@@ -436,7 +436,7 @@ public function map_tool_to_consumer() {
         // Map the consumer to the tool.
         $mappingparams = [
             'toolid' => $this->tool->id,
-            'consumer_pk' => $this->consumer->getRecordId()
+            'consumerid' => $this->consumer->getRecordId()
         ];
         $mappingexists = $DB->record_exists('enrol_lti_tool_consumer_map', $mappingparams);
         if (!$mappingexists) {

diff --git a/enrol/lti/lib.php b/enrol/lti/lib.php
@@ -167,7 +167,7 @@ public function delete_instance($instance) {
         $dataconnector = new data_connector();
         foreach ($rsmapping as $mapping) {
             $consumer = new ToolConsumer(null, $dataconnector);
-            $consumer->setRecordId($mapping->consumer_pk);
+            $consumer->setRecordId($mapping->consumerid);
             $dataconnector->deleteToolConsumer($consumer);
         }
         $rsmapping->close();

diff --git a/enrol/lti/tests/lib_test.php b/enrol/lti/tests/lib_test.php
@@ -92,7 +92,7 @@ public function test_delete_instance() {
 
         $mappingparams = [
             'toolid' => $tool->id,
-            'consumer_pk' => $tp->consumer->getRecordId()
+            'consumerid' => $tp->consumer->getRecordId()
         ];
 
         // Check first that the related records exist.

diff --git a/enrol/lti/tests/tool_provider_test.php b/enrol/lti/tests/tool_provider_test.php
@@ -268,7 +268,7 @@ public function test_on_register() {
         // Check published tool and tool consumer mapping.
         $mappingparams = [
             'toolid' => $tool->id,
-            'consumer_pk' => $tp->consumer->getRecordId()
+            'consumerid' => $tp->consumer->getRecordId()
         ];
         $this->assertTrue($DB->record_exists('enrol_lti_tool_consumer_map', $mappingparams));
     }
@@ -506,7 +506,7 @@ public function test_map_tool_to_consumer() {
         // Check published tool and tool consumer mapping.
         $mappingparams = [
             'toolid' => $this->tool->id,
-            'consumer_pk' => $tp->consumer->getRecordId()
+            'consumerid' => $tp->consumer->getRecordId()
         ];
         $this->assertTrue($DB->record_exists('enrol_lti_tool_consumer_map', $mappingparams));
     }