Skip to content

Commit fbde97f

Browse files
committed
Default order 0 for ServerResponseResultHandler
Issue: SPR-16420
1 parent b612f53 commit fbde97f

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

spring-webflux/src/main/java/org/springframework/web/reactive/config/WebFluxConfigurationSupport.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -418,14 +418,10 @@ public ViewResolutionResultHandler viewResolutionResultHandler() {
418418

419419
@Bean
420420
public ServerResponseResultHandler serverResponseResultHandler() {
421-
ViewResolverRegistry registry = getViewResolverRegistry();
422-
List<ViewResolver> resolvers = registry.getViewResolvers();
423-
421+
List<ViewResolver> resolvers = getViewResolverRegistry().getViewResolvers();
424422
ServerResponseResultHandler handler = new ServerResponseResultHandler();
425423
handler.setMessageWriters(serverCodecConfigurer().getWriters());
426424
handler.setViewResolvers(resolvers);
427-
handler.setOrder(registry.getOrder() + 1);
428-
429425
return handler;
430426
}
431427

spring-webflux/src/main/java/org/springframework/web/reactive/function/server/support/ServerResponseResultHandler.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2017 the original author or authors.
2+
* Copyright 2002-2018 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -45,7 +45,7 @@ public class ServerResponseResultHandler implements HandlerResultHandler, Initia
4545

4646
private List<ViewResolver> viewResolvers = Collections.emptyList();
4747

48-
private int order = LOWEST_PRECEDENCE;
48+
private int order = 0;
4949

5050

5151
/**
@@ -63,8 +63,8 @@ public void setViewResolvers(List<ViewResolver> viewResolvers) {
6363

6464
/**
6565
* Set the order for this result handler relative to others.
66-
* <p>By default set to {@link Ordered#LOWEST_PRECEDENCE}, however see
67-
* Javadoc of sub-classes which may change this default.
66+
* <p>By default set to 0. It is generally safe to place it early in the
67+
* order as it looks for a concrete return type.
6868
* @param order the order
6969
*/
7070
public void setOrder(int order) {

0 commit comments

Comments
 (0)