axis-java-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chuck Williams <>
Subject Re: [Axis2 0.95] SERIOUS BUG FIX + Recursive data types and choice particles
Date Mon, 27 Mar 2006 05:47:21 GMT
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<body bgcolor="#ffffff" text="#000000">
Thanks Ajith!&nbsp; I've posted a new issue
(<a class="moz-txt-link-freetext" href=""></a>)
with the patches
rather than applying them to the separate issues since I didn't have
the patches separated.&nbsp; Hope this is not too troublesome.<br>
As I use jira for other Apache projects (e.g., Lucene), I should have
known better. <br>
I greatly appreciate you efforts and that of the whole Axis2 team!<br>
 style="font-size: smaller; font-style: italic; margin-bottom: -2.4em;"><br>
<span>"Ajith Ranabahu" <a class="moz-txt-link-rfc2396E" href="">&lt;;</a></span>
wrote on
<span>03/26/2006 07:08:46 PM</span>:<br>
  <pre wrap="">Hi Chuck,
Thanks a lot for the update. I've already added the mentioned issues
to the Jira. Just for the record, it is better if you can attach the
patch into the Jira since Jira has this grant licence thing. So ASF is
safe if the patch came through Jira.
BTW we very much appreciate your efforts with Axis2. Hopefully we'll
be able to have all the features you guys need in Axis2 1.0 release


On 3/27/06, Chuck Williams <a class="moz-txt-link-rfc2396E" href="">&lt;;</a>
  <blockquote type="cite">
    <pre wrap=""> Ajith,

 Thanks for the update.  Since I need choice to upgrade to 0.95, I went
ahead and implemented it.

 ******  In the course of this, I found a serious issue in 0.95.  The
setting of the choice variable in ADBBeanTemplate is checked in,
 but with the wrong code.  It is always true, which messes up the pull
parser seriously.  I.e., for non-choice beans, there are local trackers
 and all but one are false, so code that looks for these will miss valid

 There is another serious issue, in that a bug I reported earlier was
propagated into commons.  Specifically, OMStAXWrapper does not obey the
contract of getElementText() -- it does not leave the reader at the end
element.  I had to work around that bug in the parser, and hope it gets
fixed.  There is a detailed comment with the workaround.

 The attached patches resolve these issues:

Correct the improper setting of choice that causes Axis2 to think all adb
beans are choices.

Implement choice
Fix recursive data types
Probably fix general cases still in the parser for minOccurs=0 (e.g., the
event following an empty array need not be the outer end element) and where
subelements with same name as parent element would not work.  (I didn't
trigger these problems with tests, but they seem to be there from looking at
the code -- they are definitely not there with the patch.)

Fix pull parser for serialization of 0-length arrays
Work around the OMStAXWrapper problem with getElementText() All Axis2 tests
pass, and our application which has complex wsdl runs properly.



 Ajith Ranabahu wrote:
 Hi Chuck,
Sorry for the late answer but adding choice support is still underway.
We have marked choice support as a requirement for 1.0.
I'll see to that recursive datatypes are handled properly with your pach.

On 3/26/06, Chuck Williams <a class="moz-txt-link-rfc2396E" href="">&lt;;</a>

 This time with the attachment:

 Chuck Williams <a class="moz-txt-link-rfc2396E" href="">&lt;;</a>
wrote on 03/26/2006 01:41:54 AM:

 Hi All,

Axis2 0.95 still does not handle recursive data types properly. The
SchemaCompiler recurses infinitely and generates a stack overflow. I've
attached my patch to fix this upgraded for 0.95. This is a patch
against modules/codegen/src.

I see that much of the code to implement choice particles was committed,
but that the new parser does not handle them. I need to get them
working again. Is there already work underway to do this, and/or was an
approach determined? If so, I'd appreciate any information about that.
If not, I'll figure out how to do it with the new parser and send
another patch.



Ajith Ranabahu

  <pre wrap=""><!---->

Ajith Ranabahu

View raw message