Skip to content
This repository was archived by the owner on Jun 30, 2023. It is now read-only.

Commit 8794cce

Browse files
committed
move parameterOrder order declaration closer to use
1 parent b09cd32 commit 8794cce

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

endpoints-framework/src/main/java/com/google/api/server/spi/discovery/DiscoveryGenerator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,14 +222,14 @@ private void writeApiMethod(ApiConfig config, String servicePath, RestDescriptio
222222
private List<String> computeParameterOrder(ApiMethodConfig methodConfig) {
223223
ImmutableSortedSet.Builder<String> queryParamBuilder = ImmutableSortedSet.naturalOrder();
224224
Collection<String> pathParameters = methodConfig.getPathParameters();
225-
List<String> order = new ArrayList<>(pathParameters);
226225
for (ApiParameterConfig parameterConfig : methodConfig.getParameterConfigs()) {
227226
if (parameterConfig.getClassification() == Classification.API_PARAMETER
228227
&& !pathParameters.contains(parameterConfig.getName())
229228
&& !parameterConfig.getNullable()) {
230229
queryParamBuilder.add(parameterConfig.getName());
231230
}
232231
}
232+
List<String> order = new ArrayList<>(pathParameters);
233233
order.addAll(queryParamBuilder.build());
234234
return order;
235235
}

0 commit comments

Comments
 (0)