MDL-69577 forum: Add courseid, forumid on get_discussion_posts response

moodle · Sep 1, 2020 · 150913d · 150913d
1 parent 38abfb6
commit 150913d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/mod/forum/externallib.php b/mod/forum/externallib.php
@@ -226,6 +226,8 @@ public static function get_discussion_posts(int $discussionid, ?string $sortby,
 
         return [
             'posts' => $postbuilder->build($USER, [$forum], [$discussion], $posts),
+            'forumid' => $discussion->get_forum_id(),
+            'courseid' => $discussion->get_course_id(),
             'ratinginfo' => \core_rating\external\util::get_rating_info(
                 $legacydatamapper->get_forum_data_mapper()->to_legacy_object($forum),
                 $forum->get_context(),
@@ -258,6 +260,8 @@ public static function get_discussion_posts_parameters() {
     public static function get_discussion_posts_returns() {
         return new external_single_structure([
             'posts' => new external_multiple_structure(\mod_forum\local\exporters\post::get_read_structure()),
+            'forumid' => new external_value(PARAM_INT, 'The forum id'),
+            'courseid' => new external_value(PARAM_INT, 'The forum course id'),
             'ratinginfo' => \core_rating\external\util::external_ratings_structure(),
             'warnings' => new external_warnings()
         ]);

diff --git a/mod/forum/tests/externallib_test.php b/mod/forum/tests/externallib_test.php
@@ -660,6 +660,8 @@ public function test_mod_forum_get_discussion_posts() {
         // Create what we expect to be returned when querying the discussion.
         $expectedposts = array(
             'posts' => array(),
+            'courseid' => $course1->id,
+            'forumid' => $forum1->id,
             'ratinginfo' => array(
                 'contextid' => $forum1context->id,
                 'component' => 'mod_forum',