[
https://issues.apache.org/jira/browse/AXIS2-3209?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Rich Scheuerle resolved AXIS2-3209.
-----------------------------------
Resolution: Fixed
Though they seem similar on the surface, these two code pieces are very
different.
org.apache.axis2.saaj.util.SAAJUtil.
- This is a utility class that is part of the Axis2 SAAJ implementation.
- This methods in the class convert from OM to DOM (doom).
- There are no javax.xml.saaj references in the class.
org.apache.axis2.jaxws.message.util.SAAJConverter
- This class converts OM to SAAJ objects.
- It intentionally is not coded to a specific SAAJ implementation (i.e. note
there are no references to Axis2 SAAJ objects).
There are some consumers of Axis2 that use other SAAJ implementations (Sun,
IBM, etc.). This code must be implementation
neutral to support those scenarios.
So at this time I don't see a strong reason to merge these two code pieces
together. Since one is integral to a specific implementation
and the other is intentional implementation neutral, merging the code may lead
to problems.
Thanks for the ideas Gennady. Please let me know if you have any other ideas
for merging code.
> Consolidate functionality of org.apache.axis2.saaj.util.SAAJUtil and
> org.apache.axis2.jaxws.message.util.SAAJConverter
> ----------------------------------------------------------------------------------------------------------------------
>
> Key: AXIS2-3209
> URL: https://issues.apache.org/jira/browse/AXIS2-3209
> Project: Axis 2.0 (Axis2)
> Issue Type: Improvement
> Components: jaxws, kernel
> Reporter: Gennady Shumakher
> Assignee: Rich Scheuerle
> Priority: Minor
>
> Both org.apache.axis2.saaj.util.SAAJUtil and
> org.apache.axis2.jaxws.message.util.SAAJConverter provides the ability to
> convert between SAAJ and OM Constructed elements the consolidation of twos
> should be considered.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
