Skip to content

Commit b52c203

Browse files
committed
Ref #25533: add optional name to valuesets
1 parent cf1a780 commit b52c203

3 files changed

Lines changed: 22 additions & 2 deletions

File tree

src/main/java/eu/openanalytics/containerproxy/model/runtime/ProvidedParameters.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,13 @@ public class ProvidedParameters {
3131

3232
private final Map<String, String> backendValues;
3333
private final String stringRepresentation;
34+
private final String valueSetName;
3435

3536
@JsonCreator
36-
public ProvidedParameters(Map<String, String> backendValues, String stringRepresentation) {
37+
public ProvidedParameters(Map<String, String> backendValues, String stringRepresentation, String valueSetName) {
3738
this.backendValues = backendValues;
3839
this.stringRepresentation = stringRepresentation;
40+
this.valueSetName = valueSetName;
3941
}
4042

4143
public int size() {
@@ -50,4 +52,8 @@ public String getValue(String parameterId) {
5052
public String toString() {
5153
return stringRepresentation;
5254
}
55+
56+
public String getValueSetName() {
57+
return valueSetName;
58+
}
5359
}

src/main/java/eu/openanalytics/containerproxy/model/spec/Parameters.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,8 @@ public static class ValueSet {
5555
private AccessControl accessControl;
5656
private Map<String, List<String>> values;
5757

58+
private String name = null;
59+
5860
public void setValues(Map<String, List<String>> values) {
5961
this.values = values;
6062
}
@@ -78,5 +80,13 @@ public List<String> getParameterValues(String parameterId) {
7880
public Set<String> getParameterIds() {
7981
return values.keySet();
8082
}
83+
84+
public String getName() {
85+
return name;
86+
}
87+
88+
public void setName(String name) {
89+
this.name = name;
90+
}
8191
}
8292
}

src/main/java/eu/openanalytics/containerproxy/service/ParametersService.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,11 @@ private Optional<ProvidedParameters> convertParametersIfAllowed(List<ParameterDe
206206
}
207207
// providedParameters contains an allowed value for every parameter
208208
// return the backend values (instead of the names provided by the user)
209-
return Optional.of(new ProvidedParameters(backendValues, getStringRepresentation(parameters, providedParameters)));
209+
return Optional.of(new ProvidedParameters(
210+
backendValues,
211+
getStringRepresentation(parameters, providedParameters),
212+
valueSet.getName()
213+
));
210214
}
211215

212216
public String getStringRepresentation(List<ParameterDefinition> parameters, Map<String, String> providedParameters) {

0 commit comments

Comments
 (0)