Skip to content
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

For comparing unordered collections - respect settings for casesensit… #104

Conversation

steinho
Copy link
Contributor

@steinho steinho commented Jun 12, 2018

…ivity and treat null and string.empty as same

Hi - had some trouble with Compare.Net when comparing arrays of objects where some properties on the objects was string.empty on one end and null on the other.... Found similar problems with casesensitivity

Basically config.IgnoreCollectionOrder = true did not work well with config.TreatStringEmptyAndNullTheSame = true or config.CaseSensitive = false.

Se included tests - that will fail without my fix...

Would be nice if this one could be accepted and a new nuget package published- as it would fix mentioned problems in another project im working on..

Please review my changes.

…ivity and treat null and string.empty as same
@GregFinzer GregFinzer merged commit f21a2a3 into GregFinzer:master Jun 13, 2018
@GregFinzer
Copy link
Owner

It will take me several days to do a build. I am just swamped with managing three projects at my day job and side business.

@steinho
Copy link
Contributor Author

steinho commented Jun 13, 2018

Thanks!

@steinho steinho deleted the feature/FixCaseAndNullInCollectionCompare branch June 13, 2018 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants