Moodle
  1. Moodle
  2. MDL-18483

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

    Details

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

      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

          • Votes:
            4 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: