From fd63451ecc8506dfc30019f7e4d11f6cc2675a7b Mon Sep 17 00:00:00 2001 From: Paul Holden Date: Thu, 18 Jul 2024 08:49:42 +0100 Subject: [PATCH] MDL-82525 user: correct course parameter for course profile page. --- lib/classes/user.php | 4 ++-- lib/tests/user_test.php | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/classes/user.php b/lib/classes/user.php index 77ab9304f54..a125173e170 100644 --- a/lib/classes/user.php +++ b/lib/classes/user.php @@ -1417,11 +1417,11 @@ class core_user { // Get courseid from context if provided. if ($context && $coursecontext = $context->get_course_context(false)) { - $params['courseid'] = $coursecontext->instanceid; + $params['course'] = $coursecontext->instanceid; } // If courseid is not set or is set to site id, then return profile page, otherwise return view page. - if (!isset($params['courseid']) || $params['courseid'] == SITEID) { + if (!isset($params['course']) || $params['course'] == SITEID) { return new moodle_url('/user/profile.php', $params); } else { return new moodle_url('/user/view.php', $params); diff --git a/lib/tests/user_test.php b/lib/tests/user_test.php index 182f49cd6d6..8d151974171 100644 --- a/lib/tests/user_test.php +++ b/lib/tests/user_test.php @@ -892,7 +892,7 @@ class user_test extends \advanced_testcase { // Display profile url at course context. $course = $this->getDataGenerator()->create_course(); $coursecontext = \context_course::instance($course->id); - $this->assertEquals("https://www.example.com/moodle/user/view.php?id={$user->id}&courseid={$course->id}", + $this->assertEquals("https://www.example.com/moodle/user/view.php?id={$user->id}&course={$course->id}", \core_user::get_profile_url($user, $coursecontext)); // Throw error if userid is invalid.