You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commons-dev@ws.apache.org by "Nick Gallardo (JIRA)" <ji...@apache.org> on 2007/01/31 19:37:05 UTC
[jira] Commented: (WSCOMMONS-159) OMSourcedElementImpl serialize()
violates OMNode.serialize()
[ https://issues.apache.org/jira/browse/WSCOMMONS-159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12469147 ]
Nick Gallardo commented on WSCOMMONS-159:
-----------------------------------------
This would be a very helpful fix. I agree with Rich that even though it's an OMSourcedElement, it should still respect the OMNode interface and cache the contents rather than consuming.
> OMSourcedElementImpl serialize() violates OMNode.serialize()
> -------------------------------------------------------------
>
> Key: WSCOMMONS-159
> URL: https://issues.apache.org/jira/browse/WSCOMMONS-159
> Project: WS-Commons
> Issue Type: Bug
> Components: AXIOM
> Reporter: Rich Scheuerle
> Assigned To: Rich Scheuerle
> Attachments: patch.txt
>
>
> The OMNode serialize method indicates that the OM is cached.
> /**
> * Serializes the node with caching.
> *
> * @param xmlWriter
> * @throws XMLStreamException
> */
> public void serialize(XMLStreamWriter xmlWriter) throws XMLStreamException;
> But OMSourcedElementImpl consumes the message. This results in errors when an OM tree contains a mixture of normal nodes and OMSourcedElements.
> The solution is to change OMSourcedElement to respect the OMNode "serialize" contracts.
> I am working on a patch containing the corrected code and validation tests.
--
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: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org