New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename `CsdlReader/Writer`, `EdmxReader/Writer`, etc. #632

Closed
Lingxi-Li opened this Issue Jul 11, 2016 · 0 comments

Comments

Projects
None yet
2 participants
@Lingxi-Li
Contributor

Lingxi-Li commented Jul 11, 2016

According to Sect. 2 CSDL Namespaces in the CSDL spec,

An XML document using these namespaces and having an edmx:Edmx root element will be called a CSDL document.

This implies that edmx is a mandatory part of a valid CSDL doc. Yet, the current CsdlReader is unable to read such a doc. It's only able to read the Schema part of it. The current EdmxReader, on the other hand, is able to read a complete CSDL doc. The task is therefore to rename

  • CsdlReader/Writer to SchemaReader/Writer;
  • EdmxReader/Writer to CsdlReader/Writer;
  • EdmxReaderSettings to CsdlReaderSettings;
  • EdmxTarget to CsdlTarget.

@Lingxi-Li Lingxi-Li self-assigned this Jul 11, 2016

@Lingxi-Li Lingxi-Li changed the title from Rename `CsdlReader` and `EdmxReader` to Rename `CsdlReader/Writer` and `EdmxReader/Writer` Jul 12, 2016

@Lingxi-Li Lingxi-Li changed the title from Rename `CsdlReader/Writer` and `EdmxReader/Writer` to Rename `CsdlReader/Writer`, `EdmxReader/Writer`, and `EdmxReaderSettings` Jul 12, 2016

@Lingxi-Li Lingxi-Li changed the title from Rename `CsdlReader/Writer`, `EdmxReader/Writer`, and `EdmxReaderSettings` to Rename `CsdlReader/Writer`, `EdmxReader/Writer`, etc. Jul 12, 2016

@Lingxi-Li Lingxi-Li closed this Jul 13, 2016

@LaylaLiu LaylaLiu removed the in-progress label Jul 13, 2016

MIchaelMainer added a commit to Microsoft/Vipr that referenced this issue Aug 17, 2018

Update the models used in the generator templates: OdcmEntitySet and …
…OdcmSingleton so that we can pass NavigationPropertyBindings to the generator templates.

Update the ODatalibs from 6.11.0 to latest (7.5.0). This enabled the NavigationPropertyBindings. This required minor refactoring.
http://odata.github.io/odata.net/#06-21-Multi-Binding
http://odata.github.io/odata.net/#ODL-7.0.0
OData/odata.net#632

MIchaelMainer added a commit to Microsoft/Vipr that referenced this issue Sep 5, 2018

Update the models used in the generator templates: OdcmEntitySet and …
…OdcmSingleton so that we can pass NavigationPropertyBindings to the generator templates.

Update the ODatalibs from 6.11.0 to latest (7.5.0). This enabled the NavigationPropertyBindings. This required minor refactoring.
http://odata.github.io/odata.net/#06-21-Multi-Binding
http://odata.github.io/odata.net/#ODL-7.0.0
OData/odata.net#632
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment