Skip to content

NH-3499 - Allow custom QueryModelVisitorBase to be provided through the session factory #216

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

Merged
merged 1 commit into from
Feb 22, 2015

Conversation

pvginkel
Copy link
Contributor

Implements a new parameter on the session factory to provide a IQueryModelRewriterFactory which is used in QueryModelVisitor to allow the query model to be rewritten.

@OnurGumus
Copy link
Contributor

Wonderful feature! I wonder when these merges will happen I am waiting for mine more than 2 months

@rjperes
Copy link
Member

rjperes commented Sep 6, 2014

No support for loquacious configuration? :-)

@hazzik hazzik added this to the 4.1.0 milestone Nov 18, 2014
@hazzik
Copy link
Member

hazzik commented Nov 18, 2014

@pvginkel can you please rebase this on top of master and add ability to configure this option through maping by code?

…he session factory

Implements a new parameter on the session factory to provide a IQueryModelRewriterFactory which is used in QueryModelVisitor to allow the query model to be rewritten.
@pvginkel
Copy link
Contributor Author

The code has been rebased onto the master and support for mapping by code has been implemented. It seems that the unit testing framework changed, so that has been updated to.

One thing though: I have no experience with mapping by code. Can you (somebody) check whether I did the right thing? It looks OK to me but I may have missed something.

@hazzik
Copy link
Member

hazzik commented Nov 20, 2014

Looks good. Thanks a lot @pvginkel!

@hazzik hazzik merged commit c6f0788 into nhibernate:master Feb 22, 2015
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.

4 participants