Skip to content

Commit 7268d15

Browse files
authored
Merge pull request #522 from IBM/issue-519
issue #519 - QueryParameterValue.toString should use query string format
2 parents 50a93fa + 2b958dc commit 7268d15

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

fhir-search/src/main/java/com/ibm/fhir/search/parameters/QueryParameterValue.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -145,11 +145,13 @@ public String toString() {
145145
outputBuilder(returnString, valueDate);
146146

147147
if (component != null && !component.isEmpty()) {
148-
returnString.append("composite[");
149-
// temporarily change the delimiter; NOT thread-safe
150148
String componentDelim = "";
151149
for (QueryParameter componentParam : component) {
152-
returnString.append(componentDelim).append(componentParam);
150+
List<QueryParameterValue> componentValues = componentParam.getValues();
151+
if (componentValues.size() != 1) {
152+
throw new IllegalStateException("Components of a composite search parameter may only have a single value");
153+
}
154+
returnString.append(componentDelim).append(componentValues.get(0));
153155
componentDelim = "$";
154156
}
155157
}

fhir-search/src/main/java/com/ibm/fhir/search/uri/UriBuilder.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -239,7 +239,7 @@ private String serializeSearchParmToQueryString(QueryParameter param) {
239239
return returnString.toString();
240240
}
241241

242-
/*
242+
/**
243243
* creates a normal parameter and string.
244244
*
245245
* @param param

0 commit comments

Comments
 (0)