Uploaded image for project: 'OASIS Open Data Protocol (OData) TC'
  1. OASIS Open Data Protocol (OData) TC
  2. ODATA-1371

Improve structure of CSDL clause 4

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: V4.01_OS
    • Fix Version/s: V4.01_ERRATA01
    • Component/s: CSDL JSON , CSDL XML
    • Labels:
      None
    • Environment:

      Proposed

    • Proposal:
      Hide

      Reword section 4.3 along similar lines as 4.2:
      "In addition to including whole schemas with all model constructs defined within that schema, a reference may include annotations."

      Make both references to references be links to section 4.1.

      Show
      Reword section 4.3 along similar lines as 4.2: "In addition to including whole schemas with all model constructs defined within that schema,  a reference may include  annotations." Make both references to references be links to section 4.1.

      Description

      The subclause levels of this section suggest that Included Schema and Included Annotations are independent declarations from the $Reference declaration.  However, they should be documented as parts of the $Reference declaration.

      Related to this, 4.1 Reference asserts that it is used to bring in part of the referenced document.  However, both $Include and $IncludedAnnotations is documented as optional.  Either $Reference can bring in all of a referenced document, or at least one of $Include or $IncludedAnnotations must be specified.

      Note that edmx.xsd does not resolve this issue since it also allows the possibility that neither Include or IncludeAnnotations is specified.  

      A trivial issue is that both 'include' and 'included' are used to refer to the same functionality in this section.

        Attachments

          Activity

            People

            • Assignee:
              george.ericson George Ericson
              Reporter:
              george.ericson George Ericson
            • Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: