diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/AbstractJsonpResponseBodyAdvice.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/AbstractJsonpResponseBodyAdvice.java index c595cf937c..765d93d05a 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/AbstractJsonpResponseBodyAdvice.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/AbstractJsonpResponseBodyAdvice.java @@ -23,10 +23,9 @@ import org.springframework.http.server.ServerHttpRequest; import org.springframework.http.server.ServerHttpResponse; import org.springframework.http.server.ServletServerHttpRequest; import org.springframework.util.Assert; -import org.springframework.util.CollectionUtils; +import org.springframework.util.ObjectUtils; import javax.servlet.http.HttpServletRequest; -import java.util.Collection; /** * A convenient base class for a {@code ResponseBodyAdvice} to instruct the @@ -48,9 +47,9 @@ public abstract class AbstractJsonpResponseBodyAdvice extends AbstractMappingJac private final String[] jsonpQueryParamNames; - protected AbstractJsonpResponseBodyAdvice(Collection queryParamNames) { - Assert.isTrue(!CollectionUtils.isEmpty(queryParamNames), "At least one query param name is required"); - this.jsonpQueryParamNames = queryParamNames.toArray(new String[queryParamNames.size()]); + protected AbstractJsonpResponseBodyAdvice(String... queryParamNames) { + Assert.isTrue(!ObjectUtils.isEmpty(queryParamNames), "At least one query param name is required"); + this.jsonpQueryParamNames = queryParamNames; } diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapterTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapterTests.java index 97cd2a59a0..dde3afccae 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapterTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapterTests.java @@ -355,12 +355,12 @@ public class RequestMappingHandlerAdapterTests { } } - @ControllerAdvice - private static class JsonpAdvice extends AbstractJsonpResponseBodyAdvice { +@ControllerAdvice +private static class JsonpAdvice extends AbstractJsonpResponseBodyAdvice { - public JsonpAdvice() { - super(Arrays.asList("c")); - } + public JsonpAdvice() { + super("c"); } +} }