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

Site news RSS feed doesn't work for guests

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.4, 2.3.1, 2.4
    • Fix Version/s: 2.2.5, 2.3.2, STABLE backlog
    • Component/s: RSS
    • Labels:
    • Database:
      Any
    • Testing Instructions:
      Hide
      1. In Admin > Advanced features, enable RSS feeds
      2. In Admin > Plugins > Activity modules > Forum, enable rss feeds
      3. Add the latest news block to your site front page
      4. Add a news item to the front page
      5. Click to view the news item and use the settings block to edit the forum settings
      6. Enable RSS for this forum (say posts, 10 recent articles)
      7. Click the RSS link
      8. VERIFY the feed generated contains the news item you created
      9. Log out
      10. Click the RSS link
      11. VERIFY: the generated feed contains the news item you created
      Show
      In Admin > Advanced features, enable RSS feeds In Admin > Plugins > Activity modules > Forum, enable rss feeds Add the latest news block to your site front page Add a news item to the front page Click to view the news item and use the settings block to edit the forum settings Enable RSS for this forum (say posts, 10 recent articles) Click the RSS link VERIFY the feed generated contains the news item you created Log out Click the RSS link VERIFY: the generated feed contains the news item you created
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:

      Description

      The RSS feed link, on frontpage site news block throws an error if you are not logged in. But if you use the RSS feed directly from the forum it works fine. I´ve tested this issue on a 2.1.4 version, also you can check it on moodle.org.

        Gliffy Diagrams

        1. rss_error_en.png
          58 kB
        2. rss_error.png
          16 kB
        3. rss_moodle.png
          26 kB
        4. rss_moodle-org.png
          18 kB
        5. rss_OK_logged-in.png
          74 kB

          Issue Links

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting that.

            I'm not sure it's necessary, but it might help if you could translate the error reported.

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting that. I'm not sure it's necessary, but it might help if you could translate the error reported.
            Hide
            wenderw Wenceslao Fernadez added a comment -

            Hi

            I have updated the issue adding some more images in English.

            Salu2

            Show
            wenderw Wenceslao Fernadez added a comment - Hi I have updated the issue adding some more images in English. Salu2
            Hide
            poltawski Dan Poltawski added a comment -

            Well, thanks to my former colleague Paul Verrall at LUNS, I now understand what the cause of this is. Its relating to the front page news block using 0 as the RSS feed url, rather than the siteguest.

            Show
            poltawski Dan Poltawski added a comment - Well, thanks to my former colleague Paul Verrall at LUNS, I now understand what the cause of this is. Its relating to the front page news block using 0 as the RSS feed url, rather than the siteguest.
            Hide
            poltawski Dan Poltawski added a comment -

            I've fixed this, as noted the userid when not logged in needs to be the site guest rather than '0'.

            Show
            poltawski Dan Poltawski added a comment - I've fixed this, as noted the userid when not logged in needs to be the site guest rather than '0'.
            Hide
            poltawski Dan Poltawski 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
            poltawski Dan Poltawski 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
            nebgor Aparup Banerjee added a comment -

            cool, integrated into 22, 23 and master now.

            ps: some permissions testing within test here could be extra fun here.

            Show
            nebgor Aparup Banerjee added a comment - cool, integrated into 22, 23 and master now. ps: some permissions testing within test here could be extra fun here.
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success!

            Tested in 2.2, 2.3 and master.

            Show
            salvetore Michael de Raadt added a comment - Test result: Success! Tested in 2.2, 2.3 and master.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Fixed STOP Closed STOP Thanks STOP

            Yay, imagination! Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Fixed STOP Closed STOP Thanks STOP Yay, imagination! Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Sep/12