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

New 'deleted' field for forum posts is used in the wrong way for rss feeds

    XMLWordPrintable

Details

    • MOODLE_34_STABLE, MOODLE_35_STABLE
    • MOODLE_33_STABLE, MOODLE_34_STABLE, MOODLE_35_STABLE
    • MDL-62543-master
    • Hide
      1. Go to admin/settings.php?section=optionalsubsystems
      2. Tick enablerssfeeds
      3. Go to admin/settings.php?section=modsettingforum
      4. Set forum_enablerssfeeds to Yes
      5. Create a forum with discussions and posts, e.g. through

        php admin/tool/generator/cli/maketestcourse.php --shortname=SIZE_S3 --size=S

      1. Go to the settings of the forum and set
        1. RSS feed for this activity to Posts
        2. Number of RSS recent articles to 5
      2. Click on RSS feed of posts in the Forum Administration
      3. Expected behaviour is that there are >0 entries
      4. Go to the settings of the forum and set
        1. RSS feed for this activity to Discussions
        2. Number of RSS recent articles to 5
      5. Click on RSS feed of discussions in the Forum Administration
      6. Expected behaviour is that there are >0 entries
      Show
      Go to admin/settings.php?section=optionalsubsystems Tick enablerssfeeds Go to admin/settings.php?section=modsettingforum Set forum_enablerssfeeds to Yes Create a forum with discussions and posts, e.g. through php admin/tool/generator/cli/maketestcourse.php --shortname=SIZE_S3 --size=S Go to the settings of the forum and set RSS feed for this activity to Posts Number of RSS recent articles to 5 Click on RSS feed of posts in the Forum Administration Expected behaviour is that there are >0 entries Go to the settings of the forum and set RSS feed for this activity to Discussions Number of RSS recent articles to 5 Click on RSS feed of discussions in the Forum Administration Expected behaviour is that there are >0 entries
    • Privacy Sprint 1

    Description

      Since 3.4.3 all our rss feeds are empty. With the implementation of the Privacy API the 'deleted' field was added to forum posts. In the rsslib.php this was added to two sql queries, which determine the posts, which should be displayed in the feed. However, both queries check for <> 0 instead of <> 1 and thus do not exclude all deleted queries, but all non deleted queries instead. In the lib.php, where this deleted flag is also used for filtering posts, it is correctly used with <> 1.

      This is a regression from MDL-61309.

      Attachments

        Issue Links

          Activity

            People

              tobias.reischmann Tobias Reischmann
              tobias.reischmann Tobias Reischmann
              Victor Déniz Falcón Victor Déniz Falcón
              Andrew Lyons Andrew Lyons
              Gladys Basiana Gladys Basiana
              Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              3 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                9/Jul/18

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 hour
                  1h