IharYakimush / comminity-data-odata-linq

Use OData filter text query in linq expresson for any IQuerable without ASP.NET dependency. Support netstandard2.0

Other

42 stars 16 forks source link

An attempt at making Recursive Loop Of Complex Types an opt-in feature #44

Closed FortuneN closed 2 years ago

FortuneN commented 2 years ago

This would allow the brave/stupid like me to opt into recursions and be prepared to deal with the fallout

33

I tried this on a project i'm currently working on and so far no problems.

I'm prepared to adjust this PR as necessary to meet your standards.

IharYakimush commented 2 years ago

Hi @FortuneN please add unit tests and resolve my comment related to hash calculation

FortuneN commented 2 years ago

I changed the manner in which i access the ODataSettings in StructuralTypeConfiguration. It seems a bit better than just transferring one property of the settings. This way, all other settings values are usable (if need be) at that level.

IharYakimush commented 2 years ago

Hi @FortuneN, this change available in Community.OData.Linq.2.1.24-rc