-
Notifications
You must be signed in to change notification settings - Fork 157
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
Issue #1323 - Add support for ':of-type' modifier #2116
Conversation
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
Given this identifier:
Here's what the extracted parameters look like in the x_TOKEN_VALUES_V views.
|
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
fdfd561
to
2d3af5f
Compare
Tip: you can actually have this pull request target the branch you used for the other PR...then the diff will only show the delta. If the other PR gets merged, github will automatically change this target to |
Ah, ok. I didn't realize it worked that way. Sure, go ahead! |
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
fhir-search/src/main/java/com/ibm/fhir/search/parameters/QueryParameterValue.java
Outdated
Show resolved
Hide resolved
fhir-search/src/main/java/com/ibm/fhir/search/parameters/QueryParameterValue.java
Show resolved
Hide resolved
fhir-search/src/main/java/com/ibm/fhir/search/parameters/QueryParameterValue.java
Outdated
Show resolved
Hide resolved
fhir-search/src/main/java/com/ibm/fhir/search/parameters/QueryParameterValue.java
Outdated
Show resolved
Hide resolved
...ence-jdbc/src/main/java/com/ibm/fhir/persistence/jdbc/util/JDBCParameterBuildingVisitor.java
Show resolved
Hide resolved
fhir-search/src/main/java/com/ibm/fhir/search/util/SearchUtil.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
fhir-search/src/main/java/com/ibm/fhir/search/parameters/QueryParameterValue.java
Show resolved
Hide resolved
b9db511
to
62f48c7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
62f48c7
to
7dd99b6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (once the tests pass)
Issue #1323 - Add support for ':of-type' modifier
Signed-off-by: Troy Biesterfeld tbieste@us.ibm.com