You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Christian Schulte (JIRA)" <ji...@codehaus.org> on 2009/12/11 16:42:55 UTC

[jira] Created: (MCHANGES-190) HTML in changes.xml stopped working

HTML in changes.xml stopped working
-----------------------------------

                 Key: MCHANGES-190
                 URL: http://jira.codehaus.org/browse/MCHANGES-190
             Project: Maven 2.x Changes Plugin
          Issue Type: Bug
          Components: changes-report
    Affects Versions: 2.3
         Environment: Maven version: 2.0.10
Java version: 1.5.0_17
OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"

            Reporter: Christian Schulte
            Priority: Critical
         Attachments: changes.xml

The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.




-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=249420#action_249420 ] 

Christian Schulte commented on MCHANGES-190:
--------------------------------------------

Setting 'escapeHTML' to 'false' the patch works for me.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.patch, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=202367#action_202367 ] 

Dennis Lundberg commented on MCHANGES-190:
------------------------------------------

Hi Christian

It was I who made the patch from MCHANGES-189. I was struggling to get issues that contains html-entities to show up correctly in both the announcement and in the changes-report. I'll upload an example shortly showing what I mean.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "SebbASF (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=217872#action_217872 ] 

SebbASF commented on MCHANGES-190:
----------------------------------

Note that version 2.0 also allowed HTML markup which was not in a CDATA section.

This has also been used, e.g. to add links and emphasis, both of which would also be useful in PDF output.

==

