Moodle

Always show forum description at the top of the social course format

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.4
  • Fix Version/s: 1.9.5
  • Component/s: Course, Forum
  • Labels:
    None

Description

I think it makes sense to always show the forum description at the top of the social format ... obviously if the forum doesn't have a description nothing need be shown.

I've attached a patch I'm using on Moodle.org. Does anyone object to this in 1.9.5?

Index: format.php
===================================================================
RCS file: /cvsroot/moodle/moodle/course/format/social/format.php,v
retrieving revision 1.41.2.2
diff -r1.41.2.2 format.php
41c41,44
< print_heading_block(get_string('socialheadline'));

> //print_heading_block(get_string('socialheadline'));
> $options = new stdclass; // Moodle.org
> $options->para = false; // Moodle.org
> print_box(format_text($forum->intro, FORMAT_MOODLE, $options), 'generalbox', 'intro'); // Moodle.org

Activity

Hide
Martin Dougiamas added a comment -

This is now in 1.9.5 and HEAD.

 
Index: format.php
===================================================================
RCS file: /cvsroot/moodle/moodle/course/format/social/format.php,v
retrieving revision 1.41.2.2
diff -r1.41.2.2 format.php
41c41,47
<         print_heading_block(get_string('socialheadline'));
---
> 
>     /// Print forum intro above posts  MDL-18483
>         if (trim($forum->intro) != '') {
>             $options = new stdclass;
>             $options->para = false;
>             print_box(format_text($forum->intro, FORMAT_MOODLE, $options), 'generalbox', 'intro');
>         }
Show
Martin Dougiamas added a comment - This is now in 1.9.5 and HEAD.
 
Index: format.php
===================================================================
RCS file: /cvsroot/moodle/moodle/course/format/social/format.php,v
retrieving revision 1.41.2.2
diff -r1.41.2.2 format.php
41c41,47
<         print_heading_block(get_string('socialheadline'));
---
> 
>     /// Print forum intro above posts  MDL-18483
>         if (trim($forum->intro) != '') {
>             $options = new stdclass;
>             $options->para = false;
>             print_box(format_text($forum->intro, FORMAT_MOODLE, $options), 'generalbox', 'intro');
>         }
Hide
Helen Foster added a comment -

Martin, thanks for fixing this issue

http://docs.moodle.org/en/Course_formats now includes a note about the forum introduction being displayed at the top of the course page.

Show
Helen Foster added a comment - Martin, thanks for fixing this issue http://docs.moodle.org/en/Course_formats now includes a note about the forum introduction being displayed at the top of the course page.

People

Vote (4)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: