Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-66669

Announcement forums have different maxattachments

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.7.2
    • Fix Version/s: None
    • Component/s: Forum
    • Labels:
      None
    • Testing Instructions:
      Hide

      1. Go to Site administration.
      2. Make sure the setting "forum_maxattachments" is set to 9.
      3. Create a course.
      4. Create a forum in that course.
      5. Compare the default max attachment number between the announcement forum and the custom forum you created.
      => Both should have the same max attachment number.

      Show
      1. Go to Site administration. 2. Make sure the setting "forum_maxattachments" is set to 9. 3. Create a course. 4. Create a forum in that course. 5. Compare the default max attachment number between the announcement forum and the custom forum you created. => Both should have the same max attachment number.
    • Affected Branches:
      MOODLE_37_STABLE

      Description

      The default max number of attachments for forums can be set in the forum plugin settings through "forum_maxattachments". However, announcement forums don't use that setting. They always have their max attachments set to 1 by default. The 1 is the default value of the "maxattachments" column in the DB.

      In my opinion the "forum_maxattachments" setting should also apply to announcement forums. Or is there a reason why it doesn't?

      See:
      https://github.com/moodle/moodle/blob/1c3efe48f885797433397eca56f9a0239b6fe547/mod/forum/db/install.xml#L22
      https://github.com/moodle/moodle/blob/1c3efe48f885797433397eca56f9a0239b6fe547/mod/forum/settings.php#L54-L55

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              tschroeder Tim Schroeder
              Participants:
              Component watchers:
              Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: