Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 212146 - Query in an assign operation
Summary: Query in an assign operation
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: BPEL (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: BPEL UI Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-12-06 09:05 EST by Oleg Danilov CLA
Modified: 2022-10-03 11:11 EDT (History)
1 user (show)

See Also:

Attachments
query_in_assign.patch (847 bytes, patch)
2007-12-07 08:21 EST, Oleg Danilov CLA 		no flags Details | Diff
first_slash.patch (1.17 KB, patch)
2007-12-12 05:38 EST, Oleg Danilov CLA 		bbrodt: iplog+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Oleg Danilov CLA 2007-12-06 09:05:44 EST 
Attempted to use a query in an assign statement:

<bpws:copy>
  <bpws:from>...</bpws:from>
  <bpws:to part="payload" variable="output">
  <bpws:queryqueryLanguage="urn:oasis:names:tc:wsbpel:2.0:sublang:xpath1.0"><![CDATA[/tns:result]]></bpws:query>
...

Actually seems like we should generate query [tns:result] instead of [/tns:result], but I'm not very familiar with this.
Comment 1 Oleg Danilov CLA 2007-12-07 08:21:40 EST 
Created attachment 84725 [details]
query_in_assign.patch

Simon's answer:

> you're absolutely correct - the query should not start with a '/' - not on the first token in this case.

So seems like it's enought to remove builder.append("/"). Patch has been attached.
Comment 2 Simon Moser CLA 2007-12-08 07:08:32 EST 
Oleg, 

this is not good enough. The fix that you proposed here removes *all* slashes. 
Imagine a complex XSD with many nested types. 
So, previouly you had /xsd:elm1/xsd:elm2/xsd:elm3, with your fix this becomes
 xsd:elm1xsd:elm2xsd:elm3. 

However, it should become  xsd:elm1/xsd:elm2/xsd:elm3´- so only the first slash is removed.
Comment 3 Oleg Danilov CLA 2007-12-12 05:38:48 EST 
Created attachment 85048 [details]
first_slash.patch

I see. Here is an updated patch. It's just removes the first character from the string.
Comment 4 Simon Moser CLA 2007-12-30 11:05:29 EST 
reviewed and released the patch, the new patch works.
Comment 5 Robert Brodt CLA 2012-04-18 14:45:10 EDT 
Comment on attachment 85048 [details]
first_slash.patch

Updated iplog flag for v1.0 release review.