[ 
http://jira.codehaus.org/browse/DOXIA-133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_104017
 ] 

Vincent Siveton commented on DOXIA-133:
---------------------------------------

>From an API point of view, I am not at ease with Doxia.isXmlSource( String 
>parserId ). I propose you a better approach which sounds better for me:

{code:title=Doxia.java|borderStyle=solid}
...
    /**
     * @param parserId Identifier for the parser to use.
     * @return the parser defining for parserId.
     * @throws ParserNotFoundException if no parser could be found
     * for the given id.
     */
    Parser getParser( String parserId )
        throws ParserNotFoundException;
...
{code} 

{code:title=Parser.java|borderStyle=solid}
...
    /**
     * @return xml for xml parser, text for text parser
     */
    String getType();
...
{code} 

{code:title=DefaultSiteRenderer.java|borderStyle=solid}
...
             boolean isXml = doxia.getParser( renderingContext.getParserId() 
).getType().equals("xml");
...
{code} 

Thoughts?

> default XML encoding (UTF-8) or XML encoding set in XML files is ignored: 
> inputEncoding is used instead
> -------------------------------------------------------------------------------------------------------
>
>                 Key: DOXIA-133
>                 URL: http://jira.codehaus.org/browse/DOXIA-133
>             Project: Maven Doxia
>          Issue Type: Bug
>          Components: Core, Module - Docbook Simple, Module - Fml, Module - 
> Xdoc, Module - Xhtml, Site Renderer
>    Affects Versions: 1.0-alpha-8
>            Reporter: Herve Boutemy
>             Fix For: 1.0-alpha-9
>
>         Attachments: DOXIA-133_doxia-siterenderer.diff, DOXIA-133_doxia.diff, 
> DOXIA-133_doxia.diff, DOXIA-133_doxia.diff
>
>
> Encoding can be specified per file, in the XML header: <?xml version="1.0" 
> encoding="xxx"?>, or defaults to UTF-8
> But DefaultSiteRenderer class always read files with inputEncoding: {{reader 
> = new InputStreamReader( new FileInputStream( fullPathDoc ), 
> context.getInputEncoding() );}}
> When the source file is XML (xdoc, xhtml), should use XmlReader from 
> PLXUTILS-11 to detect the XML stream encoding instead.
> Test case included in MSITE-239, site-plugin-test14

-- 
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

        

Reply via email to