SpringdocRouteBuilder.onError is overriding last route defined. Fixes #2399

Author: bnasslahsen

springdoc-openapi-starter-webflux-api/src/main/java/org/springdoc/webflux/core/fn/SpringdocRouteBuilder.java

@@ -651,12 +651,10 @@ public SpringdocRouteBuilder after(BiFunction<ServerRequest, ServerResponse, Ser
 	 *
 	 * @param predicate the predicate
 	 * @param responseProvider the response provider
-	 * @param operationsConsumer the operations consumer
 	 * @return the springdoc route builder
 	 */
-	public SpringdocRouteBuilder onError(Predicate<? super Throwable> predicate, BiFunction<? super Throwable, ServerRequest, Mono<ServerResponse>> responseProvider, Consumer<Builder> operationsConsumer) {
-		Builder builder = getOperationBuilder(operationsConsumer);
-		this.delegate.onError(predicate, responseProvider).withAttribute(OPERATION_ATTRIBUTE, builder);
+	public SpringdocRouteBuilder onError(Predicate<? super Throwable> predicate, BiFunction<? super Throwable, ServerRequest, Mono<ServerResponse>> responseProvider) {
+		this.delegate.onError(predicate, responseProvider);
 		return this;
 	}
 
@@ -667,12 +665,10 @@ public SpringdocRouteBuilder onError(Predicate<? super Throwable> predicate, BiF
 	 * @param <T>    the type parameter
 	 * @param exceptionType the exception type
 	 * @param responseProvider the response provider
-	 * @param operationsConsumer the operations consumer
 	 * @return the springdoc route builder
 	 */
-	public <T extends Throwable> SpringdocRouteBuilder onError(Class<T> exceptionType, BiFunction<? super T, ServerRequest, Mono<ServerResponse>> responseProvider, Consumer<Builder> operationsConsumer) {
-		Builder builder = getOperationBuilder(operationsConsumer);
-		this.delegate.onError(exceptionType, responseProvider).withAttribute(OPERATION_ATTRIBUTE, builder);
+	public <T extends Throwable> SpringdocRouteBuilder onError(Class<T> exceptionType, BiFunction<? super T, ServerRequest, Mono<ServerResponse>> responseProvider) {
+		this.delegate.onError(exceptionType, responseProvider);
 		return this;
 	}
 

springdoc-openapi-starter-webmvc-api/src/main/java/org/springdoc/webmvc/core/fn/SpringdocRouteBuilder.java

@@ -651,12 +651,10 @@ public SpringdocRouteBuilder after(BiFunction<ServerRequest, ServerResponse, Ser
 	 *
 	 * @param predicate the predicate
 	 * @param responseProvider the response provider
-	 * @param operationsConsumer the operations consumer
 	 * @return the springdoc route builder
 	 */
-	public SpringdocRouteBuilder onError(Predicate<Throwable> predicate, BiFunction<Throwable, ServerRequest, ServerResponse> responseProvider, Consumer<Builder> operationsConsumer) {
-		Builder builder = getOperationBuilder(operationsConsumer);
-		this.delegate.onError(predicate, responseProvider).withAttribute(OPERATION_ATTRIBUTE, builder);
+	public SpringdocRouteBuilder onError(Predicate<Throwable> predicate, BiFunction<Throwable, ServerRequest, ServerResponse> responseProvider) {
+		this.delegate.onError(predicate, responseProvider);
 		return this;
 	}
 
@@ -667,12 +665,10 @@ public SpringdocRouteBuilder onError(Predicate<Throwable> predicate, BiFunction<
 	 * @param <T>    the type parameter
 	 * @param exceptionType the exception type
 	 * @param responseProvider the response provider
-	 * @param operationsConsumer the operations consumer
 	 * @return the springdoc route builder
 	 */
-	public <T extends Throwable> SpringdocRouteBuilder onError(Class<T> exceptionType, BiFunction<Throwable, ServerRequest, ServerResponse> responseProvider, Consumer<Builder> operationsConsumer) {
-		Builder builder = getOperationBuilder(operationsConsumer);
-		this.delegate.onError(exceptionType, responseProvider).withAttribute(OPERATION_ATTRIBUTE, builder);
+	public <T extends Throwable> SpringdocRouteBuilder onError(Class<T> exceptionType, BiFunction<Throwable, ServerRequest, ServerResponse> responseProvider) {
+		this.delegate.onError(exceptionType, responseProvider);
 		return this;
 	}