Polishing contribution

Closes spring-projectsgh-27203
lxbzmy · Mar 26, 2022 · 973833d · 973833d
1 parent 214a3f8
commit 973833d
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 14 deletions.
diff --git a/spring-web/src/main/java/org/springframework/web/method/HandlerMethod.java b/spring-web/src/main/java/org/springframework/web/method/HandlerMethod.java
@@ -105,12 +105,12 @@ public HandlerMethod(Object bean, Method method) {
 		this(bean, method, null);
 	}
 
-
 	/**
 	 * Variant of {@link #HandlerMethod(Object, Method)} that
-	 * also accepts a {@link MessageSource}.
+	 * also accepts a {@link MessageSource} for use from sub-classes.
+	 * @since 5.3.10
 	 */
-	public HandlerMethod(Object bean, Method method, @Nullable MessageSource messageSource) {
+	protected HandlerMethod(Object bean, Method method, @Nullable MessageSource messageSource) {
 		Assert.notNull(bean, "Bean is required");
 		Assert.notNull(method, "Method is required");
 		this.bean = bean;

diff --git a/spring-web/src/main/java/org/springframework/web/method/support/InvocableHandlerMethod.java b/spring-web/src/main/java/org/springframework/web/method/support/InvocableHandlerMethod.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2020 the original author or authors.
+ * Copyright 2002-2021 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -74,9 +74,10 @@ public InvocableHandlerMethod(Object bean, Method method) {
 
 	/**
 	 * Variant of {@link #InvocableHandlerMethod(Object, Method)} that
-	 * also accepts a {@link MessageSource}.
+	 * also accepts a {@link MessageSource}, for use in sub-classes.
+	 * @since 5.3.10
 	 */
-	public InvocableHandlerMethod(Object bean, Method method, @Nullable MessageSource messageSource) {
+	protected InvocableHandlerMethod(Object bean, Method method, @Nullable MessageSource messageSource) {
 		super(bean, method, messageSource);
 	}
 

diff --git a/.../org/springframework/web/servlet/mvc/method/annotation/ServletInvocableHandlerMethod.java b/.../org/springframework/web/servlet/mvc/method/annotation/ServletInvocableHandlerMethod.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2019 the original author or authors.
+ * Copyright 2002-2021 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -69,13 +69,6 @@ public class ServletInvocableHandlerMethod extends InvocableHandlerMethod {
 	@Nullable
 	private HandlerMethodReturnValueHandlerComposite returnValueHandlers;
 
-	/**
-	 * Variant of {@link #ServletInvocableHandlerMethod(Object, Method)} that
-	 * also accepts a {@link MessageSource}.
-	 */
-	public ServletInvocableHandlerMethod(Object handler, Method method, @Nullable MessageSource messageSource) {
-		super(handler, method, messageSource);
-	}
 
 	/**
 	 * Creates an instance from the given handler and method.
@@ -84,6 +77,16 @@ public ServletInvocableHandlerMethod(Object handler, Method method) {
 		super(handler, method);
 	}
 
+	/**
+	 * Variant of {@link #ServletInvocableHandlerMethod(Object, Method)} that
+	 * also accepts a {@link MessageSource}, e.g. to resolve
+	 * {@code @ResponseStatus} messages with.
+	 * @since 5.3.10
+	 */
+	public ServletInvocableHandlerMethod(Object handler, Method method, @Nullable MessageSource messageSource) {
+		super(handler, method, messageSource);
+	}
+
 	/**
 	 * Create an instance from a {@code HandlerMethod}.
 	 */