Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: current
    • Fix Version/s: milestone 1
    • Component/s: www
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      110

      Description

      In RFC4287, an Atom person construct may be extended, from page 35:

      1. Person Construct

      atomPersonConstruct =
      atomCommonAttributes,
      (element atom:name

      { text }

      & element atom:uri

      { atomUri }

      ?
      & element atom:email

      { atomEmailAddress }

      ?
      & extensionElement*)

      Rome currently does not support this. This could be supported by making
      SyndPerson extend Extendable, adding new configuration items to rome.properties,
      ie atom_1.0.person.ModuleGenerator.classes etc, and updating the relevant parsers.

        Activity

        Hide
        jamesroper added a comment -

        Created an attachment (id=24)
        Proposed patch to implement, implements for Atom 1.0 only (doesn't make sense for any other syndication format).

        Show
        jamesroper added a comment - Created an attachment (id=24) Proposed patch to implement, implements for Atom 1.0 only (doesn't make sense for any other syndication format).
        Hide
        jamesroper added a comment -

        I've attached a patch for this issue, diffed against CVS HEAD on November 24.
        Comments/criticisms most welcome.

        Show
        jamesroper added a comment - I've attached a patch for this issue, diffed against CVS HEAD on November 24. Comments/criticisms most welcome.
        Hide
        nicklothian added a comment -

        I don't see any problems with this patch (except the lack of test cases), so I
        am applying.

        Show
        nicklothian added a comment - I don't see any problems with this patch (except the lack of test cases), so I am applying.
        Hide
        nicklothian added a comment -

        Patch applied

        Show
        nicklothian added a comment - Patch applied

          People

          • Assignee:
            rome-issues
            Reporter:
            jamesroper
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: