| RE: [bpel-dev] Bug fix for BPELReader -- incorrect handling of forEach's "parallel" attribute |
Hi Hanyu, Oleg,
I understand the issue now. However, line 49 has a comment that says:
// (DO): Disabled to avoid <extensibilityAttributes> in sourceTab
// unknownExt.setElement((Element)node);
Assuming that "DO" stands for "Oleg Danilov" - Oleg, can you shed some
light here why this is commented out?
Mit freundlichen Grüßen / Kind regards
Simon Daniel Moser
Team Lead BPEL Editor - Websphere Integration Developer (WID)
IBM Software Group, Application and Integration Middleware Software
Business Process Solutions Development 1
Phone: +49-7031-16-4304 IBM Deutschland (Embedded
image moved
to file:
pic29083.gif)
Fax: +49-7031-16-4890 Schoenaicher Str. 220
E-Mail: smoser@xxxxxxxxxx 71032 Boeblingen
Germany
IBM Deutschland
Research &
Development
GmbH /
Vorsitzender des
Aufsichtsrats:
Martin Jetter
Geschäftsführung:
Erich Baier
Sitz der
Gesellschaft:
Böblingen /
Registergericht:
Amtsgericht
Stuttgart, HRB
243294
From: "Hanyu Xiao" <hxiao@xxxxxxxxx>
To: "BPEL Designer project developer discussions." <bpel-dev@xxxxxxxxxxx>
Date: 12.03.2009 01:53
Subject: RE: [bpel-dev] Bug fix for BPELReader -- incorrect handling of forEach's "parallel" attribute
Sent by: bpel-dev-bounces@xxxxxxxxxxx
Thanks, Simon.
Sorry I didn't make myself clear regarding the question for the line
"unknownExt.setElement((Element)node);". I'm referring to line 49 in
BPELUnknownExtensionDeserializer, which is currently commented out. I'm now
attaching a sample test case to demonstrate why it should not have been:
The sample BPEL process "ExtAttributeTest" has an extended attribute
tibex:uid="_oUGLAPiMEd2ss5OjE2CMIA"
Now ExtAttributeTest.java is trying to load the BPEL file and read this
attribute. You can comment/uncomment line 49 of
BPELUnknownExtensionDeserializer, and observe different console output.
Regards,
Hanyu
-----------------------
Hanyu Xiao
Tibco Software Inc.
3303 Hillview Ave..
Palo Alto, CA 94304
-----Original Message-----
From: bpel-dev-bounces@xxxxxxxxxxx [mailto:bpel-dev-bounces@xxxxxxxxxxx] On
Behalf Of Simon D Moser
Sent: Wednesday, March 11, 2009 1:58 AM
To: BPEL Designer project developer discussions.
Cc: BPEL Designer project developer discussions.;
bpel-dev-bounces@xxxxxxxxxxx
Subject: Re: [bpel-dev] Bug fix for BPELReader -- incorrect handlingof
forEach's "parallel" attribute
Hi Hanyu,
I looked at the patch, and released it to CVS. Please note that the Eclipse
Foundation does not encourage sending patches in email, but rather asks for
opening bugzillas and providing the patches there. Just to keep in mind for
the next time ;-)
As for the 2nd question: Not sure I understood it correctly -
BPELUnknownExtensionDeserializer does not have a line
"unknownExt.setElement((Element)node);" ?!
What is your concern here - must this be added ? Or ... Please shed some
more light on this ?
Mit freundlichen Grüßen / Kind regards
Simon Daniel Moser
Team Lead BPEL Editor - Websphere Integration Developer (WID)
IBM Software Group, Application and Integration Middleware Software
Business Process Solutions Development 1
Phone: +49-7031-16-4304 IBM Deutschland
(Embedded
image moved
to file:
pic31447.gif)
Fax: +49-7031-16-4890 Schoenaicher Str. 220
E-Mail: smoser@xxxxxxxxxx 71032 Boeblingen
Germany
IBM Deutschland
Research &
Development
GmbH /
Vorsitzender des
Aufsichtsrats:
Martin Jetter
Geschäftsführung:
Erich Baier
Sitz der
Gesellschaft:
Böblingen /
Registergericht:
Amtsgericht
Stuttgart, HRB
243294
From: "Hanyu Xiao" <hxiao@xxxxxxxxx>
To: "BPEL Designer project developer discussions."
<bpel-dev@xxxxxxxxxxx>
Date: 11.03.2009 01:24
Subject: [bpel-dev] Bug fix for BPELReader -- incorrect handling of
forEach's "parallel" attribute
Sent by: bpel-dev-bounces@xxxxxxxxxxx
Hi,
I'd like to submit a bug fix for BPELReader, which was not handling
forEach's "parallel" attribute correctly. Please see the attached patch
file.
In addition, we have a question regarding revision 1.2 of
BPELUnknownExtensionDeserializer. The extended attributes would not be
deserialized correctly with this line commented out:
unknownExt.setElement((Element)node);
Regards,
Hanyu
-----------------------
Hanyu Xiao
Tibco Software Inc.
3303 Hillview Ave..
Palo Alto, CA 94304
[attachment "BPELReader.patch" deleted by Simon D Moser/Germany/IBM]
_______________________________________________
bpel-dev mailing list
bpel-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/bpel-dev
[attachment "ExtAttributeTest.java" deleted by Simon D Moser/Germany/IBM]
[attachment "ExtAttributeTest.bpel" deleted by Simon D Moser/Germany/IBM]
_______________________________________________
bpel-dev mailing list
bpel-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/bpel-dev
Attachment:
pic29083.gif
Description: GIF image
