add a new method PropertyMapper.from(value).

polish

Author: nosan

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/batch/BasicBatchConfigurer.java

@@ -122,7 +122,7 @@ protected JobLauncher createJobLauncher() throws Exception {
 	protected JobRepository createJobRepository() throws Exception {
 		JobRepositoryFactoryBean factory = new JobRepositoryFactoryBean();
 		PropertyMapper map = PropertyMapper.get();
-		map.from(() -> this.dataSource).to(factory::setDataSource);
+		map.from(this.dataSource).to(factory::setDataSource);
 		map.from(this::determineIsolationLevel).whenNonNull()
 				.to(factory::setIsolationLevelForCreate);
 		map.from(this.properties::getTablePrefix).whenHasText()

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/kafka/ConcurrentKafkaListenerContainerFactoryConfigurer.java

@@ -85,9 +85,8 @@ private void configureListenerFactory(
 		PropertyMapper map = PropertyMapper.get();
 		Listener properties = this.properties.getListener();
 		map.from(properties::getConcurrency).whenNonNull().to(factory::setConcurrency);
-		map.from(() -> this.messageConverter).whenNonNull()
-				.to(factory::setMessageConverter);
-		map.from(() -> this.replyTemplate).whenNonNull().to(factory::setReplyTemplate);
+		map.from(this.messageConverter).whenNonNull().to(factory::setMessageConverter);
+		map.from(this.replyTemplate).whenNonNull().to(factory::setReplyTemplate);
 		map.from(properties::getType).whenEqualTo(Listener.Type.BATCH)
 				.toCall(() -> factory.setBatchListener(true));
 	}

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/embedded/TomcatWebServerFactoryCustomizer.java

@@ -82,7 +82,7 @@ public void customize(ConfigurableTomcatWebServerFactory factory) {
 						tomcatProperties.getMaxThreads()));
 		propertyMapper.from(tomcatProperties::getMinSpareThreads).when(this::isPositive)
 				.to((minSpareThreads) -> customizeMinThreads(factory, minSpareThreads));
-		propertyMapper.from(() -> determineMaxHttpHeaderSize()).when(this::isPositive)
+		propertyMapper.from(this::determineMaxHttpHeaderSize).when(this::isPositive)
 				.to((maxHttpHeaderSize) -> customizeMaxHttpHeaderSize(factory,
 						maxHttpHeaderSize));
 		propertyMapper.from(tomcatProperties::getMaxHttpPostSize)

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/embedded/UndertowWebServerFactoryCustomizer.java

@@ -81,7 +81,7 @@ public void customize(ConfigurableUndertowWebServerFactory factory) {
 				.to(factory::setAccessLogSuffix);
 		propertyMapper.from(accesslogProperties::isRotate)
 				.to(factory::setAccessLogRotate);
-		propertyMapper.from(() -> getOrDeduceUseForwardHeaders())
+		propertyMapper.from(this::getOrDeduceUseForwardHeaders)
 				.to(factory::setUseForwardHeaders);
 		propertyMapper.from(properties::getMaxHttpHeaderSize).when(this::isPositive)
 				.to((maxHttpHeaderSize) -> customizeMaxHttpHeaderSize(factory,

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/PropertyMapper.java

@@ -91,6 +91,21 @@ public PropertyMapper alwaysApplying(SourceOperator operator) {
 		return new PropertyMapper(this, operator);
 	}
 
+	/**
+	 * Return a new {@link Source} from the specified value that can be used to perform
+	 * the mapping.
+	 * @param <T> the source type
+	 * @param value the value
+	 * @return a {@link Source} that can be used to complete the mapping
+	 */
+	public <T> Source<T> from(T value) {
+		Source<T> source = getSource(() -> value);
+		if (this.sourceOperator != null) {
+			source = this.sourceOperator.apply(source);
+		}
+		return source;
+	}
+
 	/**
 	 * Return a new {@link Source} from the specified value supplier that can be used to
 	 * perform the mapping.

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/webservices/client/WebServiceTemplateBuilder.java

@@ -502,13 +502,12 @@ public <T extends WebServiceTemplate> T configure(T webServiceTemplate) {
 		configureMessageSenders(webServiceTemplate);
 		PropertyMapper map = PropertyMapper.get().alwaysApplyingWhenNonNull();
 		applyCustomizers(webServiceTemplate, this.internalCustomizers);
-		map.from(() -> this.marshaller).to(webServiceTemplate::setMarshaller);
-		map.from(() -> this.unmarshaller).to(webServiceTemplate::setUnmarshaller);
-		map.from(() -> this.destinationProvider)
-				.to(webServiceTemplate::setDestinationProvider);
-		map.from(() -> this.transformerFactoryClass)
+		map.from(this.marshaller).to(webServiceTemplate::setMarshaller);
+		map.from(this.unmarshaller).to(webServiceTemplate::setUnmarshaller);
+		map.from(this.destinationProvider).to(webServiceTemplate::setDestinationProvider);
+		map.from(this.transformerFactoryClass)
 				.to(webServiceTemplate::setTransformerFactoryClass);
-		map.from(() -> this.messageFactory).to(webServiceTemplate::setMessageFactory);
+		map.from(this.messageFactory).to(webServiceTemplate::setMessageFactory);
 		if (!CollectionUtils.isEmpty(this.interceptors)) {
 			Set<ClientInterceptor> merged = new LinkedHashSet<>(this.interceptors);
 			if (webServiceTemplate.getInterceptors() != null) {

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/context/properties/PropertyMapperTests.java

@@ -37,11 +37,37 @@ public class PropertyMapperTests {
 	@Rule
 	public ExpectedException thrown = ExpectedException.none();
 
+	@Test
+	public void fromNullValue() {
+		ExampleDest dest = new ExampleDest();
+		this.map.from((String) null).to(dest::setName);
+		assertThat(dest.getName()).isNull();
+	}
+
+	@Test
+	public void fromValue() {
+		ExampleDest dest = new ExampleDest();
+		this.map.from("Hello World").to(dest::setName);
+		assertThat(dest.getName()).isEqualTo("Hello World");
+	}
+
+	@Test
+	public void fromValueAsIntShouldAdaptSupplier() {
+		Integer result = this.map.from("123").asInt(Long::valueOf)
+				.toInstance(Integer::new);
+		assertThat(result).isEqualTo(123);
+	}
+
+	@Test
+	public void fromValueAlwaysApplyingWhenNonNullShouldAlwaysApplyNonNullToSource() {
+		this.map.alwaysApplyingWhenNonNull().from((String) null).toCall(Assert::fail);
+	}
+
 	@Test
 	public void fromWhenSupplierIsNullShouldThrowException() {
 		this.thrown.expect(IllegalArgumentException.class);
 		this.thrown.expectMessage("Supplier must not be null");
-		this.map.from(null);
+		this.map.from((Supplier<?>) null);
 	}
 
 	@Test