-
Notifications
You must be signed in to change notification settings - Fork 34
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
Allow use of RegExp in FieldIn Common CriteriaVisitor #106
base: 1.5
Are you sure you want to change the base?
Conversation
Above feedback isn't relevant within the scope of this PR. |
no worries, just a cs-fixer update. I'll let you know when I fix it :) |
I'm not sure that that's entirely valid: Currently, it's possible to do the exact opposite with the SQL engine, i.e. insert SQL wildcards. |
Ok, we (@alongosz, ..) will try to investigate that with integration tests on API to better define this, thanks for finding this. |
@alongosz We'll need to check from API perspective what we want to do with this one, for instance if we want to limit use to % to LIKE across engines or otherwise. Related, got report from @danielmuehlbacher:
In FullTextCriterion |
exciting times!! |
This is to match the behaviour of the equivalent CustomField visitor.
Use-case: We would like to search for a substring within an ezstring, in Solr. Wildcards are always escaped using this criterion.