OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

sca-assembly message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: Re: [sca-assembly] NEW ISSUE (1.2): Business data filter elementneeds better name, definition


On 7/22/2010 12:12 PM, Eric Johnson wrote:
> Target assembly-1.2 WD01 (PDF, regular form, not diff)
>
> Description:
> In section 9.3.1
> The current name of the Business data filter element has an extremely
> generic name of "body.xpath1" - this is not a useful name for describing
> what it does, and putting xpath1 in the name is inconsistent with the
> other places where XPath applies (properties) and presumably has a version.
>
> Also, the description says the context node is : "Context Node: the root
> element of the document being searched based upon the subject. In this
> case (the Business Data Subject) it is the root element of the event
> business data."
>
> Assuming that the business data element is the root element of the
> transport payload is confusing, because it could be nested under some
> enveloping scheme (SOAP, anyone?)  So it would seem that "root element
> of the document", and "root element of the event business data" can mean
> two different things.
>
> Proposal:
>
> Change the name of the filter to eventBodyFilter.
>

What's wrong with the current name? The filter element names capture two 
things: the subject of the filter and the dialect. Therefore, 
body.xpath1. I suppose eventBody.xpath1 would be equally appropriate but 
a little redundant.

> Change the description of the Context Node to:
> "The root element of the event business data."

+1
The term "Business Data Subject is never defined.

-Anish
--

>
>
> ---------------------------------------------------------------------
> To unsubscribe from this mail list, you must leave the OASIS TC that
> generates this mail.  Follow this link to all your TCs in OASIS at:
> https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php
>


[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]