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 navigation property used in multi bindings with different path #629

Open
Zoe-ms opened this Issue Jul 7, 2016 · 0 comments

Comments

Projects
None yet
2 participants
@Zoe-ms
Contributor

Zoe-ms commented Jul 7, 2016

Goal: In order to support binding nav property under complex and containment to different path, for example, Cityunder complex type Address, and an entity type can have different property HomeAddressand WorkAddressbased on Address type, then HomeAddress/City and WorkAddress/City can have different bindings.

Done:

  • Add overloaded AddNavigationTarget()/FindNavigationTarget() to accept binding path as parameter
  • Remove model validation about unique binding of one navigation property
  • Change FindNavigationTarget() in Reader, Writer and URL parser to resolve navigation target with binding path

TODO:

  • Change FindNavigationTarget() in delta reader to resolve navigation target with binding path
  • Add binding path validation to model validation rules

@Zoe-ms Zoe-ms added the breaking label Jul 7, 2016

@Zoe-ms Zoe-ms added this to the 7.0 milestone Jul 7, 2016

@Zoe-ms Zoe-ms added the ready label Jul 7, 2016

@Zoe-ms Zoe-ms changed the title from Support navigation property used in 2 bindings with different path to Support navigation property used in multi bindings with different path Jul 8, 2016

@Zoe-ms Zoe-ms added in-progress and removed ready labels Jul 18, 2016

@Zoe-ms Zoe-ms self-assigned this Jul 18, 2016

@Zoe-ms Zoe-ms added docs and removed in-progress labels Aug 19, 2016

@robward-ms robward-ms added the P3 label Jun 12, 2017

@robward-ms robward-ms removed the breaking label Sep 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment