Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added yanking for null propagation in MethodCall expressions (ToLower/ToUpper) #1

Merged
merged 2 commits into from Apr 18, 2012

Conversation

Projects
None yet
2 participants
Contributor

ray2k commented Apr 10, 2012

The existing NullPropagationYanker only handles boolean conditional expressions like Contains/StartsWith/EndsWith. Using a the reflection provider for WCF data services over top a linq provider that does not support sql's equivalent CASE semantics (for me, mongodb) will still have queries fail unless these are removed.

Added separate extenstion method to remove methodcall null propagation. Did NOT edit current behavior of .WithoutNullPropagation() extension method, not sure what you want to do there.

Also added xunit test project. Because of heavy internal-visibility usage, this required public ctor.

Owner

davidfowl commented Apr 10, 2012

Awesome!

Didn't think there'd be much use for this project. Glad you found another case.

davidfowl added a commit that referenced this pull request Apr 18, 2012

Merge pull request #1 from ray2k/master
Added yanking for  null propagation in MethodCall expressions (ToLower/ToUpper)

@davidfowl davidfowl merged commit 33541a6 into davidfowl:master Apr 18, 2012

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