As to announcements, these are processed using a Velocity template. 
Velocity is powerful enough to be able to handle markup so surely it ought to be possible to fix this somehow?

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Lukas Theussl (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=212947#action_212947 ] 

Lukas Theussl commented on MCHANGES-190:
----------------------------------------

bq. no processing of the content from the changes.xml file should be performed at all.

How do you want to generate anything without processing the content of changes.xml?

bq. What if someone decides to send out HTML announcement mails ?

Then the content of changes.xml should be processed to produce an HTML mail (eg via an option in the changes:announcement-mail goal).

The point of this issue is really just backward compatibility, the current behavior of the plugin is perfectly correct IMO. The fact that in earlier versions you could put arbitrary html inside CDATA is an undocumented (and I would guess unintentional) feature, which should really be considered a bug (it only works for html output, ie site generation, but you can't use it for eg producing a pdf of the changes report via the pdf plugin).

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Alexander Daniel (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=235177#action_235177 ] 

Alexander Daniel commented on MCHANGES-190:
-------------------------------------------

> Note that version 2.0 also allowed HTML markup which was not in a CDATA section.

For me HTML markup only works in a CDATA section even with version 2.0

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=249628#action_249628 ] 

Dennis Lundberg commented on MCHANGES-190:
------------------------------------------

Right. Do you think I should add a disclaimer in the parameter documentation about this? Something like this:

"Putting any kind of markup inside a CDATA section might mess up the Changes Report or other generated documents, such as PDFs, that are based on your changes.xml file."

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190-2.patch, MCHANGES-190.patch, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christian Schulte updated MCHANGES-190:
---------------------------------------

    Attachment: changes.xml

Example file containing HTML tags.


> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=203725#action_203725 ] 

Dennis Lundberg commented on MCHANGES-190:
------------------------------------------

I think we need to have a configuration parameter for this.

Some people, like me, wants to use the issue text for both the announcement and the report and can't have html code in the changes.xml file. Others, like yourself, wants to enhance their issue text with html code and only use it for the report.

Now we only need to figure out a characteristic name for the parameter and a good default value. The parameter will go into the changes-report goal. How about allowHtml with a default value of "false"?

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christian Schulte updated MCHANGES-190:
---------------------------------------

    Attachment: MCHANGES-190.zip

The changes file using HTML entities for the < and > characters does not work for me also.
Please unzip MCHANGES-190.zip and execute

{code}
mvn site
{code}

and compare the result using the 2.2 version and the 2.3 version. The changes.xml file therein contains two actions using the same HTML. In a cdata section and using HTML entities.

{code}
      <action><![CDATA[<b>bold</b> - <i>italic</i>]]></action>
      <action>&lt;b&gt;bold&lt;/b&gt; - &lt;i&gt;italic&lt;/i&gt;</action>
{code}

Using version 2.2, the raw HTML is rendered so a browser actually displays bold and italic text. Using version 2.3, the HTML got encoded so a browser displays the tags.


> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Lukas Theussl (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=249056#action_249056 ] 

Lukas Theussl commented on MCHANGES-190:
----------------------------------------

I don't agree with 2. I think the most transparent way *is* to change the XSD. However, I would like to see your proposed implementation first, if you can come up with an solution that doesn't need a new xsd and that satisfies everyone, I will happily accept it. :)

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Lukas Theussl (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=203728#action_203728 ] 

Lukas Theussl commented on MCHANGES-190:
----------------------------------------

According to the current changes.xsd schema, the action element in changes.xml has text content only, ie no markup is allowed inside <action>. So I think the current behavior is actually correct. However, wrt to backward compatibility and functionality, this is not very satisfactory.

Note that in maven 1, the <action> element could contain arbitrary xdoc markup, for the announcement only the text content was extracted.

I think the proper solution would be to mimic this behavior, ie generalize the xsd schema and process the content of changes.xml according to where the output goes. A configuration parameter seems unnecessary to me, at least you would have to make it consistent with the xsd.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christian Schulte updated MCHANGES-190:
---------------------------------------

    Attachment: MCHANGES-190-2.patch

Although unrelated to this issue, looking at the announcement mail feature, I noticed that the 'announcement-generate' goal supports template encodings but the 'announcement-mail' goal does not. This patch adds a 'templateEncoding' parameter to the 'announcement-mail' goal.


> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190-2.patch, MCHANGES-190.patch, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Closed: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg closed MCHANGES-190.
------------------------------------

       Resolution: Fixed
    Fix Version/s: 2.4
         Assignee: Dennis Lundberg

Fixed in [r1054123|http://svn.apache.org/viewvc?view=revision&revision=1054123].

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Assignee: Dennis Lundberg
>            Priority: Critical
>             Fix For: 2.4
>
>         Attachments: changes.xml, changes.xml, MCHANGES-190-2.patch, MCHANGES-190.patch, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=212946#action_212946 ] 

Christian Schulte commented on MCHANGES-190:
--------------------------------------------

When generalizing the xsd schema, support for action details (in addition to short descriptions) could be added so that current behaviour could be retained. Those details could be ignored by default when generating announcement mails. Seems the intended use of the changes.xml file is to specify a *summary* of changes rather than specifying a detailed changes report.

> ...and process the content of changes.xml according to where the output goes.

I would still say that no processing of the content from the changes.xml file should be performed at all. What if someone decides to send out HTML announcement mails ? Please note that I am currently only using the changes-report goal during site generation so I am very unfamiliar with the announcement mail feature.


> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg updated MCHANGES-190:
-------------------------------------

    Attachment: MCHANGES-190.patch

Here is my proposed patch to fix this issue.

It should satisfy those users who want to use HTML code inside a CDATA section to format the contents of their <action>s in the changes report. They would simply add this to their POM:
{noformat}
        <configuration>
          <escapeHTML>false</escapeHTML>
        </configuration>
{noformat}

Users of the {{announcement-generate}} goal are unaffected by this change (and this issue for that matter). If you put HTML code inside your <action>s it will be written to the announcement file.

If someone later on want to add a new feature to this plugin that would allow sending HTML emails, that would work too.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.patch, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Jörg Hohwiller (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=230409#action_230409 ] 

Jörg Hohwiller commented on MCHANGES-190:
-----------------------------------------

IMHO adding markup as text in CDATA sections was always a hack.
The best way to fix this would be to allow xsd:any in action and put the content as is in the result.
Then we also do not need a trigger as commented in
"Dennis Lundberg added a comment - 21/Dec/09 2:31 PM"
because one can still have plain text in CDATA and does not need to care about escaping.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=248959#action_248959 ] 

Dennis Lundberg commented on MCHANGES-190:
------------------------------------------

Here's an attempt to objectively summarize the comments on this issue so far.

# We want it to be possible to somehow have HTML markup in the changes report, at least for backwards compatibility reasons.
# The most transparent way is to put HTML markup inside CDATA sections, that way we don't have to change the XSD.
# Different mojos have different things they want achieve, and so do different users. We need some means to allow this diversity.


Please let med know if we agree on these points.

I'll follow up after that with my suggested way to implement them.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Joerg Schaible (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=209929#action_209929 ] 

Joerg Schaible commented on MCHANGES-190:
-----------------------------------------

This *is* really a major regression. I had to detect now that roughly 50 change reports for our last release cycle are simply rubbish! We added HTML for years in CDATA sections without any problems. This was valid XML and conforming to the schema.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Lukas Theussl (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=249626#action_249626 ] 

Lukas Theussl commented on MCHANGES-190:
----------------------------------------

@Christian: If it's unrelated, please attach it to a new issue.

@Dennis: as a hack, this is fine, I am just somewhat reluctant as introducing this means that we 'officially' support this feature, which was never the intention. Just making sure you are aware of the implications... ;)

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes.xml
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml, MCHANGES-190-2.patch, MCHANGES-190.patch, MCHANGES-190.zip
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MCHANGES-190) HTML in changes.xml stopped working

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MCHANGES-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg updated MCHANGES-190:
-------------------------------------

    Attachment: changes.xml

A file with < and > encoded as html entities. This file works as expected for both the announcement and the changes-report.

> HTML in changes.xml stopped working
> -----------------------------------
>
>                 Key: MCHANGES-190
>                 URL: http://jira.codehaus.org/browse/MCHANGES-190
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: changes-report
>    Affects Versions: 2.3
>         Environment: Maven version: 2.0.10
> Java version: 1.5.0_17
> OS name: "linux" version: "2.6.32-686" arch: "i386" Family: "unix"
>            Reporter: Christian Schulte
>            Priority: Critical
>         Attachments: changes.xml, changes.xml
>
>
> The fix for MCHANGES-189 does not seem to be correct. A changes.xml file containing HTML-Tags got rendered correctly using version 2.2. Starting with version 2.3, HTML-Tags will be encoded using HTML entities for the '<' and '>' characters leading to the actual tags getting shown in the report. See the attached example changes.xml file containing HTML no longer working with version 2.3.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira