diff --git a/spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java b/spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java index a1497b459db..04ad1ac424d 100644 --- a/spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java +++ b/spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java @@ -695,7 +695,7 @@ public abstract class DataBufferUtils { @Override public byte[] delimiter() { - Assert.state(this.longestDelimiter != NO_DELIMITER, "Illegal state!"); + Assert.state(this.longestDelimiter != NO_DELIMITER, "'delimiter' not set"); return this.longestDelimiter; } diff --git a/spring-messaging/src/main/java/org/springframework/messaging/support/MessageHeaderAccessor.java b/spring-messaging/src/main/java/org/springframework/messaging/support/MessageHeaderAccessor.java index 8ea3f285396..afbdd8d2888 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/support/MessageHeaderAccessor.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/support/MessageHeaderAccessor.java @@ -43,7 +43,7 @@ import org.springframework.util.StringUtils; * strongly typed accessors for specific headers, the ability to leave headers * in a {@link Message} mutable, and the option to suppress automatic generation * of {@link MessageHeaders#ID id} and {@link MessageHeaders#TIMESTAMP - * timesteamp} headers. Subclasses such as {@link NativeMessageHeaderAccessor} + * timestamp} headers. Subclasses such as {@link NativeMessageHeaderAccessor} * and others provide support for managing processing vs external source headers * as well as protocol specific headers. * diff --git a/spring-r2dbc/src/main/java/org/springframework/r2dbc/core/DefaultDatabaseClient.java b/spring-r2dbc/src/main/java/org/springframework/r2dbc/core/DefaultDatabaseClient.java index 366254ae4cf..91b21d74f22 100644 --- a/spring-r2dbc/src/main/java/org/springframework/r2dbc/core/DefaultDatabaseClient.java +++ b/spring-r2dbc/src/main/java/org/springframework/r2dbc/core/DefaultDatabaseClient.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2020 the original author or authors. + * Copyright 2002-2022 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. @@ -273,7 +273,7 @@ class DefaultDatabaseClient implements DatabaseClient { public DefaultGenericExecuteSpec bind(String name, Object value) { assertNotPreparedOperation(); - Assert.hasText(name, "Parameter name must not be null or empty!"); + Assert.hasText(name, "Parameter name must not be null or empty"); Assert.notNull(value, () -> String.format( "Value for parameter %s must not be null. Use bindNull(…) instead.", name)); @@ -291,7 +291,7 @@ class DefaultDatabaseClient implements DatabaseClient { @Override public DefaultGenericExecuteSpec bindNull(String name, Class type) { assertNotPreparedOperation(); - Assert.hasText(name, "Parameter name must not be null or empty!"); + Assert.hasText(name, "Parameter name must not be null or empty"); Map byName = new LinkedHashMap<>(this.byName); byName.put(name, Parameter.empty(type)); @@ -301,7 +301,7 @@ class DefaultDatabaseClient implements DatabaseClient { @Override public DefaultGenericExecuteSpec filter(StatementFilterFunction filter) { - Assert.notNull(filter, "Statement FilterFunction must not be null"); + Assert.notNull(filter, "StatementFilterFunction must not be null"); return new DefaultGenericExecuteSpec( this.byIndex, this.byName, this.sqlSupplier, this.filterFunction.andThen(filter)); } @@ -449,7 +449,7 @@ class DefaultDatabaseClient implements DatabaseClient { private String getRequiredSql(Supplier sqlSupplier) { String sql = sqlSupplier.get(); - Assert.state(StringUtils.hasText(sql), "SQL returned by SQL supplier must not be empty!"); + Assert.state(StringUtils.hasText(sql), "SQL returned by supplier must not be empty"); return sql; } } diff --git a/spring-test/src/main/java/org/springframework/mock/web/PassThroughFilterChain.java b/spring-test/src/main/java/org/springframework/mock/web/PassThroughFilterChain.java index dfc5f7b303a..372fb1f5dd2 100644 --- a/spring-test/src/main/java/org/springframework/mock/web/PassThroughFilterChain.java +++ b/spring-test/src/main/java/org/springframework/mock/web/PassThroughFilterChain.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2018 the original author or authors. + * Copyright 2002-2022 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. @@ -84,7 +84,7 @@ public class PassThroughFilterChain implements FilterChain { this.filter.doFilter(request, response, this.nextFilterChain); } else { - Assert.state(this.servlet != null, "Neither a Filter not a Servlet set"); + Assert.state(this.servlet != null, "Neither a Filter nor a Servlet has been set"); this.servlet.service(request, response); } } diff --git a/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java b/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java index 960ead38d05..14b17ebe098 100644 --- a/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java +++ b/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2020 the original author or authors. + * Copyright 2002-2022 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. @@ -196,7 +196,7 @@ public final class WebAsyncManager { */ public void registerCallableInterceptor(Object key, CallableProcessingInterceptor interceptor) { Assert.notNull(key, "Key is required"); - Assert.notNull(interceptor, "CallableProcessingInterceptor is required"); + Assert.notNull(interceptor, "CallableProcessingInterceptor is required"); this.callableInterceptors.put(key, interceptor); } diff --git a/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/PassThroughFilterChain.java b/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/PassThroughFilterChain.java index e3537156ac5..10a1eba2d06 100644 --- a/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/PassThroughFilterChain.java +++ b/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/PassThroughFilterChain.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2018 the original author or authors. + * Copyright 2002-2022 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. @@ -84,7 +84,7 @@ public class PassThroughFilterChain implements FilterChain { this.filter.doFilter(request, response, this.nextFilterChain); } else { - Assert.state(this.servlet != null, "Neither a Filter not a Servlet set"); + Assert.state(this.servlet != null, "Neither a Filter nor a Servlet has been set"); this.servlet.service(request, response); } } diff --git a/spring-webflux/src/main/java/org/springframework/web/reactive/result/view/script/ScriptTemplateView.java b/spring-webflux/src/main/java/org/springframework/web/reactive/result/view/script/ScriptTemplateView.java index b831397518e..415b126475a 100644 --- a/spring-webflux/src/main/java/org/springframework/web/reactive/result/view/script/ScriptTemplateView.java +++ b/spring-webflux/src/main/java/org/springframework/web/reactive/result/view/script/ScriptTemplateView.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2019 the original author or authors. + * Copyright 2002-2022 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. @@ -227,12 +227,12 @@ public class ScriptTemplateView extends AbstractUrlBasedView { engineCount++; } Assert.isTrue(engineCount == 1, - "You should define either 'engine', 'engineSupplier' or 'engineName'."); + "You should define either 'engine', 'engineSupplier', or 'engineName'."); if (Boolean.FALSE.equals(this.sharedEngine)) { Assert.isTrue(this.engine == null, "When 'sharedEngine' is set to false, you should specify the " + - "script engine using 'engineName' or 'engineSupplier' , not 'engine'."); + "script engine using 'engineName' or 'engineSupplier', not 'engine'."); } else if (this.engine != null) { loadScripts(this.engine); diff --git a/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/script/ScriptTemplateViewTests.java b/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/script/ScriptTemplateViewTests.java index e583e4abbd2..879b5e00419 100644 --- a/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/script/ScriptTemplateViewTests.java +++ b/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/script/ScriptTemplateViewTests.java @@ -174,7 +174,7 @@ public class ScriptTemplateViewTests { this.view.setRenderFunction("render"); assertThatIllegalArgumentException().isThrownBy(() -> this.view.setApplicationContext(this.context)) - .withMessageContaining("You should define either 'engine', 'engineSupplier' or 'engineName'."); + .withMessageContaining("You should define either 'engine', 'engineSupplier', or 'engineName'."); } @Test // gh-23258 @@ -185,7 +185,7 @@ public class ScriptTemplateViewTests { this.view.setRenderFunction("render"); assertThatIllegalArgumentException().isThrownBy(() -> this.view.setApplicationContext(this.context)) - .withMessageContaining("You should define either 'engine', 'engineSupplier' or 'engineName'."); + .withMessageContaining("You should define either 'engine', 'engineSupplier', or 'engineName'."); } @Test // gh-23258 @@ -195,7 +195,7 @@ public class ScriptTemplateViewTests { this.view.setRenderFunction("render"); assertThatIllegalArgumentException().isThrownBy(() -> this.view.setApplicationContext(this.context)) - .withMessageContaining("You should define either 'engine', 'engineSupplier' or 'engineName'."); + .withMessageContaining("You should define either 'engine', 'engineSupplier', or 'engineName'."); } @Test diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilder.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilder.java index 0f5fa243aa0..80dcdd41105 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilder.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilder.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2021 the original author or authors. + * Copyright 2002-2022 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. @@ -449,7 +449,7 @@ public class MvcUriComponentsBuilder { */ public static MethodArgumentBuilder fromMappingName(@Nullable UriComponentsBuilder builder, String name) { WebApplicationContext wac = getWebApplicationContext(); - Assert.notNull(wac, "No WebApplicationContext. "); + Assert.notNull(wac, "No WebApplicationContext"); Map map = wac.getBeansOfType(RequestMappingInfoHandlerMapping.class); List handlerMethods = null; for (RequestMappingInfoHandlerMapping mapping : map.values()) { diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/StreamingResponseBody.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/StreamingResponseBody.java index eb307b4a62a..3c6fe1575ce 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/StreamingResponseBody.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/StreamingResponseBody.java @@ -16,7 +16,6 @@ package org.springframework.web.servlet.mvc.method.annotation; - import java.io.IOException; import java.io.OutputStream; diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/view/script/ScriptTemplateView.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/view/script/ScriptTemplateView.java index 1593a88ae41..5c5c8f19e23 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/view/script/ScriptTemplateView.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/view/script/ScriptTemplateView.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2019 the original author or authors. + * Copyright 2002-2022 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. @@ -258,12 +258,12 @@ public class ScriptTemplateView extends AbstractUrlBasedView { engineCount++; } Assert.isTrue(engineCount == 1, - "You should define either 'engine', 'engineSupplier' or 'engineName'."); + "You should define either 'engine', 'engineSupplier', or 'engineName'."); if (Boolean.FALSE.equals(this.sharedEngine)) { Assert.isTrue(this.engine == null, "When 'sharedEngine' is set to false, you should specify the " + - "script engine using 'engineName' or 'engineSupplier' , not 'engine'."); + "script engine using 'engineName' or 'engineSupplier', not 'engine'."); } else if (this.engine != null) { loadScripts(this.engine); diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/view/script/ScriptTemplateViewTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/view/script/ScriptTemplateViewTests.java index d546737cb80..6246272a6fa 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/view/script/ScriptTemplateViewTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/view/script/ScriptTemplateViewTests.java @@ -189,7 +189,7 @@ public class ScriptTemplateViewTests { this.view.setRenderFunction("render"); assertThatIllegalArgumentException().isThrownBy(() -> this.view.setApplicationContext(this.wac)) - .withMessageContaining("You should define either 'engine', 'engineSupplier' or 'engineName'."); + .withMessageContaining("You should define either 'engine', 'engineSupplier', or 'engineName'."); } @Test // gh-23258 @@ -200,7 +200,7 @@ public class ScriptTemplateViewTests { this.view.setRenderFunction("render"); assertThatIllegalArgumentException().isThrownBy(() -> this.view.setApplicationContext(this.wac)) - .withMessageContaining("You should define either 'engine', 'engineSupplier' or 'engineName'."); + .withMessageContaining("You should define either 'engine', 'engineSupplier', or 'engineName'."); } @Test // gh-23258 @@ -210,7 +210,7 @@ public class ScriptTemplateViewTests { this.view.setRenderFunction("render"); assertThatIllegalArgumentException().isThrownBy(() -> this.view.setApplicationContext(this.wac)) - .withMessageContaining("You should define either 'engine', 'engineSupplier' or 'engineName'."); + .withMessageContaining("You should define either 'engine', 'engineSupplier', or 'engineName'."); } @Test