[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [bpel-dev] Beautify XML for extension activities
|
Hi Bob,
what I'm referencing are the children of the child element of <extensionActivity>. As far as I understand your code you are removing any text nodes, recursivly?
To my p.s. - I'm talking about the fact you already mentioned in your reply on 01/14.
In XML you have:
<sequence>
<extensionActivity>
<proc:invokePipelet name="...">
<proc:pipelet class="..." />
...
</proc:invokePipelet>
</extensionActivity>
</sequence>
But the created objects are: SequenceImpl -> InvokePipeletImpl -> PipeletImpl
Further more, even if I set the connected element of InvokePipeletImpl to <proc:invokePipelet> in my deserializer, you set it to <extensionActivity> later on. The resulting problem: If I have to add/remove child elements of invokePipelet because of structural changes in the object, I always need to find the correct child of the connected <extensionActivity>.
Maybe we should introduce an own object for <extensionActivity> and change the example project accordingly? That way the extension activities would work like the rest of the elements.
Tobias
________________________________
Von: bpel-dev-bounces@xxxxxxxxxxx im Auftrag von Bob Brodt
Gesendet: Mo 31.1.11 22:40
An: BPEL Designer project developer discussions.
Betreff: Re: [bpel-dev] Beautify XML for extension activities
Hi Tobias,
If you have a look at section 10.9 of the WS-BPEL spec, it says that the child of an <extensionActivity> MUST be a single element, so that means it can't have any Text elements. I think my fix is correct.
I'm not sure I understand your P.S. - can you explain further? Where is the <extensionActivity> element connected, and to what? The BPEL editor Diagram page correctly shows an InvokePipelet activity.
_______________________________________
Robert ("Bob") Brodt
Senior Software Engineer, JBoss Riftsaw
JBoss by Red Hat
----- Original Message -----
> Hi Bob,
>
> your solution is dangerous, as a child element may contain text nodes
> with more than whitespaces in it and you remove any text nodes, even
> those with real text in it.
>
> I didn't noticed the equals and hashCode methods up to now, you are
> right, the current implementaion is incorrect for many use cases. I
> removed them now.
>
> Greetings,
> Tobias
>
> p.s. I just noticed: After loading the pipeline the
> InvokePipelet-Object has the <extensionActivity> element connected and
> not the <invokePipelet> as expected. Thats confusing, especially in
> DOM updates and should be documented.
>
> ________________________________
>
[snip]
<<winmail.dat>>