diff --git a/classes/lti_flow.php b/classes/lti_flow.php index aecc79f..2dc3517 100644 --- a/classes/lti_flow.php +++ b/classes/lti_flow.php @@ -154,8 +154,8 @@ public static function init_resource_link( string $deploymentid, string $moodleuserid, string $discussionurl, - ?string $groupid = null, - ?string $groupname = null + string $groupid, + string $groupname ): LtiMessageInterface { $context = context_module::instance($coursemoduleid); $roles = self::assign_lti_roles($context); diff --git a/tests/classes/lti_flow_test.php b/tests/classes/lti_flow_test.php index bdbe97c..4178c57 100644 --- a/tests/classes/lti_flow_test.php +++ b/tests/classes/lti_flow_test.php @@ -346,7 +346,15 @@ public function test_init_resource_link(): void { // Construct the initial LTI message sent to Kialo when the user clicks on the activity. $deploymentid = "random-string-123"; $discussionurl = "random-discussion-url.com"; - $message = lti_flow::init_resource_link($this->course->id, $this->cmid, $deploymentid, $this->user->id, $discussionurl); + $message = lti_flow::init_resource_link( + $this->course->id, + $this->cmid, + $deploymentid, + $this->user->id, + $discussionurl, + null, + null + ); $this->assertNotNull($message); $params = $message->getParameters()->jsonSerialize();