I could not exactly understand what you mean by putting comments creation stuff in StaxBuilder. It has to be there in next() method which is currently present in both OM and SOAP builders. It means the entire next() method should go in StaxBuilder?

I’ll anyway create a JIRA issue so that we do not miss it.

-Ashutosh


From: Eran Chinthaka [mailto:[EMAIL PROTECTED]]
Sent: Thursday, July 28, 2005 1:06 AM
To: [email protected]
Subject: RE: [Axis2] staxSOAPModelBuilder

 

Hi Ashu, Venkat and all,

 

I’m 0- on making StAXOMBuilder extend from StAXOMBuilder. Remember our intention in “Axis2” is to make a fast and efficient object model for SOAP processing. We do lot of stuff specific to SOAP in StAXSOAPModelBuilder. And we know for sure StAXSOAPModelBuilder doesn’t need to support full infoset.

AND, if you look at the code carefully, you might have seen that for each and every information item creation, I have introduced a method in the StAXOMBUilder. Example, createOMELement() etc.,

 

So if you wanna share code between SOAP builder and OM builder, please do put them in StAXBuilder. So perhaps, Ashu you can put your comments creation stuff in StAXBuilder.

 

Regards,

Chinthaka

 


From: Shahi, Ashutosh [mailto:[EMAIL PROTECTED]
Sent: Wednesday, July 27, 2005 1:06 PM
To: [email protected]
Subject: [Axis2] staxSOAPModelBuilder

 

Hi OMers  ;-),

As part of XML Infoset, there was some code added to next() method in staxOMBuilder to process comments etc.

But, we have staxSOAPModelBuilder also, where we do not have any such capability yet. Now comments can be part of soapMessage as well.

I have a test soap message :

"<?xml version='1.0' encoding='UTF-8'?>" +

                                    "<!-- Comment -->" +

                                    "<env:Envelope xmlns:env='http://schemas.xmlsoap.org/soap/envelope/'>" +

                                    "<env:Body><echo><arg0>Hello</arg0></echo></env:Body>" +

                                    "</env:Envelope>";

 

And doing something like:

stAXSOAPModelBuilder =

                        new StAXSOAPModelBuilder(

                                XMLInputFactory.newInstance()

                        .createXMLStreamReader(inputStream));

Where input stream has soap message with comments gives an OMException.

 

Now my question is:

Both StaxSOAPModelBuilder and StaxOMBuilder extend the abstract class StaxBuilder which has next() as abstract method. Why not put the functionality in next() method of StaxBuilder itself? Or atleast we should process comments in StaxSOAPModelBuilder as well.

 

-Ashutosh

 

Reply via email to