L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'MDL-87966-main' of https://github.com/aanabit/moodle

Browse Source
This commit is contained in:
Sara Arjona
2026-03-09 17:18:45 +01:00
parent 1ad9883a6b acbeb4e4ab
commit 46497e5efd
4 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
public/lib/classes/router/parameters/coursemodule_parameter_trait.php
+2 -1
View File
@@ -108,7 +108,8 @@ trait coursemodule_parameter_trait {
return $request
->withAttribute($this->get_value_name('data'), $data)
->withAttribute($this->get_value_name('context'), \core\context\module::instance($data->id));
->withAttribute($this->get_value_name('context'), \core\context\module::instance($data->id))
->withAttribute('course', $data->course);
}
/**
public/lib/classes/router/parameters/path_section.php
+1 -1
View File
@@ -69,7 +69,7 @@ class path_section extends \core\router\schema\parameters\path_parameter impleme
$section = $this->get_section_for_value($value);
return $request
->withAttribute($this->name, $section)
->withAttribute("coursecontext", \context_course::instance($section->course));
->withAttribute("course", $section->course);
}
/**
public/lib/tests/router/parameters/path_coursemodule_test.php
+1
View File
@@ -46,6 +46,7 @@ final class path_coursemodule_test extends route_testcase {
$this->assertEquals($mod->cmid, $newrequest->getAttribute('coursemoduledata')->id);
$this->assertEquals($modcontext->id, $newrequest->getAttribute('coursemodulecontext')->id);
$this->assertEquals($course->id, $newrequest->getAttribute('course'));
}
/**
public/lib/tests/router/parameters/path_section_test.php
+2 -6
View File
@@ -27,8 +27,8 @@ use stdClass;
* @package core
* @copyright Amaia Anabitarte <[email protected]>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @covers \core\router\parameters\path_section
*/
#[CoversClass(\core\router\parameters\path_section::class)]
final class path_section_test extends route_testcase {
public function test_section_id(): void {
$this->resetAfterTest();
@@ -36,17 +36,14 @@ final class path_section_test extends route_testcase {
$course = $this->getDataGenerator()->create_course();
$modinfo = get_fast_modinfo($course);
$section = $modinfo->get_section_info(1);
$context = \context_course::instance($course->id);
$param = new path_section();
$request = new ServerRequest('GET', '/course/sections/' . $section->id . '/restricted');
$newrequest = $param->add_attributes_for_parameter_value($request, $section->id);
$this->assertInstanceOf(stdClass::class, $newrequest->getAttribute('section'));
$this->assertInstanceOf(\core\context\course::class, $newrequest->getAttribute('coursecontext'));
$this->assertEquals($section->id, $newrequest->getAttribute('section')->id);
$this->assertEquals($context->id, $newrequest->getAttribute('coursecontext')->id);
$this->assertEquals($course->id, $newrequest->getAttribute('course'));
}
/**
@@ -57,7 +54,6 @@ final class path_section_test extends route_testcase {
$param = new path_section();
$course = $this->getDataGenerator()->create_course();
$sectionid = 9999;
$request = new ServerRequest('GET', '/course/sections/' . $sectionid . '/restricted');