GroupedOpenApi.builder addOpenApiCustomiser execution order is reversed. Fixes #1633

Author: bnasslahsen

springdoc-openapi-common/src/main/java/org/springdoc/core/GroupedOpenApi.java

@@ -25,7 +25,6 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
 
@@ -52,12 +51,12 @@ public class GroupedOpenApi {
 	/**
 	 * The Open api customisers.
 	 */
-	private final List<OpenApiCustomiser> openApiCustomisers;
+	private List<OpenApiCustomiser> openApiCustomisers;
 
 	/**
 	 * The Operation customizers.
 	 */
-	private final List<OperationCustomizer> operationCustomizers;
+	private List<OperationCustomizer> operationCustomizers;
 
 	/**
 	 * The Paths to match.
@@ -97,7 +96,7 @@ public class GroupedOpenApi {
 	/**
 	 * The method filters to use.
 	 */
-	private final List<OpenApiMethodFilter> openApiMethodFilters;
+	private List<OpenApiMethodFilter> openApiMethodFilters;
 
 	/**
 	 * The Display name.
@@ -473,8 +472,10 @@ public GroupedOpenApi build() {
 	 * @return the grouped open api
 	 */
 	public GroupedOpenApi addAllOpenApiCustomizer(Collection<? extends OpenApiCustomiser> openApiCustomizerCollection) {
-		this.openApiCustomisers.addAll(openApiCustomizerCollection);
-		Collections.reverse(openApiCustomisers);
+		List<OpenApiCustomiser> result = new ArrayList<>();
+		result.addAll(openApiCustomizerCollection);
+		result.addAll(openApiCustomisers);
+		openApiCustomisers = result;
 		return this;
 	}
 
@@ -485,8 +486,10 @@ public GroupedOpenApi addAllOpenApiCustomizer(Collection<? extends OpenApiCustom
 	 * @return the grouped open api
 	 */
 	public GroupedOpenApi addAllOperationCustomizer(Collection<? extends OperationCustomizer> operationCustomizerCollection) {
-		this.operationCustomizers.addAll(operationCustomizerCollection);
-		Collections.reverse(operationCustomizers);
+		List<OperationCustomizer> result = new ArrayList<>();
+		result.addAll(operationCustomizerCollection);
+		result.addAll(operationCustomizers);
+		operationCustomizers = result;
 		return this;
 	}
 
@@ -497,8 +500,10 @@ public GroupedOpenApi addAllOperationCustomizer(Collection<? extends OperationCu
 	 * @return the grouped open api
 	 */
 	public GroupedOpenApi addAllOpenApiMethodFilter(Collection<? extends OpenApiMethodFilter> openApiMethodFilterCollection) {
-		this.openApiMethodFilters.addAll(openApiMethodFilterCollection);
-		Collections.reverse(openApiMethodFilters);
+		List<OpenApiMethodFilter> result = new ArrayList<>();
+		result.addAll(openApiMethodFilterCollection);
+		result.addAll(openApiMethodFilters);
+		openApiMethodFilters = result;
 		return this;
 	}
 }