moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-05 08:56:36 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'wip-mdl-30421' of git://github.com/rajeshtaneja/moodle

Browse source
This commit is contained in:
Sam Hemelryk 2011-12-01 15:48:20 +13:00
commit 7bebeb2a29
3 changed files with 29 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

1
mod/forum/lang/en/forum.php
View file

@ -408,6 +408,7 @@ $string['usermarksread'] = 'Manual message read marking';
$string['viewalldiscussions'] = 'View all discussions'; $string['viewalldiscussions'] = 'View all discussions';
$string['warnafter'] = 'Post threshold for warning'; $string['warnafter'] = 'Post threshold for warning';
$string['warnafter_help'] = 'Students can be warned as they approach the maximum number of posts allowed in a given period. This setting specifies after how many posts they are warned. Users with the capability mod/forum:postwithoutthrottling are exempt from post limits.'; $string['warnafter_help'] = 'Students can be warned as they approach the maximum number of posts allowed in a given period. This setting specifies after how many posts they are warned. Users with the capability mod/forum:postwithoutthrottling are exempt from post limits.';
$string['warnformorepost'] = 'Warning! There is more than one discussion in this forum - using the most recent';
$string['yournewquestion'] = 'Your new question'; $string['yournewquestion'] = 'Your new question';
$string['yournewtopic'] = 'Your new discussion topic'; $string['yournewtopic'] = 'Your new discussion topic';
$string['yourreply'] = 'Your reply'; $string['yourreply'] = 'Your reply';

43
mod/forum/lib.php
View file

@ -153,30 +153,31 @@ function forum_update_instance($forum, $mform) {
} }
if ($forum->type == 'single') { // Update related discussion and post. if ($forum->type == 'single') { // Update related discussion and post.
if (! $discussion = $DB->get_record('forum_discussions', array('forum'=>$forum->id))) { $discussions = $DB->get_records('forum_discussions', array('forum'=>$forum->id), 'timemodified ASC');
if ($discussions = $DB->get_records('forum_discussions', array('forum'=>$forum->id), 'timemodified ASC')) { if (!empty($discussions)) {
echo $OUTPUT->notification('Warning! There is more than one discussion in this forum - using the most recent'); if (count($discussions) > 1) {
$discussion = array_pop($discussions); echo $OUTPUT->notification(get_string('warnformorepost', 'forum'));
} else { }
// try to recover by creating initial discussion - MDL-16262 $discussion = array_pop($discussions);
$discussion = new stdClass(); } else {
$discussion->course = $forum->course; // try to recover by creating initial discussion - MDL-16262
$discussion->forum = $forum->id; $discussion = new stdClass();
$discussion->name = $forum->name; $discussion->course = $forum->course;
$discussion->assessed = $forum->assessed; $discussion->forum = $forum->id;
$discussion->message = $forum->intro; $discussion->name = $forum->name;
$discussion->messageformat = $forum->introformat; $discussion->assessed = $forum->assessed;
$discussion->messagetrust = true; $discussion->message = $forum->intro;
$discussion->mailnow = false; $discussion->messageformat = $forum->introformat;
$discussion->groupid = -1; $discussion->messagetrust = true;
$discussion->mailnow = false;
$discussion->groupid = -1;
$message = ''; $message = '';
forum_add_discussion($discussion, null, $message); forum_add_discussion($discussion, null, $message);
if (! $discussion = $DB->get_record('forum_discussions', array('forum'=>$forum->id))) { if (! $discussion = $DB->get_record('forum_discussions', array('forum'=>$forum->id))) {
print_error('cannotadd', 'forum'); print_error('cannotadd', 'forum');
}
} }
} }
if (! $post = $DB->get_record('forum_posts', array('id'=>$discussion->firstpost))) { if (! $post = $DB->get_record('forum_posts', array('id'=>$discussion->firstpost))) {

17
mod/forum/view.php
View file

@ -141,10 +141,10 @@
// If it's a simple single discussion forum, we need to print the display // If it's a simple single discussion forum, we need to print the display
// mode control. // mode control.
if ($forum->type == 'single') { if ($forum->type == 'single') {
if (! $discussion = $DB->get_record("forum_discussions", array("forum" => $forum->id))) { $discussion = NULL;
if ($discussions = $DB->get_records("forum_discussions", array("forum", $forum->id), "timemodified ASC")) { $discussions = $DB->get_records('forum_discussions', array('forum'=>$forum->id), 'timemodified ASC');
$discussion = array_pop($discussions); if (!empty($discussions)) {
} $discussion = array_pop($discussions);
} }
if ($discussion) { if ($discussion) {
if ($mode) { if ($mode) {
@ -167,13 +167,8 @@
switch ($forum->type) { switch ($forum->type) {
case 'single': case 'single':
if (! $discussion = $DB->get_record("forum_discussions", array("forum" => $forum->id))) { if (!empty($discussions) && count($discussions) > 1) {
if ($discussions = $DB->get_records("forum_discussions", array("forum" => $forum->id), "timemodified ASC")) { echo $OUTPUT->notification(get_string('warnformorepost', 'forum'));
echo $OUTPUT->notification("Warning! There is more than one discussion in this forum - using the most recent");
$discussion = array_pop($discussions);
} else {
print_error('nodiscussions', 'forum');
}
} }
if (! $post = forum_get_post_full($discussion->firstpost)) { if (! $post = forum_get_post_full($discussion->firstpost)) {
print_error('cannotfindfirstpost', 'forum'); print_error('cannotfindfirstpost', 'forum');