From 5f9997ae959674e231c9eb9495af25bec0efb22f Mon Sep 17 00:00:00 2001 From: Andrew Nicols Date: Wed, 21 Jan 2015 12:13:26 +0800 Subject: [PATCH] MDL-48720 mod_forum: Tidy up whitespace to comply with coding style --- mod/forum/discuss.php | 562 +++++++++++++++++++++--------------------- 1 file changed, 281 insertions(+), 281 deletions(-) diff --git a/mod/forum/discuss.php b/mod/forum/discuss.php index 15f1a8e8f37..6fe3bbf3447 100644 --- a/mod/forum/discuss.php +++ b/mod/forum/discuss.php @@ -24,346 +24,346 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ - require_once('../../config.php'); +require_once('../../config.php'); - $d = required_param('d', PARAM_INT); // Discussion ID - $parent = optional_param('parent', 0, PARAM_INT); // If set, then display this post and all children. - $mode = optional_param('mode', 0, PARAM_INT); // If set, changes the layout of the thread - $move = optional_param('move', 0, PARAM_INT); // If set, moves this discussion to another forum - $mark = optional_param('mark', '', PARAM_ALPHA); // Used for tracking read posts if user initiated. - $postid = optional_param('postid', 0, PARAM_INT); // Used for tracking read posts if user initiated. +$d = required_param('d', PARAM_INT); // Discussion ID +$parent = optional_param('parent', 0, PARAM_INT); // If set, then display this post and all children. +$mode = optional_param('mode', 0, PARAM_INT); // If set, changes the layout of the thread +$move = optional_param('move', 0, PARAM_INT); // If set, moves this discussion to another forum +$mark = optional_param('mark', '', PARAM_ALPHA); // Used for tracking read posts if user initiated. +$postid = optional_param('postid', 0, PARAM_INT); // Used for tracking read posts if user initiated. - $url = new moodle_url('/mod/forum/discuss.php', array('d'=>$d)); - if ($parent !== 0) { - $url->param('parent', $parent); - } - $PAGE->set_url($url); +$url = new moodle_url('/mod/forum/discuss.php', array('d'=>$d)); +if ($parent !== 0) { + $url->param('parent', $parent); +} +$PAGE->set_url($url); - $discussion = $DB->get_record('forum_discussions', array('id' => $d), '*', MUST_EXIST); - $course = $DB->get_record('course', array('id' => $discussion->course), '*', MUST_EXIST); - $forum = $DB->get_record('forum', array('id' => $discussion->forum), '*', MUST_EXIST); - $cm = get_coursemodule_from_instance('forum', $forum->id, $course->id, false, MUST_EXIST); +$discussion = $DB->get_record('forum_discussions', array('id' => $d), '*', MUST_EXIST); +$course = $DB->get_record('course', array('id' => $discussion->course), '*', MUST_EXIST); +$forum = $DB->get_record('forum', array('id' => $discussion->forum), '*', MUST_EXIST); +$cm = get_coursemodule_from_instance('forum', $forum->id, $course->id, false, MUST_EXIST); - require_course_login($course, true, $cm); +require_course_login($course, true, $cm); - // move this down fix for MDL-6926 - require_once($CFG->dirroot.'/mod/forum/lib.php'); +// move this down fix for MDL-6926 +require_once($CFG->dirroot.'/mod/forum/lib.php'); - $modcontext = context_module::instance($cm->id); - require_capability('mod/forum:viewdiscussion', $modcontext, NULL, true, 'noviewdiscussionspermission', 'forum'); +$modcontext = context_module::instance($cm->id); +require_capability('mod/forum:viewdiscussion', $modcontext, NULL, true, 'noviewdiscussionspermission', 'forum'); - if (!empty($CFG->enablerssfeeds) && !empty($CFG->forum_enablerssfeeds) && $forum->rsstype && $forum->rssarticles) { - require_once("$CFG->libdir/rsslib.php"); +if (!empty($CFG->enablerssfeeds) && !empty($CFG->forum_enablerssfeeds) && $forum->rsstype && $forum->rssarticles) { + require_once("$CFG->libdir/rsslib.php"); - $rsstitle = format_string($course->shortname, true, array('context' => context_course::instance($course->id))) . ': ' . format_string($forum->name); - rss_add_http_header($modcontext, 'mod_forum', $forum, $rsstitle); + $rsstitle = format_string($course->shortname, true, array('context' => context_course::instance($course->id))) . ': ' . format_string($forum->name); + rss_add_http_header($modcontext, 'mod_forum', $forum, $rsstitle); +} + +// Move discussion if requested. +if ($move > 0 and confirm_sesskey()) { + $return = $CFG->wwwroot.'/mod/forum/discuss.php?d='.$discussion->id; + + if (!$forumto = $DB->get_record('forum', array('id' => $move))) { + print_error('cannotmovetonotexist', 'forum', $return); } - // Move discussion if requested. - if ($move > 0 and confirm_sesskey()) { - $return = $CFG->wwwroot.'/mod/forum/discuss.php?d='.$discussion->id; + require_capability('mod/forum:movediscussions', $modcontext); - if (!$forumto = $DB->get_record('forum', array('id' => $move))) { - print_error('cannotmovetonotexist', 'forum', $return); + if ($forum->type == 'single') { + print_error('cannotmovefromsingleforum', 'forum', $return); + } + + if (!$forumto = $DB->get_record('forum', array('id' => $move))) { + print_error('cannotmovetonotexist', 'forum', $return); + } + + if ($forumto->type == 'single') { + print_error('cannotmovetosingleforum', 'forum', $return); + } + + // Get target forum cm and check it is visible to current user. + $modinfo = get_fast_modinfo($course); + $forums = $modinfo->get_instances_of('forum'); + if (!array_key_exists($forumto->id, $forums)) { + print_error('cannotmovetonotfound', 'forum', $return); + } + $cmto = $forums[$forumto->id]; + if (!$cmto->uservisible) { + print_error('cannotmovenotvisible', 'forum', $return); + } + + $destinationctx = context_module::instance($cmto->id); + require_capability('mod/forum:startdiscussion', $destinationctx); + + if (!forum_move_attachments($discussion, $forum->id, $forumto->id)) { + echo $OUTPUT->notification("Errors occurred while moving attachment directories - check your file permissions"); + } + // For each subscribed user in this forum and discussion, copy over per-discussion subscriptions if required. + $discussiongroup = $discussion->groupid == -1 ? 0 : $discussion->groupid; + $potentialsubscribers = \mod_forum\subscriptions::fetch_subscribed_users( + $forum, + $discussiongroup, + $modcontext, + 'u.id', + true + ); + + // Pre-seed the subscribed_discussion caches. + // Firstly for the forum being moved to. + \mod_forum\subscriptions::fill_subscription_cache($forumto->id); + // And also for the discussion being moved. + \mod_forum\subscriptions::fill_subscription_cache($forum->id); + $subscriptionchanges = array(); + $subscriptiontime = time(); + foreach ($potentialsubscribers as $subuser) { + $userid = $subuser->id; + $targetsubscription = \mod_forum\subscriptions::is_subscribed($userid, $forumto, null, $cmto); + $discussionsubscribed = \mod_forum\subscriptions::is_subscribed($userid, $forum, $discussion->id); + $forumsubscribed = \mod_forum\subscriptions::is_subscribed($userid, $forum); + + if ($forumsubscribed && !$discussionsubscribed && $targetsubscription) { + // The user has opted out of this discussion and the move would cause them to receive notifications again. + // Ensure they are unsubscribed from the discussion still. + $subscriptionchanges[$userid] = \mod_forum\subscriptions::FORUM_DISCUSSION_UNSUBSCRIBED; + } else if (!$forumsubscribed && $discussionsubscribed && !$targetsubscription) { + // The user has opted into this discussion and would otherwise not receive the subscription after the move. + // Ensure they are subscribed to the discussion still. + $subscriptionchanges[$userid] = $subscriptiontime; } + } - require_capability('mod/forum:movediscussions', $modcontext); + $DB->set_field('forum_discussions', 'forum', $forumto->id, array('id' => $discussion->id)); + $DB->set_field('forum_read', 'forumid', $forumto->id, array('discussionid' => $discussion->id)); - if ($forum->type == 'single') { - print_error('cannotmovefromsingleforum', 'forum', $return); - } - - if (!$forumto = $DB->get_record('forum', array('id' => $move))) { - print_error('cannotmovetonotexist', 'forum', $return); - } - - if ($forumto->type == 'single') { - print_error('cannotmovetosingleforum', 'forum', $return); - } - - // Get target forum cm and check it is visible to current user. - $modinfo = get_fast_modinfo($course); - $forums = $modinfo->get_instances_of('forum'); - if (!array_key_exists($forumto->id, $forums)) { - print_error('cannotmovetonotfound', 'forum', $return); - } - $cmto = $forums[$forumto->id]; - if (!$cmto->uservisible) { - print_error('cannotmovenotvisible', 'forum', $return); - } - - $destinationctx = context_module::instance($cmto->id); - require_capability('mod/forum:startdiscussion', $destinationctx); - - if (!forum_move_attachments($discussion, $forum->id, $forumto->id)) { - echo $OUTPUT->notification("Errors occurred while moving attachment directories - check your file permissions"); - } - // For each subscribed user in this forum and discussion, copy over per-discussion subscriptions if required. - $discussiongroup = $discussion->groupid == -1 ? 0 : $discussion->groupid; - $potentialsubscribers = \mod_forum\subscriptions::fetch_subscribed_users( - $forum, - $discussiongroup, - $modcontext, - 'u.id', - true - ); - - // Pre-seed the subscribed_discussion caches. - // Firstly for the forum being moved to. - \mod_forum\subscriptions::fill_subscription_cache($forumto->id); - // And also for the discussion being moved. - \mod_forum\subscriptions::fill_subscription_cache($forum->id); - $subscriptionchanges = array(); - $subscriptiontime = time(); - foreach ($potentialsubscribers as $subuser) { - $userid = $subuser->id; - $targetsubscription = \mod_forum\subscriptions::is_subscribed($userid, $forumto, null, $cmto); - $discussionsubscribed = \mod_forum\subscriptions::is_subscribed($userid, $forum, $discussion->id); - $forumsubscribed = \mod_forum\subscriptions::is_subscribed($userid, $forum); - - if ($forumsubscribed && !$discussionsubscribed && $targetsubscription) { - // The user has opted out of this discussion and the move would cause them to receive notifications again. - // Ensure they are unsubscribed from the discussion still. - $subscriptionchanges[$userid] = \mod_forum\subscriptions::FORUM_DISCUSSION_UNSUBSCRIBED; - } else if (!$forumsubscribed && $discussionsubscribed && !$targetsubscription) { - // The user has opted into this discussion and would otherwise not receive the subscription after the move. - // Ensure they are subscribed to the discussion still. - $subscriptionchanges[$userid] = $subscriptiontime; + // Delete the existing per-discussion subscriptions and replace them with the newly calculated ones. + $DB->delete_records('forum_discussion_subs', array('discussion' => $discussion->id)); + $newdiscussion = clone $discussion; + $newdiscussion->forum = $forumto->id; + foreach ($subscriptionchanges as $userid => $preference) { + if ($preference != \mod_forum\subscriptions::FORUM_DISCUSSION_UNSUBSCRIBED) { + // Users must have viewdiscussion to a discussion. + if (has_capability('mod/forum:viewdiscussion', $destinationctx, $userid)) { + \mod_forum\subscriptions::subscribe_user_to_discussion($userid, $newdiscussion, $destinationctx); } + } else { + \mod_forum\subscriptions::unsubscribe_user_from_discussion($userid, $newdiscussion, $destinationctx); } - - $DB->set_field('forum_discussions', 'forum', $forumto->id, array('id' => $discussion->id)); - $DB->set_field('forum_read', 'forumid', $forumto->id, array('discussionid' => $discussion->id)); - - // Delete the existing per-discussion subscriptions and replace them with the newly calculated ones. - $DB->delete_records('forum_discussion_subs', array('discussion' => $discussion->id)); - $newdiscussion = clone $discussion; - $newdiscussion->forum = $forumto->id; - foreach ($subscriptionchanges as $userid => $preference) { - if ($preference != \mod_forum\subscriptions::FORUM_DISCUSSION_UNSUBSCRIBED) { - // Users must have viewdiscussion to a discussion. - if (has_capability('mod/forum:viewdiscussion', $destinationctx, $userid)) { - \mod_forum\subscriptions::subscribe_user_to_discussion($userid, $newdiscussion, $destinationctx); - } - } else { - \mod_forum\subscriptions::unsubscribe_user_from_discussion($userid, $newdiscussion, $destinationctx); - } - } - - $params = array( - 'context' => $destinationctx, - 'objectid' => $discussion->id, - 'other' => array( - 'fromforumid' => $forum->id, - 'toforumid' => $forumto->id, - ) - ); - $event = \mod_forum\event\discussion_moved::create($params); - $event->add_record_snapshot('forum_discussions', $discussion); - $event->add_record_snapshot('forum', $forum); - $event->add_record_snapshot('forum', $forumto); - $event->trigger(); - - // Delete the RSS files for the 2 forums to force regeneration of the feeds - require_once($CFG->dirroot.'/mod/forum/rsslib.php'); - forum_rss_delete_file($forum); - forum_rss_delete_file($forumto); - - redirect($return.'&moved=-1&sesskey='.sesskey()); } $params = array( - 'context' => $modcontext, + 'context' => $destinationctx, 'objectid' => $discussion->id, + 'other' => array( + 'fromforumid' => $forum->id, + 'toforumid' => $forumto->id, + ) ); - $event = \mod_forum\event\discussion_viewed::create($params); + $event = \mod_forum\event\discussion_moved::create($params); $event->add_record_snapshot('forum_discussions', $discussion); $event->add_record_snapshot('forum', $forum); + $event->add_record_snapshot('forum', $forumto); $event->trigger(); - unset($SESSION->fromdiscussion); + // Delete the RSS files for the 2 forums to force regeneration of the feeds + require_once($CFG->dirroot.'/mod/forum/rsslib.php'); + forum_rss_delete_file($forum); + forum_rss_delete_file($forumto); - if ($mode) { - set_user_preference('forum_displaymode', $mode); + redirect($return.'&moved=-1&sesskey='.sesskey()); +} + +$params = array( + 'context' => $modcontext, + 'objectid' => $discussion->id, +); +$event = \mod_forum\event\discussion_viewed::create($params); +$event->add_record_snapshot('forum_discussions', $discussion); +$event->add_record_snapshot('forum', $forum); +$event->trigger(); + +unset($SESSION->fromdiscussion); + +if ($mode) { + set_user_preference('forum_displaymode', $mode); +} + +$displaymode = get_user_preferences('forum_displaymode', $CFG->forum_displaymode); + +if ($parent) { + // If flat AND parent, then force nested display this time + if ($displaymode == FORUM_MODE_FLATOLDEST or $displaymode == FORUM_MODE_FLATNEWEST) { + $displaymode = FORUM_MODE_NESTED; } +} else { + $parent = $discussion->firstpost; +} - $displaymode = get_user_preferences('forum_displaymode', $CFG->forum_displaymode); +if (! $post = forum_get_post_full($parent)) { + print_error("notexists", 'forum', "$CFG->wwwroot/mod/forum/view.php?f=$forum->id"); +} - if ($parent) { - // If flat AND parent, then force nested display this time - if ($displaymode == FORUM_MODE_FLATOLDEST or $displaymode == FORUM_MODE_FLATNEWEST) { - $displaymode = FORUM_MODE_NESTED; - } - } else { - $parent = $discussion->firstpost; - } +if (!forum_user_can_see_post($forum, $discussion, $post, null, $cm)) { + print_error('noviewdiscussionspermission', 'forum', "$CFG->wwwroot/mod/forum/view.php?id=$forum->id"); +} - if (! $post = forum_get_post_full($parent)) { - print_error("notexists", 'forum', "$CFG->wwwroot/mod/forum/view.php?f=$forum->id"); - } - - if (!forum_user_can_see_post($forum, $discussion, $post, null, $cm)) { - print_error('noviewdiscussionspermission', 'forum', "$CFG->wwwroot/mod/forum/view.php?id=$forum->id"); - } - - if ($mark == 'read' or $mark == 'unread') { - if ($CFG->forum_usermarksread && forum_tp_can_track_forums($forum) && forum_tp_is_tracked($forum)) { - if ($mark == 'read') { - forum_tp_add_read_record($USER->id, $postid); - } else { - // unread - forum_tp_delete_read_records($USER->id, $postid); - } +if ($mark == 'read' or $mark == 'unread') { + if ($CFG->forum_usermarksread && forum_tp_can_track_forums($forum) && forum_tp_is_tracked($forum)) { + if ($mark == 'read') { + forum_tp_add_read_record($USER->id, $postid); + } else { + // unread + forum_tp_delete_read_records($USER->id, $postid); } } +} - $searchform = forum_search_form($course); +$searchform = forum_search_form($course); - $forumnode = $PAGE->navigation->find($cm->id, navigation_node::TYPE_ACTIVITY); - if (empty($forumnode)) { - $forumnode = $PAGE->navbar; - } else { - $forumnode->make_active(); - } - $node = $forumnode->add(format_string($discussion->name), new moodle_url('/mod/forum/discuss.php', array('d'=>$discussion->id))); - $node->display = false; - if ($node && $post->id != $discussion->firstpost) { - $node->add(format_string($post->subject), $PAGE->url); - } - - $PAGE->set_title("$course->shortname: ".format_string($discussion->name)); - $PAGE->set_heading($course->fullname); - $PAGE->set_button($searchform); - $renderer = $PAGE->get_renderer('mod_forum'); - - echo $OUTPUT->header(); - - echo $OUTPUT->heading(format_string($forum->name), 2); - echo $OUTPUT->heading(format_string($discussion->name), 3, 'discussionname'); - - // is_guest should be used here as this also checks whether the user is a guest in the current course. - // Guests and visitors cannot subscribe - only enrolled users. - if ((!is_guest($modcontext, $USER) && isloggedin()) && has_capability('mod/forum:viewdiscussion', $modcontext)) { - // Discussion subscription. - if (\mod_forum\subscriptions::is_subscribable($forum)) { - echo html_writer::div( - forum_get_discussion_subscription_icon($forum, $post->discussion, null, true), - 'discussionsubscription' - ); - echo forum_get_discussion_subscription_icon_preloaders(); - } +$forumnode = $PAGE->navigation->find($cm->id, navigation_node::TYPE_ACTIVITY); +if (empty($forumnode)) { + $forumnode = $PAGE->navbar; +} else { + $forumnode->make_active(); +} +$node = $forumnode->add(format_string($discussion->name), new moodle_url('/mod/forum/discuss.php', array('d'=>$discussion->id))); +$node->display = false; +if ($node && $post->id != $discussion->firstpost) { + $node->add(format_string($post->subject), $PAGE->url); +} + +$PAGE->set_title("$course->shortname: ".format_string($discussion->name)); +$PAGE->set_heading($course->fullname); +$PAGE->set_button($searchform); +$renderer = $PAGE->get_renderer('mod_forum'); + +echo $OUTPUT->header(); + +echo $OUTPUT->heading(format_string($forum->name), 2); +echo $OUTPUT->heading(format_string($discussion->name), 3, 'discussionname'); + +// is_guest should be used here as this also checks whether the user is a guest in the current course. +// Guests and visitors cannot subscribe - only enrolled users. +if ((!is_guest($modcontext, $USER) && isloggedin()) && has_capability('mod/forum:viewdiscussion', $modcontext)) { + // Discussion subscription. + if (\mod_forum\subscriptions::is_subscribable($forum)) { + echo html_writer::div( + forum_get_discussion_subscription_icon($forum, $post->discussion, null, true), + 'discussionsubscription' + ); + echo forum_get_discussion_subscription_icon_preloaders(); } +} /// Check to see if groups are being used in this forum /// If so, make sure the current person is allowed to see this discussion /// Also, if we know they should be able to reply, then explicitly set $canreply for performance reasons - $canreply = forum_user_can_post($forum, $discussion, $USER, $cm, $course, $modcontext); - if (!$canreply and $forum->type !== 'news') { - if (isguestuser() or !isloggedin()) { - $canreply = true; - } - if (!is_enrolled($modcontext) and !is_viewing($modcontext)) { - // allow guests and not-logged-in to see the link - they are prompted to log in after clicking the link - // normal users with temporary guest access see this link too, they are asked to enrol instead - $canreply = enrol_selfenrol_available($course->id); - } +$canreply = forum_user_can_post($forum, $discussion, $USER, $cm, $course, $modcontext); +if (!$canreply and $forum->type !== 'news') { + if (isguestuser() or !isloggedin()) { + $canreply = true; } + if (!is_enrolled($modcontext) and !is_viewing($modcontext)) { + // allow guests and not-logged-in to see the link - they are prompted to log in after clicking the link + // normal users with temporary guest access see this link too, they are asked to enrol instead + $canreply = enrol_selfenrol_available($course->id); + } +} - // Output the links to neighbour discussions. - $neighbours = forum_get_discussion_neighbours($cm, $discussion); - $neighbourlinks = $renderer->neighbouring_discussion_navigation($neighbours['prev'], $neighbours['next']); - echo $neighbourlinks; +// Output the links to neighbour discussions. +$neighbours = forum_get_discussion_neighbours($cm, $discussion); +$neighbourlinks = $renderer->neighbouring_discussion_navigation($neighbours['prev'], $neighbours['next']); +echo $neighbourlinks; /// Print the controls across the top - echo '
'; +echo '
'; - if (!empty($CFG->enableportfolios) && has_capability('mod/forum:exportdiscussion', $modcontext)) { - require_once($CFG->libdir.'/portfoliolib.php'); - $button = new portfolio_add_button(); - $button->set_callback_options('forum_portfolio_caller', array('discussionid' => $discussion->id), 'mod_forum'); - $button = $button->to_html(PORTFOLIO_ADD_FULL_FORM, get_string('exportdiscussion', 'mod_forum')); - $buttonextraclass = ''; - if (empty($button)) { - // no portfolio plugin available. - $button = ' '; - $buttonextraclass = ' noavailable'; - } - echo html_writer::tag('div', $button, array('class' => 'discussioncontrol exporttoportfolio'.$buttonextraclass)); - } else { - echo html_writer::tag('div', ' ', array('class'=>'discussioncontrol nullcontrol')); +if (!empty($CFG->enableportfolios) && has_capability('mod/forum:exportdiscussion', $modcontext)) { + require_once($CFG->libdir.'/portfoliolib.php'); + $button = new portfolio_add_button(); + $button->set_callback_options('forum_portfolio_caller', array('discussionid' => $discussion->id), 'mod_forum'); + $button = $button->to_html(PORTFOLIO_ADD_FULL_FORM, get_string('exportdiscussion', 'mod_forum')); + $buttonextraclass = ''; + if (empty($button)) { + // no portfolio plugin available. + $button = ' '; + $buttonextraclass = ' noavailable'; } + echo html_writer::tag('div', $button, array('class' => 'discussioncontrol exporttoportfolio'.$buttonextraclass)); +} else { + echo html_writer::tag('div', ' ', array('class'=>'discussioncontrol nullcontrol')); +} - // groups selector not needed here - echo '
'; - forum_print_mode_form($discussion->id, $displaymode); - echo "
"; +// groups selector not needed here +echo '
'; +forum_print_mode_form($discussion->id, $displaymode); +echo "
"; - if ($forum->type != 'single' - && has_capability('mod/forum:movediscussions', $modcontext)) { +if ($forum->type != 'single' + && has_capability('mod/forum:movediscussions', $modcontext)) { - echo '
'; - // Popup menu to move discussions to other forums. The discussion in a - // single discussion forum can't be moved. - $modinfo = get_fast_modinfo($course); - if (isset($modinfo->instances['forum'])) { - $forummenu = array(); - // Check forum types and eliminate simple discussions. - $forumcheck = $DB->get_records('forum', array('course' => $course->id),'', 'id, type'); - foreach ($modinfo->instances['forum'] as $forumcm) { - if (!$forumcm->uservisible || !has_capability('mod/forum:startdiscussion', - context_module::instance($forumcm->id))) { - continue; - } - $section = $forumcm->sectionnum; - $sectionname = get_section_name($course, $section); - if (empty($forummenu[$section])) { - $forummenu[$section] = array($sectionname => array()); - } - $forumidcompare = $forumcm->instance != $forum->id; - $forumtypecheck = $forumcheck[$forumcm->instance]->type !== 'single'; - if ($forumidcompare and $forumtypecheck) { - $url = "/mod/forum/discuss.php?d=$discussion->id&move=$forumcm->instance&sesskey=".sesskey(); - $forummenu[$section][$sectionname][$url] = format_string($forumcm->name); - } + echo '
'; + // Popup menu to move discussions to other forums. The discussion in a + // single discussion forum can't be moved. + $modinfo = get_fast_modinfo($course); + if (isset($modinfo->instances['forum'])) { + $forummenu = array(); + // Check forum types and eliminate simple discussions. + $forumcheck = $DB->get_records('forum', array('course' => $course->id),'', 'id, type'); + foreach ($modinfo->instances['forum'] as $forumcm) { + if (!$forumcm->uservisible || !has_capability('mod/forum:startdiscussion', + context_module::instance($forumcm->id))) { + continue; } - if (!empty($forummenu)) { - echo '
'; - $select = new url_select($forummenu, '', - array(''=>get_string("movethisdiscussionto", "forum")), - 'forummenu', get_string('move')); - echo $OUTPUT->render($select); - echo "
"; + $section = $forumcm->sectionnum; + $sectionname = get_section_name($course, $section); + if (empty($forummenu[$section])) { + $forummenu[$section] = array($sectionname => array()); + } + $forumidcompare = $forumcm->instance != $forum->id; + $forumtypecheck = $forumcheck[$forumcm->instance]->type !== 'single'; + if ($forumidcompare and $forumtypecheck) { + $url = "/mod/forum/discuss.php?d=$discussion->id&move=$forumcm->instance&sesskey=".sesskey(); + $forummenu[$section][$sectionname][$url] = format_string($forumcm->name); } } - echo "
"; + if (!empty($forummenu)) { + echo '
'; + $select = new url_select($forummenu, '', + array(''=>get_string("movethisdiscussionto", "forum")), + 'forummenu', get_string('move')); + echo $OUTPUT->render($select); + echo "
"; + } } - echo '
 
'; echo "
"; +} +echo '
 
'; +echo "
"; - if (!empty($forum->blockafter) && !empty($forum->blockperiod)) { - $a = new stdClass(); - $a->blockafter = $forum->blockafter; - $a->blockperiod = get_string('secondstotime'.$forum->blockperiod); - echo $OUTPUT->notification(get_string('thisforumisthrottled','forum',$a)); - } +if (!empty($forum->blockafter) && !empty($forum->blockperiod)) { + $a = new stdClass(); + $a->blockafter = $forum->blockafter; + $a->blockperiod = get_string('secondstotime'.$forum->blockperiod); + echo $OUTPUT->notification(get_string('thisforumisthrottled','forum',$a)); +} - if ($forum->type == 'qanda' && !has_capability('mod/forum:viewqandawithoutposting', $modcontext) && - !forum_user_has_posted($forum->id,$discussion->id,$USER->id)) { - echo $OUTPUT->notification(get_string('qandanotify','forum')); - } +if ($forum->type == 'qanda' && !has_capability('mod/forum:viewqandawithoutposting', $modcontext) && + !forum_user_has_posted($forum->id,$discussion->id,$USER->id)) { + echo $OUTPUT->notification(get_string('qandanotify','forum')); +} - if ($move == -1 and confirm_sesskey()) { - echo $OUTPUT->notification(get_string('discussionmoved', 'forum', format_string($forum->name,true))); - } +if ($move == -1 and confirm_sesskey()) { + echo $OUTPUT->notification(get_string('discussionmoved', 'forum', format_string($forum->name,true))); +} - $canrate = has_capability('mod/forum:rate', $modcontext); - forum_print_discussion($course, $cm, $forum, $discussion, $post, $displaymode, $canreply, $canrate); +$canrate = has_capability('mod/forum:rate', $modcontext); +forum_print_discussion($course, $cm, $forum, $discussion, $post, $displaymode, $canreply, $canrate); - echo $neighbourlinks; +echo $neighbourlinks; - // Add the subscription toggle JS. - $PAGE->requires->yui_module('moodle-mod_forum-subscriptiontoggle', 'Y.M.mod_forum.subscriptiontoggle.init'); +// Add the subscription toggle JS. +$PAGE->requires->yui_module('moodle-mod_forum-subscriptiontoggle', 'Y.M.mod_forum.subscriptiontoggle.init'); - echo $OUTPUT->footer(); +echo $OUTPUT->footer();