Details

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

      Operating System: All
      Platform: All

    • Issuezilla Id:
      113

      Description

      Hi,

      at this time, it isn't possible to have different modules with different
      namespaces but the same namespace prefix in rome. The problem is:
      RSS090Generator, RSS091UserlandGenerator, Atom03Generator and Atom10Generator
      are using generateModuleNamespaceDefs in BaseWireFeedGenerator, which tries to
      add all modules namespaces to the DOM and afterwards the not needed namespaces
      are removed by purgeUnusedNamespaceDeclarations in the same class.

      Now i went into trouble when trying to implement two modules with different
      version of a rss-extension that use both the same prefix (they are never used in
      the same feed of course but in the same application). This results in a JDOM
      exception when trying to add a further namespace with the same prefix that isn't
      needed in the feed.

      My solution is: Instead of first adding all available module-namespaces to the
      feed and afterwards removing those not needed by traversing the dom, I'm adding
      all the elements in my modules with namespaces added locally and afterwards
      traversing the complete dom, removing the local namespaces an collecting them
      for attaching at root level.

      A patch will follow soon.

      Greetings,

      Martin

        Activity

        Hide
        martinkurz added a comment -

        Created an attachment (id=26)
        patch for adding just the used namespaces in feed

        Show
        martinkurz added a comment - Created an attachment (id=26) patch for adding just the used namespaces in feed

          People

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

            Dates

            • Created:
              Updated: