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

          Activity

          Hide
          salvetore Michael de Raadt added a comment -

          Thanks for reporting that and providing a patch.

          Show
          salvetore Michael de Raadt added a comment - Thanks for reporting that and providing a patch.
          Hide
          rajeshtaneja Rajesh Taneja added a comment -

          Thanks for providing patch Matt,

          Patch looks Good. I have created local branches with your commit.
          Pushing it for peer-review.

          Show
          rajeshtaneja Rajesh Taneja added a comment - Thanks for providing patch Matt, Patch looks Good. I have created local branches with your commit. Pushing it for peer-review.
          Hide
          markn Mark Nelson added a comment -

          Hi Raj, looks good. However you may want to rebase for 2.4/master or change the diff URL to point directly to the commit rather than diff.

          eg.
          2.4. https://github.com/rajeshtaneja/moodle/commit/92e16636d71b5e9e907884df3340e3ab50c88da6
          2.5. https://github.com/rajeshtaneja/moodle/commit/3cfe11b99ca433e07dc34458eeedbc0c63dce6e5

          Also the commit message contains '()' which looks odd.

          Thanks!

          Show
          markn Mark Nelson added a comment - Hi Raj, looks good. However you may want to rebase for 2.4/master or change the diff URL to point directly to the commit rather than diff. eg. 2.4. https://github.com/rajeshtaneja/moodle/commit/92e16636d71b5e9e907884df3340e3ab50c88da6 2.5. https://github.com/rajeshtaneja/moodle/commit/3cfe11b99ca433e07dc34458eeedbc0c63dce6e5 Also the commit message contains '()' which looks odd. Thanks!
          Hide
          rajeshtaneja Rajesh Taneja added a comment -

          Thanks Mark,
          Rebased branches and updated commit message

          Show
          rajeshtaneja Rajesh Taneja added a comment - Thanks Mark, Rebased branches and updated commit message
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Raj, this has been integrated.

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Raj, this has been integrated.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          This is working as expected.

          Test passed.

          Show
          rwijaya Rossiani Wijaya added a comment - This is working as expected. Test passed.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          And your fantastic code has met core, hope they become good friends for a long period.

          Closing, thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - And your fantastic code has met core, hope they become good friends for a long period. Closing, thanks!

            People

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

              Dates

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