Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Nightly Builds
    • Fix Version/s: None
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      Discussion ongoing on dev list about actual implementation, but I wanted to put
      out a test case that illustrates the problem. Basically, the current Digester
      implementation concatenates all the body text together in a particular element,
      without care for the separation of the text parts by XML elements.

        Activity

        Justin Edelson created issue -
        Hide
        Justin Edelson added a comment -

        Created an attachment (id=11065)
        adds some value objects and a test case to org.apache.commons.digester.mixed

        Show
        Justin Edelson added a comment - Created an attachment (id=11065) adds some value objects and a test case to org.apache.commons.digester.mixed
        Hide
        Simon Kitching added a comment -

        Hi,

        I've committed some code to CVS at src/examples/api/document-markup.
        This code demonstrates one method for subclassing Digester to implement the
        parsing of "document markup" style XML.

        After the 1.6 release would be a good time to push for a solution "built-in" to
        Digester. I'm in favour of this in general, but not in favour of holding the
        release up to do it now, given that the example seems to show quite a reasonable
        solution.

        Justin: If this is satisfactory, could you please close the bug?
        If you have any concerns, or suggestions for better solutions, then please bring
        them up here or on the mailing list ASAP, to get them in before 1.6.

        Thanks, Simon

        Show
        Simon Kitching added a comment - Hi, I've committed some code to CVS at src/examples/api/document-markup. This code demonstrates one method for subclassing Digester to implement the parsing of "document markup" style XML. After the 1.6 release would be a good time to push for a solution "built-in" to Digester. I'm in favour of this in general, but not in favour of holding the release up to do it now, given that the example seems to show quite a reasonable solution. Justin: If this is satisfactory, could you please close the bug? If you have any concerns, or suggestions for better solutions, then please bring them up here or on the mailing list ASAP, to get them in before 1.6. Thanks, Simon
        Hide
        Simon Kitching added a comment -

        Ok, as there has been no further comments, I will close this item.

        Show
        Simon Kitching added a comment - Ok, as there has been no further comments, I will close this item.
        Henri Yandell made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 28068 12341380
        Henri Yandell made changes -
        Component/s Digester [ 12311111 ]
        Affects Version/s Nightly Builds [ 12311648 ]
        Assignee Jakarta Commons Developers Mailing List [ commons-dev@jakarta.apache.org ]
        Key COM-1228 DIGESTER-56
        Project Commons [ 12310458 ] Commons Digester [ 12310471 ]
        Henri Yandell made changes -
        Affects Version/s Nightly Builds [ 12311810 ]
        Henri Yandell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Justin Edelson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development