Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: None
    • Labels:
      None
    • Number of attachments :
      1

      Description

      There are a couple of Doxia 1.0 specific quirks in the report-generation code, in particular several calls to sink.rawText(), which makes the plugin unusable with different sinks (cf the pdf plugin).

        Activity

        Hide
        Lukas Theussl added a comment -

        Attached patch would break compatibility with maven 2.0.9, I couldn't get it to work with the documented procedure (using maven 2.2.1), see the commented remarks and links. IMO we could just require maven 2.1 as prereq.

        Show
        Lukas Theussl added a comment - Attached patch would break compatibility with maven 2.0.9, I couldn't get it to work with the documented procedure (using maven 2.2.1), see the commented remarks and links. IMO we could just require maven 2.1 as prereq.
        Hide
        Vincent Siveton added a comment -

        agree

        Show
        Vincent Siveton added a comment - agree
        Hide
        Dennis Lundberg added a comment -

        Do we want to apply this patch before or after we release version 1.1?

        Show
        Dennis Lundberg added a comment - Do we want to apply this patch before or after we release version 1.1?
        Hide
        Lukas Theussl added a comment -

        Not necessary for this release IMO, unless you take the time (which I haven't) to test and confirm that it works with the pdf plugin and different maven versions. I'd say we apply the patch right after the release and have the snapshot tested for a while.

        Show
        Lukas Theussl added a comment - Not necessary for this release IMO, unless you take the time (which I haven't) to test and confirm that it works with the pdf plugin and different maven versions. I'd say we apply the patch right after the release and have the snapshot tested for a while.
        Hide
        Dennis Lundberg added a comment -

        That is my preference as well.
        I'll go ahead and start the release of 1.1 without this issue.

        Show
        Dennis Lundberg added a comment - That is my preference as well. I'll go ahead and start the release of 1.1 without this issue.
        Hide
        Karl-Heinz Marbaise added a comment -

        Fixed already. doxia version is currently at 1.4.

        Show
        Karl-Heinz Marbaise added a comment - Fixed already. doxia version is currently at 1.4.

          People

          • Assignee:
            Karl-Heinz Marbaise
            Reporter:
            Lukas Theussl
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: