OData / WebApi

OData Web API: A server library built upon ODataLib and WebApi
https://docs.microsoft.com/odata
Other
855 stars 473 forks source link

$orderby with duplicate property in odata v4 failed. #630

Open janvanderhaegen opened 8 years ago

janvanderhaegen commented 8 years ago

This issue was supposedly fixed in #376 , but the fix does not cover all use cases. For example Customer > City(string) Customer > Address(Address) > City(string) Customer > WorkAddress(Address) > City(string)

Thanks to the fix in #376, one can order by Customer/City, Customer/Address/City without exception (they are different IEdmProperties)

However, one cannot order by Customer/Address/City, Customer/WorkAddress/City (they are the 'same' IEdmProperty).

VikingsFan commented 8 years ago

@janvanderhaegen Thanks! Would you like to submit a PR for this issue :) maybe we can use Path instead of IEdmProperty.

mirsking commented 8 years ago

This issue is already fixed by PR #799 .

Malyngo commented 7 years ago

Is this fix in version 6.0.0? I just updated, but still see that issue.

thelongshot commented 7 years ago

As Malyngo says above, this bug still exists in 6.0.0. Please reopen.

mikepizzo commented 7 years ago

More info in Issue #907.