Polishing

Author: jhoeller

spring-test/src/main/java/org/springframework/test/util/XpathExpectationsHelper.java

@@ -16,13 +16,9 @@
 
 package org.springframework.test.util;
 
-import static org.hamcrest.MatcherAssert.*;
-import static org.springframework.test.util.AssertionErrors.*;
-
 import java.io.ByteArrayInputStream;
 import java.util.Collections;
 import java.util.Map;
-
 import javax.xml.namespace.QName;
 import javax.xml.parsers.DocumentBuilder;
 import javax.xml.parsers.DocumentBuilderFactory;
@@ -42,6 +38,9 @@
 import org.springframework.util.StringUtils;
 import org.springframework.util.xml.SimpleNamespaceContext;
 
+import static org.hamcrest.MatcherAssert.*;
+import static org.springframework.test.util.AssertionErrors.*;
+
 /**
  * A helper class for applying assertions via XPath expressions.
  *
@@ -113,7 +112,7 @@ protected Document parseXmlByteArray(byte[] xml, String encoding) throws Excepti
 		factory.setNamespaceAware(this.hasNamespaces);
 		DocumentBuilder documentBuilder = factory.newDocumentBuilder();
 		InputSource inputSource = new InputSource(new ByteArrayInputStream(xml));
-		if(StringUtils.hasText(encoding)) {
+		if (StringUtils.hasText(encoding)) {
 			inputSource.setEncoding(encoding);
 		}
 		return documentBuilder.parse(inputSource);

spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java

@@ -30,12 +30,12 @@
 import org.springframework.util.Assert;
 
 /**
- * Response extractor that uses the given {@linkplain HttpMessageConverter entity
- * converters} to convert the response into a type {@code T}.
+ * Response extractor that uses the given {@linkplain HttpMessageConverter entity converters}
+ * to convert the response into a type {@code T}.
  *
  * @author Arjen Poutsma
- * @see RestTemplate
  * @since 3.0
+ * @see RestTemplate
  */
 public class HttpMessageConverterExtractor<T> implements ResponseExtractor<T> {
 
@@ -47,19 +47,18 @@ public class HttpMessageConverterExtractor<T> implements ResponseExtractor<T> {
 
 	private final Log logger;
 
+
 	/**
-	 * Creates a new instance of the {@code HttpMessageConverterExtractor} with the given
-	 * response type and message converters. The given converters must support the response
-	 * type.
+	 * Create a new instance of the {@code HttpMessageConverterExtractor} with the given response
+	 * type and message converters. The given converters must support the response type.
 	 */
 	public HttpMessageConverterExtractor(Class<T> responseType, List<HttpMessageConverter<?>> messageConverters) {
 		this((Type) responseType, messageConverters);
 	}
 
 	/**
-	 * Creates a new instance of the {@code HttpMessageConverterExtractor} with the given
-	 * response type and message converters. The given converters must support the response
-	 * type.
+	 * Creates a new instance of the {@code HttpMessageConverterExtractor} with the given response
+	 * type and message converters. The given converters must support the response type.
 	 */
 	public HttpMessageConverterExtractor(Type responseType, List<HttpMessageConverter<?>> messageConverters) {
 		this(responseType, messageConverters, LogFactory.getLog(HttpMessageConverterExtractor.class));
@@ -75,10 +74,10 @@ public HttpMessageConverterExtractor(Type responseType, List<HttpMessageConverte
 		this.logger = logger;
 	}
 
+
 	@Override
-	@SuppressWarnings({ "unchecked", "rawtypes" })
+	@SuppressWarnings({"unchecked", "rawtypes"})
 	public T extractData(ClientHttpResponse response) throws IOException {
-
 		MessageBodyClientHttpResponseWrapper responseWrapper = new MessageBodyClientHttpResponseWrapper(response);
 		if (!responseWrapper.hasMessageBody() || responseWrapper.hasEmptyMessageBody()) {
 			return null;
@@ -106,9 +105,9 @@ public T extractData(ClientHttpResponse response) throws IOException {
 				}
 			}
 		}
-		throw new RestClientException(
-				"Could not extract response: no suitable HttpMessageConverter found for response type [" +
-						this.responseType + "] and content type [" + contentType + "]");
+
+		throw new RestClientException("Could not extract response: no suitable HttpMessageConverter found " +
+				"for response type [" + this.responseType + "] and content type [" + contentType + "]");
 	}
 
 	private MediaType getContentType(ClientHttpResponse response) {

spring-webmvc/src/main/java/org/springframework/web/servlet/config/ResourcesBeanDefinitionParser.java

@@ -310,7 +310,7 @@ private void parseResourceResolversTransformers(boolean isAutoRegistration,
 		}
 
 		if (isAutoRegistration) {
-			if(isWebJarsAssetLocatorPresent) {
+			if (isWebJarsAssetLocatorPresent) {
 				RootBeanDefinition webJarsResolverDef = new RootBeanDefinition(WebJarsResourceResolver.class);
 				webJarsResolverDef.setSource(source);
 				webJarsResolverDef.setRole(BeanDefinition.ROLE_INFRASTRUCTURE);

spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/ResourceChainRegistration.java

@@ -45,6 +45,7 @@ public class ResourceChainRegistration {
 	private static final boolean isWebJarsAssetLocatorPresent = ClassUtils.isPresent(
 			"org.webjars.WebJarAssetLocator", ResourceChainRegistration.class.getClassLoader());
 
+
 	private final List<ResourceResolver> resolvers = new ArrayList<ResourceResolver>(4);
 
 	private final List<ResourceTransformer> transformers = new ArrayList<ResourceTransformer>(4);
@@ -103,7 +104,7 @@ public ResourceChainRegistration addTransformer(ResourceTransformer transformer)
 	protected List<ResourceResolver> getResourceResolvers() {
 		if (!this.hasPathResolver) {
 			List<ResourceResolver> result = new ArrayList<ResourceResolver>(this.resolvers);
-			if(isWebJarsAssetLocatorPresent) {
+			if (isWebJarsAssetLocatorPresent) {
 				result.add(new WebJarsResourceResolver());
 			}
 			result.add(new PathResourceResolver());

spring-webmvc/src/main/java/org/springframework/web/servlet/resource/CachingResourceResolver.java

@@ -86,9 +86,9 @@ protected Resource resolveResourceInternal(HttpServletRequest request, String re
 	protected String computeKey(HttpServletRequest request, String requestPath) {
 		StringBuilder key = new StringBuilder(RESOLVED_RESOURCE_CACHE_KEY_PREFIX);
 		key.append(requestPath);
-		if(request != null) {
+		if (request != null) {
 			String encoding = request.getHeader("Accept-Encoding");
-			if(encoding != null && encoding.contains("gzip")) {
+			if (encoding != null && encoding.contains("gzip")) {
 				key.append("+encoding=gzip");
 			}
 		}

spring-webmvc/src/main/java/org/springframework/web/servlet/resource/PathResourceResolver.java

@@ -161,8 +161,10 @@ private boolean isResourceUnderLocation(Resource resource, Resource location) th
 		if (!resource.getClass().equals(location.getClass())) {
 			return false;
 		}
+
 		String resourcePath;
 		String locationPath;
+
 		if (resource instanceof UrlResource) {
 			resourcePath = resource.getURL().toExternalForm();
 			locationPath = StringUtils.cleanPath(location.getURL().toString());
@@ -179,13 +181,15 @@ else if (resource instanceof ServletContextResource) {
 			resourcePath = resource.getURL().getPath();
 			locationPath = StringUtils.cleanPath(location.getURL().getPath());
 		}
-		if(locationPath.equals(resourcePath)) {
+
+		if (locationPath.equals(resourcePath)) {
 			return true;
 		}
 		locationPath = (locationPath.endsWith("/") || locationPath.isEmpty() ? locationPath : locationPath + "/");
 		if (!resourcePath.startsWith(locationPath)) {
 			return false;
 		}
+
 		if (resourcePath.contains("%")) {
 			// Use URLDecoder (vs UriUtils) to preserve potentially decoded UTF-8 chars...
 			if (URLDecoder.decode(resourcePath, "UTF-8").contains("../")) {
@@ -195,6 +199,7 @@ else if (resource instanceof ServletContextResource) {
 				return false;
 			}
 		}
+
 		return true;
 	}
 

spring-webmvc/src/main/java/org/springframework/web/servlet/resource/ResourceHttpRequestHandler.java

@@ -416,7 +416,7 @@ protected MediaType getMediaType(Resource resource) {
 	 */
 	protected void setETagHeader(HttpServletRequest request, HttpServletResponse response) {
 		String versionString = (String) request.getAttribute(VersionResourceResolver.RESOURCE_VERSION_ATTRIBUTE);
-		if(versionString != null) {
+		if (versionString != null) {
 			response.setHeader(HttpHeaders.ETAG, "\"" + versionString + "\"");
 		}
 	}

spring-webmvc/src/main/java/org/springframework/web/servlet/resource/ResourceUrlProvider.java

@@ -36,7 +36,6 @@
 import org.springframework.web.servlet.handler.SimpleUrlHandlerMapping;
 import org.springframework.web.util.UrlPathHelper;
 
-
 /**
  * A central component to use to obtain the public URL path that clients should
  * use to access a static resource.
@@ -130,7 +129,7 @@ public void onApplicationEvent(ContextRefreshedEvent event) {
 			if (this.handlerMap.isEmpty() && logger.isDebugEnabled()) {
 				logger.debug("No resource handling mappings found");
 			}
-			if(!this.handlerMap.isEmpty()) {
+			if (!this.handlerMap.isEmpty()) {
 				this.autodetect = false;
 			}
 		}

spring-webmvc/src/main/java/org/springframework/web/servlet/resource/VersionResourceResolver.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.springframework.web.servlet.resource;
 
 import java.util.ArrayList;
@@ -167,7 +168,7 @@ protected Resource resolveResourceInternal(HttpServletRequest request, String re
 			if (logger.isTraceEnabled()) {
 				logger.trace("resource matches extracted version");
 			}
-			if(request != null) {
+			if (request != null) {
 				request.setAttribute(VersionResourceResolver.RESOURCE_VERSION_ATTRIBUTE, candidateVersion);
 			}
 			return baseResource;

spring-webmvc/src/main/java/org/springframework/web/servlet/resource/WebJarsResourceResolver.java

@@ -25,18 +25,18 @@
 import org.springframework.core.io.Resource;
 
 /**
- * A {@code ResourceResolver} that delegates to the chain to locate a resource
- * and then attempts to find a matching versioned resource contained in a WebJar JAR file.
+ * A {@code ResourceResolver} that delegates to the chain to locate a resource and then
+ * attempts to find a matching versioned resource contained in a WebJar JAR file.
  *
  * <p>This allows WebJars.org users to write version agnostic paths in their templates,
  * like {@code <script src="/jquery/jquery.min.js"/>}.
  * This path will be resolved to the unique version {@code <script src="/jquery/1.2.0/jquery.min.js"/>},
  * which is a better fit for HTTP caching and version management in applications.
  *
- * <p>This also resolves Resources for version agnostic HTTP requests {@code "GET /jquery/jquery.min.js"}.
+ * <p>This also resolves resources for version agnostic HTTP requests {@code "GET /jquery/jquery.min.js"}.
  *
- * <p>This resolver requires the "org.webjars:webjars-locator" library on classpath, and is automatically
- * registered if that library is present.
+ * <p>This resolver requires the "org.webjars:webjars-locator" library on classpath,
+ * and is automatically registered if that library is present.
  *
  * @author Brian Clozel
  * @since 4.2
@@ -49,12 +49,7 @@ public class WebJarsResourceResolver extends AbstractResourceResolver {
 
 	private final static int WEBJARS_LOCATION_LENGTH = WEBJARS_LOCATION.length();
 
-	private final WebJarAssetLocator webJarAssetLocator;
-
-
-	public WebJarsResourceResolver() {
-		this.webJarAssetLocator = new WebJarAssetLocator();
-	}
+	private final WebJarAssetLocator webJarAssetLocator = new WebJarAssetLocator();
 
 
 	@Override
@@ -64,7 +59,7 @@ protected Resource resolveResourceInternal(HttpServletRequest request, String re
 		Resource resolved = chain.resolveResource(request, requestPath, locations);
 		if (resolved == null) {
 			String webJarResourcePath = findWebJarResourcePath(requestPath);
-			if(webJarResourcePath != null) {
+			if (webJarResourcePath != null) {
 				return chain.resolveResource(request, webJarResourcePath, locations);
 			}
 		}
@@ -78,7 +73,7 @@ protected String resolveUrlPathInternal(String resourceUrlPath,
 		String path = chain.resolveUrlPath(resourceUrlPath, locations);
 		if (path == null) {
 			String webJarResourcePath = findWebJarResourcePath(resourceUrlPath);
-			if(webJarResourcePath != null) {
+			if (webJarResourcePath != null) {
 				return chain.resolveUrlPath(webJarResourcePath, locations);
 			}
 		}