Moodle
  1. Moodle
  2. MDL-24887

Issue with External RSS feed block validator.org

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.9.10
    • Fix Version/s: 1.9.14
    • Component/s: RSS
    • Labels:
    • Environment:
      Linux, PSQL, 1.9.6 (Build: 20091021) or 1.9.10 (Build: 20101025)
    • Testing Instructions:
      Hide

      1/ Add a new rss block
      2/ Configure the block
      3/ Click 'edit news feeds'
      4/ Put the url: http://news.lancs.ac.uk/Web/_layouts/LUNews/rss.ashx?S=15&C=63
      5/ Click validate feed

      Expected result:
      The external validator validates the feed specified

      Actual result:
      The validator does not validate the feed because it has part of the string stripped

      Show
      1/ Add a new rss block 2/ Configure the block 3/ Click 'edit news feeds' 4/ Put the url: http://news.lancs.ac.uk/Web/_layouts/LUNews/rss.ashx?S=15&C=63 5/ Click validate feed Expected result: The external validator validates the feed specified Actual result: The validator does not validate the feed because it has part of the string stripped
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE
    • Pull from Repository:
    • Rank:
      6430

      Description

      If you add a rss link from Echo 360, it has two links, one for mp3 and another for m4v. If you run these links through the 'Validate Feed' fails because the parameter is stripped off when calling validator.org.

      Fix is to add escape() to the javascript call.

        Activity

        Hide
        Tim Lock added a comment -

        Patch to fix lib/rsslib.php

        Show
        Tim Lock added a comment - Patch to fix lib/rsslib.php
        Hide
        Michael Blake added a comment -

        This issue is causing problems for MP client. Please give it priority.

        Show
        Michael Blake added a comment - This issue is causing problems for MP client. Please give it priority.
        Hide
        Dan Poltawski added a comment -

        A 'test rss' feed url would be useful to explain this - i've found one http://news.lancs.ac.uk/Web/_layouts/LUNews/rss.ashx?S=15&C=63

        Show
        Dan Poltawski added a comment - A 'test rss' feed url would be useful to explain this - i've found one http://news.lancs.ac.uk/Web/_layouts/LUNews/rss.ashx?S=15&C=63
        Hide
        Dan Poltawski added a comment -

        Integrators: This only affects 1.9 and I believe it to be a trivial issue so I would not be upset if this is not integrated

        Show
        Dan Poltawski added a comment - Integrators: This only affects 1.9 and I believe it to be a trivial issue so I would not be upset if this is not integrated
        Hide
        Eloy Lafuente (stronk7) added a comment -

        The integration of this issue has been delayed to next week due to time constraints. Thanks for your support and patience!

        Sorry and ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The integration of this issue has been delayed to next week due to time constraints. Thanks for your support and patience! Sorry and ciao
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Integrated, thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
        Hide
        Rajesh Taneja added a comment -

        Rss link provided in testcase show error,
        But tried with http://thefonecast.com/RSS.ashx?4 and it works fine.

        Thanks for fixing this Tim and Dan

        Show
        Rajesh Taneja added a comment - Rss link provided in testcase show error, But tried with http://thefonecast.com/RSS.ashx?4 and it works fine. Thanks for fixing this Tim and Dan
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Such an effort has get its prize. Universal Award to the very-best Moodle developer in the family. Thanks!

        Closing. Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Such an effort has get its prize. Universal Award to the very-best Moodle developer in the family. Thanks! Closing. Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: