[MPLUGIN-43] Adding validation in PluginXdocGenerator class?

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.4
• Assignee: Siveton Vincent
• Reporter: Siveton Vincent
• Created: 05/Aug/05 06:21
• Updated: 14/Mar/08 14:04
• Resolved: 02/Feb/08 08:43

Description

If we call the plugin:xdoc goal on a mojo defined like the following:
/**

• Generates something <b> for the project.
• @goal agoal
  */
  The generated file (mymojo-mojo.xml) is not valid but the build is successful.

Thus, the site:site goal generates the following exception:
Caused by: org.codehaus.plexus.util.xml.pull.XmlPullParserException: end tag name </p> must be the same as start tag <b> from line 7 (position: TEXT seen ...<b> Checkstyle definitions defined by the project.</p>... @7:150)

I think that the PluginXdocGenerator class should validate the generated file.

Issue Links

• relates to: MPLUGIN-96 Handle character encoding properly in makeHtmlValid()
• relates to: MPLUGIN-90 Refactor makeValidHtml into PluginUtils for better reusage