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

Feature/custom named parameters #400

Open
wants to merge 5 commits into
base: master
from

Conversation

@bls220
Copy link

bls220 commented Jul 14, 2020

Enabled custom parameter names to be used instead of positional names.

These changes are useful when the user wants to do their own parameter binding/execution.

bls220 added 5 commits Jul 11, 2020
Cleaned unused references.
Updated Logger method.
… parameters.

Added flag to compilers to toggle custom named parameters.
Useful for when the user only wants to generate a SQL string.

namespace SqlKata
{
public interface IWithQuery<Q> where Q : BaseQuery<Q>

This comment has been minimized.

@bls220

bls220 Jul 14, 2020 Author

This is not the best name. Any suggestions welcome!

{
value = ctx.Query.FindVariable(variable.Name);
}
catch (Exception) when (AllowUndefinedNamedVariables)

This comment has been minimized.

@bls220

bls220 Jul 14, 2020 Author

I don't particularly like catching general exceptions, but I thought this might be useful if someone is just trying to generate a statement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.