Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.3.8, 2.4.5, 2.5.1
    • Component/s: RSS
    • Labels:
    • Testing Instructions:
      Hide
      1. Enable RSS feeds
      2. Enable RSS in forum global settings
      3. Setup a forum with rss items set to posts, and 2 entries
      4. Add a forum post
      5. View the rss feeds from navigation
      6. VERIFY: that the html entity for copyright is not used

      For bonus points:

      1. Run on public site (QA site) and run the validator: http://validator.w3.org/feed/check.cgi
      2. VERIFY: no errors about copyright
      Show
      Enable RSS feeds Enable RSS in forum global settings Setup a forum with rss items set to posts, and 2 entries Add a forum post View the rss feeds from navigation VERIFY: that the html entity for copyright is not used For bonus points: Run on public site (QA site) and run the validator: http://validator.w3.org/feed/check.cgi VERIFY: no errors about copyright
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-25901-master

      Description

      As the RSS validator says:
      In addition, interoperability with the widest range of feed readers could be improved by implementing the following recommendations.
      line 9, column 41: copyright should not contain HTML: © [help]
      <copyright>&#169; 2011 Moodle.org</copyright>
      ^

        Gliffy Diagrams

          Activity

          Hide
          Dan Poltawski added a comment - - edited

          Sending for integration

          Show
          Dan Poltawski added a comment - - edited Sending for integration
          Hide
          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
          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
          Sam Hemelryk added a comment -

          Thanks Dan - this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks Dan - this has been integrated now.
          Hide
          Adrian Greeve added a comment -

          Tested on the 2.3, 2.4, 2.5 and master branches, as well as on QA for the bonus points.
          Everything works as it should.
          Test passed.

          Show
          Adrian Greeve added a comment - Tested on the 2.3, 2.4, 2.5 and master branches, as well as on QA for the bonus points. Everything works as it should. Test passed.
          Hide
          Marina Glancy added a comment -

          Thanks for your awesome work! This has now become a part of Moodle.

          Closing as fixed!

          Show
          Marina Glancy added a comment - Thanks for your awesome work! This has now become a part of Moodle. Closing as fixed!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: