[hibernate-issues] [Hibernate-JIRA] Commented: (HV-302) JAXB shouldn't be required when not using XML configuration

Gunnar Morling (JIRA) noreply at atlassian.com
Wed Mar 31 15:49:33 EDT 2010


    [ http://opensource.atlassian.com/projects/hibernate/browse/HV-302?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=36060#action_36060 ] 

Gunnar Morling commented on HV-302:
-----------------------------------

Patch for ValidatorFactoryImpl follows (initBeanMetaData() shouldn't be called, when no mapping streams are to be processed)

> JAXB shouldn't be required when not using XML configuration
> -----------------------------------------------------------
>
>                 Key: HV-302
>                 URL: http://opensource.atlassian.com/projects/hibernate/browse/HV-302
>             Project: Hibernate Validator
>          Issue Type: Improvement
>          Components: engine
>    Affects Versions: 4.1.0.Beta1
>            Reporter: Gunnar Morling
>            Assignee: Hardy Ferentschik
>             Fix For: 4.1.0
>
>
> By calling Configuration#ignoreXmlConfiguration() any XML config files (validation.xml, mapping XML files) are ignored by HV. Nevertheless JAXB is required on the class path, as in any case the class org.hibernate.validator.xml.XmlMappingParser (which imports javax.xml.bind.*) is loaded, also if no XML mapping files are to be processed.
> XmlMappingParser should only be loaded, when at least one XML mapping exists. While this makes no difference when running on Java 6 (as JAXB is part of the Java runtime itself), this allows to use HV on Java 5 without adding JAXB (and its transitive dependencies) to the class path.

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

        


More information about the hibernate-issues mailing list