Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 1.9.4
-
Fix Version/s: 1.9.5
-
Labels:None
-
Difficulty:Easy
-
Affected Branches:MOODLE_19_STABLE
-
Fixed Branches:MOODLE_19_STABLE
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
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'); > }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'); > }