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

 


Help: OASIS Mailing Lists Help | MarkMail Help

odata message

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


Subject: [OASIS Issue Tracker] Issue Comment Edited: (ODATA-250) For instance annotations in Atom, "Annotation" element, "Type", "Term" and "Target" attributes should be lower case


    [ http://tools.oasis-open.org/issues/browse/ODATA-250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=32557#action_32557 ] 

Ralf Handl edited comment on ODATA-250 at 2/25/13 5:56 PM:
-----------------------------------------------------------

When applying this issue I noticed two inconsistencies.

(1) The metadata:annotation element allows values to be specified in "attribute form" as String="Some Value" or Int="42". With everything else lowercase these attributes look odd,th so we COULD lowercase them. But in addition they are incomplete compared to CSDL (e.g. no Duration or DateTimeOffset), and in CSDL their CSDL counterparts are (and hopefully will remain) CamelCased, so however we adapt them, they stay inconsistent.

Also they add little value over element notation in Atom, which is famous for its verbosity. A handful of shortcuts won't change that.

So I completely removed the Attribute Value Notation from Atom, and I like the new brevity :-)

(2) The target attribute of the metadata:annotation attribute is optional, and if omitted it means "the surrounding parent". In addition it allowed the special value "." to also mean "the surrounding parent". I don't like two ways to specify the same meaning, so I removed the special value ".".

Please comment if you disagree.

      was (Author: ralfhandl):
    When applying this issue I noticed two inconsistencies.

(1) The metadata:annotation element allows values to be specified in "attribute form" as String="Some Value" or Int="42". With everything else lowercase these attributes look odd,th so we COULD lowercase them. But in addition they are incomplete compared to CSDL (e.g. no Duration or DateTimeOffset), and in CSDL their CSDL counterparts are (and hopefully will remain) CamelCased, so however we adapt them, they stay inconsistent.

Also they add little value over element notation in Atom, which is famous for its verbosity. A handful of shortcuts won't change that.

So I completely removed the Attribute Value Notation from Atom, and I like the new brevity :-)

(2) The target attribute of the metadata:annotation attribute is optional, and if omitted it means "the surrounding parent". In addition it allowed the special value "." to also mean "the surrounding parent". The natural choice would have been "empty means the same as omitting it", and we use "empty" as "surrounding parent" already in CSDL Path expressions.

So I removed the special meaning of "." and gave its meaning to "" (empty string).

Please comment if you disagree.
  
> For instance annotations in Atom, "Annotation" element, "Type", "Term" and "Target" attributes should be lower case
> -------------------------------------------------------------------------------------------------------------------
>
>                 Key: ODATA-250
>                 URL: http://tools.oasis-open.org/issues/browse/ODATA-250
>             Project: OASIS Open Data Protocol (OData) TC
>          Issue Type: Bug
>          Components: OData ATOM Format 
>    Affects Versions: V4.0_WD01
>         Environment: [Proposed]
>            Reporter: Michael Pizzo
>            Assignee: Ralf Handl
>             Fix For: V4.0_WD01
>
>
> In the current [ATOM] spec, the "Annotation" element, as well as the child "Type", "Term" and "Target" attributes are defined in upper  case. This is inconsistent with other element that we define in the ATOM payload (for example, the metadata:function element with attributes "metadata" "target" and "title", the metadata:error element and its attributes, etc.). 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://tools.oasis-open.org/issues/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


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