Moodle
  1. Moodle
  2. MDL-34775

Forum RSS count of posts/discussions is not dependent on RSS setting

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.1, 2.3.4, 2.4.1
    • Fix Version/s: 2.3.5, 2.4.2, 2.5
    • Component/s: Forum, RSS
    • Labels:

      Description

      This is a minor usability issue.

      When RSS is enabled a forum's settings includes settings to control the RSS feed coming out of the forum. The first setting is labelled "RSS feed for this activity" and has three options: "None", "Discussions" and "Posts". When this setting is set to none, no RSS feed is generated for the forum. The second RSS setting is the number of recent articles. This setting is active regardless of the state of the first setting. This second setting should be disabled when the first is set to "None".

      Reproduction steps:

      1. Log in as admin
      2. In Settings > Site admin > Advanced features, enable RSS feeds
      3. In Settings > Site admin > Plugins > Activity modules > Forum, enable RSS feeds
      4. Navigate into a course
      5. Create a new forum or edit an existing one
      6. Set the setting labelled "RSS feed for this activity" to "None"
      7. Attempt to change the setting labelled "Number of RSS recent articles"

      Expected result: You should not be able to change "Number of RSS recent articles" while "RSS feed for this activity" is "None".

      Actual result: You can.

        Gliffy Diagrams

        1. mod_form.php.patch
          0.7 kB
          Michael de Raadt
        1. screenshot-1.jpg
          32 kB

          Issue Links

            Activity

            Hide
            Jason Fowler added a comment -

            I've only got the patch for master so far, will backport once I get the all clear from the peer review.

            Show
            Jason Fowler added a comment - I've only got the patch for master so far, will backport once I get the all clear from the peer review.
            Hide
            Mark Nelson added a comment -

            Patch looks good and the logic is sound, thanks Michael. Jason feel free to backport and submit to integration.

            Show
            Mark Nelson added a comment - Patch looks good and the logic is sound, thanks Michael. Jason feel free to backport and submit to integration.
            Hide
            Jason Fowler added a comment -

            Thanks Mark, Back ported and pushed for integration

            Show
            Jason Fowler added a comment - Thanks Mark, Back ported and pushed for integration
            Hide
            Eloy Lafuente (stronk7) added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            Aparup Banerjee added a comment -

            thanks, thats been integrated into 23, 24 and master.

            Show
            Aparup Banerjee added a comment - thanks, thats been integrated into 23, 24 and master.
            Hide
            Rossiani Wijaya added a comment -

            This is working as expected.

            Tested for 2.3, 2.4 and master

            Test passed.

            Show
            Rossiani Wijaya added a comment - This is working as expected. Tested for 2.3, 2.4 and master Test passed.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            This is valid for unlimited entries to the, soon to be unveiled, Moodle Codebase Gardens. It includes free access to all facilities.

            Personal and non-transferable to all assignees, reviewers and testers in this issue. Valid until switching to Blackboard (100000€ penalization will be applied).

            Thanks, closing as fixed!

            Show
            Eloy Lafuente (stronk7) added a comment - This is valid for unlimited entries to the, soon to be unveiled, Moodle Codebase Gardens. It includes free access to all facilities. Personal and non-transferable to all assignees, reviewers and testers in this issue. Valid until switching to Blackboard (100000€ penalization will be applied). Thanks, closing as fixed!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: