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

forum_print_discussion now checks to see whether there are any

Browse Source 
ratings that haven't been done yet on a discussion.  If not, then
it won't print the ratings form.  This avoids a slight confusion
about the interface that existed previously.
This commit is contained in:
martin
2002-08-06 06:09:44 +00:00
parent f7fe342d6a
commit 9d1b97c5d4
1 changed files with 38 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mod/forum/lib.php
+38 -4
View File
@@ -305,6 +305,7 @@ function forum_print_search_form($course, $search="") {
function forum_count_discussion_replies($forum="0") {
// Returns an array of counts of replies to each discussion (optionally in one forum)
if ($forum) {
$forumselect = " AND d.forum = '$forum'";
}
@@ -314,6 +315,31 @@ function forum_count_discussion_replies($forum="0") {
GROUP BY p.discussion");
}
function forum_count_unrated_posts($discussionid, $userid) {
// How many unrated posts are in the given discussion for a given user?
if ($posts = get_record_sql("SELECT count(*) as num
FROM forum_posts
WHERE parent > 0 AND
discussion = '$discussionid' AND
user <> '$userid' ")) {
if ($rated = get_record_sql("SELECT count(*) as num
FROM forum_posts p, forum_ratings r
WHERE p.id = r.post AND r.user = '$userid'")) {
$difference = $posts->num - $rated->num;
if ($difference > 0) {
return $difference;
} else {
return 0; // Just in case there was a counting error
}
} else {
return $posts->num;
}
} else {
return 0;
}
}
function forum_set_return() {
global $SESSION, $HTTP_REFERER;
@@ -872,7 +898,15 @@ function forum_print_discussion($course, $forum, $discussion, $post, $mode) {
forum_print_mode_form($discussion->id, $mode);
$ratingform = false;
if ($forum->assessed && $USER->id) {
$unrated = forum_count_unrated_posts($discussion->id, $USER->id);
if ($unrated > 0) {
$ratingform = true;
}
}
if ($ratingform) {
echo "<FORM NAME=form METHOD=POST ACTION=rate.php>";
echo "<INPUT TYPE=hidden NAME=id VALUE=\"$course->id\">";
}
@@ -882,20 +916,20 @@ function forum_print_discussion($course, $forum, $discussion, $post, $mode) {
case -1 : // Flat descending
default:
echo "<UL>";
forum_print_posts_flat($post->discussion, $course->id, $mode, $forum->assessed);
forum_print_posts_flat($post->discussion, $course->id, $mode, $ratingform);
echo "</UL>";
break;
case 2 : // Threaded
forum_print_posts_threaded($post->id, $course->id, 0, $forum->assessed);
forum_print_posts_threaded($post->id, $course->id, 0, $ratingform);
break;
case 3 : // Nested
forum_print_posts_nested($post->id, $course->id, $forum->assessed);
forum_print_posts_nested($post->id, $course->id, $ratingform);
break;
}
if ($forum->assessed && $USER->id) {
if ($ratingform) {
echo "<CENTER><P ALIGN=center><INPUT TYPE=submit VALUE=\"".get_string("sendinratings", "forum")."\"></P></CENTER>";
echo "</FORM>";
}