Skip to content

Support for abstract complex types with no properties - #1731

Merged
KanishManuja-MS merged 1 commit into
OData:masterfrom
ElizabethOkerio:fix/1581-support-for-abstract-complex-types-with-no-properties
May 19, 2020
Merged

KanishManuja-MS merged 1 commit into
OData:masterfrom
ElizabethOkerio:fix/1581-support-for-abstract-complex-types-with-no-properties

Conversation

@ElizabethOkerio

@ElizabethOkerio ElizabethOkerio commented Apr 3, 2020 •

Copy link
Copy Markdown
Contributor

Issues

This pull request fixes issue #1581.

Description

Since Web API OData V5.5-beta, it is allowed to:

  1. define abstract type (entity & complex) without any properties.

But in this case, when you define an abstract complex type without any properties an exception is thrown.

Checklist (Uncheck if it is not completed)

  • Test cases added
  • Build and test with one-click build and test script passed

Additional work necessary

If documentation update is needed, please add "Docs Needed" label to the issue and provide details about the required document change in the issue.

@ElizabethOkerio
ElizabethOkerio force-pushed the fix/1581-support-for-abstract-complex-types-with-no-properties branch from ee08abc to 6ccdf35 Compare April 8, 2020 06:17
@ElizabethOkerio
ElizabethOkerio requested a review from xuzhg April 9, 2020 06:12
@ElizabethOkerio ElizabethOkerio added the Ready for review Use this label if a pull request is ready to be reviewed label Apr 9, 2020
@ElizabethOkerio
ElizabethOkerio force-pushed the fix/1581-support-for-abstract-complex-types-with-no-properties branch from 6ccdf35 to edd4294 Compare April 9, 2020 07:34
@xuzhg

xuzhg commented Apr 9, 2020

Copy link
Copy Markdown
Contributor

@ElizabethOkerio It's better to have a test case to cover the changes.

@odero odero changed the title Support for abstract complex types with no properies Support for abstract complex types with no properties Apr 15, 2020
Comment thread src/Microsoft.OData.Client/ClientEdmModel.cs Outdated
@ElizabethOkerio
ElizabethOkerio force-pushed the fix/1581-support-for-abstract-complex-types-with-no-properties branch from edd4294 to e072628 Compare April 15, 2020 07:05
@ElizabethOkerio
ElizabethOkerio requested a review from odero April 15, 2020 09:23
@ElizabethOkerio
ElizabethOkerio force-pushed the fix/1581-support-for-abstract-complex-types-with-no-properties branch from ba8b2e8 to 308980b Compare April 20, 2020 06:53
@ElizabethOkerio
ElizabethOkerio force-pushed the fix/1581-support-for-abstract-complex-types-with-no-properties branch 2 times, most recently from 8836090 to ac7a5ae Compare May 6, 2020 06:50
@ElizabethOkerio
ElizabethOkerio force-pushed the fix/1581-support-for-abstract-complex-types-with-no-properties branch from 98e3296 to 89b7412 Compare May 13, 2020 09:26
@KanishManuja-MS KanishManuja-MS added this to the 7.7.0 milestone May 18, 2020
@KanishManuja-MS
KanishManuja-MS self-requested a review May 18, 2020 17:29
@KanishManuja-MS
KanishManuja-MS merged commit 98c0e97 into OData:master May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready for review Use this label if a pull request is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to create a new entity with a nested complex type collection

5 participants