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

ANSI_NULLS ON and reflection fixes #481

Merged
merged 2 commits into from Dec 10, 2019
Merged

Conversation

@jeremeguenther
Copy link
Collaborator

jeremeguenther commented Dec 7, 2019

Fixes to bring the sprocs in line with the standards so they work correctly with the ANSI_NULLS toggle.

Also some minor adjustments to the NetTiersDataException object for easier use with reflection and serialization.

There were a couple of sprocs that did not conform to the standard the other sprocs were built with in reguards to ANSI_NULLS.
This patch brings them in line with the standard.
The NetTiersDataException had a mix of exposed Fields and Properties.
This patch converts them all into Properties for easier use with reflection.
It also adds a default empty constructor.
Copy link
Contributor

niemyjski left a comment

LGTM

@jeremeguenther jeremeguenther merged commit 76df447 into netTiers:master Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.