Skip to content

Commit

Permalink
Include variables Query Parameter Names when listing template paramet…
Browse files Browse the repository at this point in the history
…ers (#1144)

Fixes #1089

Query Template names, which also happen to be templates, were not being
considered when listing out all of the variable names in a Query Template.
  • Loading branch information
kdavisk6 committed Dec 27, 2019
1 parent a7b7c01 commit f73fe70
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/src/main/java/feign/template/QueryTemplate.java
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ public List<String> getValues() {
}

public List<String> getVariables() {
List<String> variables = new ArrayList<>();
List<String> variables = new ArrayList<>(this.name.getVariables());
for (Template template : this.values) {
variables.addAll(template.getVariables());
}
Expand Down

0 comments on commit f73fe70

Please sign in to comment.