Moodle
  1. Moodle
  2. MDL-19274

Blog RSS feed should include tag information for each post

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.5
    • Fix Version/s: 1.9.6
    • Component/s: Blog
    • Labels:
      None
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE
    • Rank:
      36633

      Description

      The blog RSS feeds do not include tags for each post. You can get an RSS feed that filters by tag, but you can't have a feed which includes all posts, and get information about the tags in those posts.

      Some RSS feeds include tags using the category tag. See this blog post:

      http://www.niallkennedy.com/blog/2005/02/tagaware-rss-fe.html

      I have done a patch which adds support for this feature in Moodle blog RSS feeds and rsslib.php. (Not coincidentally, this also allows oublog's implementation of the feature to work in RSS feeds as well as Atom - see CONTRIB-1307.)

      Since this is hopefully for 1.9, and I haven't looked at blog code much, I'd appreciate review/approval. The patch is very short.

        Activity

        Hide
        Eloy Lafuente (stronk7) added a comment -

        Has sense, nice improvement IMO. Implementation looks correct (untested here).

        Addressing this to 1.9.6.

        My +1, one more required, so please, comment. Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Has sense, nice improvement IMO. Implementation looks correct (untested here). Addressing this to 1.9.6. My +1, one more required, so please, comment. Ciao
        Hide
        Helen Foster added a comment -

        I've voted. Does that count as +1?

        Show
        Helen Foster added a comment - I've voted. Does that count as +1?
        Hide
        Sam Marshall added a comment -

        Helen: I decided it did Thanks.

        I've committed this patch now. If anyone else has a chance to test it, that would be good. Of course I did test it already, but...

        Show
        Sam Marshall added a comment - Helen: I decided it did Thanks. I've committed this patch now. If anyone else has a chance to test it, that would be good. Of course I did test it already, but...
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Tested under 1.9 (HEAD is broken) and worked perfectly in one RSS reader able to handle those tags (to find such reader was the hard part, in fact)

        http://tt-rss.org (for reference)

        Thanks... closing. Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Tested under 1.9 (HEAD is broken) and worked perfectly in one RSS reader able to handle those tags (to find such reader was the hard part, in fact) http://tt-rss.org (for reference) Thanks... closing. Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: