Skip to content
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

Support media entities and stream properties #149

Closed
LianwMS opened this issue Jan 12, 2015 · 2 comments
Closed

Support media entities and stream properties #149

LianwMS opened this issue Jan 12, 2015 · 2 comments
Assignees
Labels
Milestone

Comments

@LianwMS
Copy link
Contributor

LianwMS commented Jan 12, 2015

OData v4 spec has [ stream properties ](http://docs.oasis-open.org/odata
/odata-json-format/v4.0/csprd01/odata-json-
format-v4.0-csprd01.html#_Stream_Property) (aka named resource stream) and [
media entities ](http://docs.oasis-open.org/odata/odata-json-
format/v4.0/csprd01/odata-json-format-v4.0-csprd01.html#_Toc355172943) (i.e.,
media link entries.) Both are without built-in support.

Especially, for media entities, should consider changing model builder’s code
to call:

public EdmEntityType(string namespaceName, string name, IEdmEntityType baseType, bool isAbstract, bool isOpen, bool hasStream)

instead of:

new EdmEntityType(config.Namespace, config.Name, baseType,
                        entity.IsAbstract ?? false, entity.IsOpen)

and expose a property e.g., HasStream to EntityTypeConfiguration etc.

Existing code:

    internal class EdmTypeBuilder
    {
        private void CreateEdmTypeHeader(IEdmTypeConfiguration config)
        {
...
                    EdmEntityType entityType = new EdmEntityType(config.Namespace, config.Name, baseType,
                        entity.IsAbstract ?? false, entity.IsOpen);
...

Work Item Details

Original CodePlex Issue: Issue 2118
Status: Proposed
Reason Closed: Unassigned
Assigned to: Unassigned
Reported on: Aug 26, 2014 at 10:15 AM
Reported by: cysu
Updated on: Dec 12, 2014 at 8:28 PM
Updated by: francoismackandal

@LianwMS LianwMS added this to the vNext milestone Jan 12, 2015
@vojtechvit
Copy link

+1

@xuzhg
Copy link
Member

xuzhg commented Jun 10, 2015

Merged 1e57d2e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants