PM-46464 adjusting typing to appease bug in moodle-plugin-ci linting …

…rules

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);

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,
+            "",
+            ""
+        );
         $this->assertNotNull($message);
 
         $params = $message->getParameters()->jsonSerialize();

view.php

@@ -71,8 +71,8 @@
         $moduleinstance->deployment_id,
         $USER->id,
         $moduleinstance->discussion_url,
-        $groupinfo->groupid,
-        $groupinfo->groupname,
+        $groupinfo->groupid ?? "",
+        $groupinfo->groupname ?? "",
     );
 
     $output = $PAGE->get_renderer('mod_kialo');