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

use FORUM_CHOOSESUBSCRIBE instead of 0, etc. in mod/forum/subscribe.php

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.6, 2.1.9, 2.2.3, 2.2.6, 2.3.3
    • Fix Version/s: 2.3.4, 2.4.1
    • Component/s: Forum
    • Labels:
    • Testing Instructions:
      Hide

      It probably don't need any testing instructions.
      Some instructions to check existing behaviour

      1. Log in as admin
      2. Select forum in a course
      3. Change forum subscription and make sure it is retained without any error/warning. (Settings -> Forum administration -> Subscription mode -> Optional/forced/auto/disable )
      Show
      It probably don't need any testing instructions. Some instructions to check existing behaviour Log in as admin Select forum in a course Change forum subscription and make sure it is retained without any error/warning. (Settings -> Forum administration -> Subscription mode -> Optional/forced/auto/disable )
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull Master Branch:
      wip-mdl-36958

      Description

      This four-line patch changes hard-coded numbers into the define()d words that should be used instead.

      Patch is here: https://github.com/moquist/moodle/commit/8cef53e89b6afa8aa2fc854177006a4e1cc1124b

        Gliffy Diagrams

          Attachments

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/Jan/13