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: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: V4.01_OS
    • Fix Version/s: V4.01_ERRATA01
    • Component/s: CSDL JSON, CSDL XML
    • Labels:
      None
    • 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.
    • Resolution:
      Hide

      https://github.com/oasis-tcs/odata-specs/pull/74

      Show
      https://github.com/oasis-tcs/odata-specs/pull/74

      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:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: