Merge branch '5.2.x'

Author: jhoeller

spring-beans/src/main/java/org/springframework/beans/factory/config/YamlProcessor.java

@@ -432,16 +432,14 @@ public enum ResolutionMethod {
 	/**
 	 * {@link Constructor} that supports filtering of unsupported types.
 	 * <p>If an unsupported type is encountered in a YAML document, an
-	 * {@link IllegalStateException} will be thrown from {@link #getClassForName(String)}.
-	 * @since 5.1.16
+	 * {@link IllegalStateException} will be thrown from {@link #getClassForName}.
 	 */
 	private class FilteringConstructor extends Constructor {
 
 		FilteringConstructor(LoaderOptions loaderOptions) {
 			super(loaderOptions);
 		}
 
-
 		@Override
 		protected Class<?> getClassForName(String name) throws ClassNotFoundException {
 			Assert.state(YamlProcessor.this.supportedTypes.contains(name),

spring-context/src/main/java/org/springframework/context/support/AbstractApplicationContext.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2019 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -79,6 +79,7 @@
 import org.springframework.core.io.support.ResourcePatternResolver;
 import org.springframework.lang.Nullable;
 import org.springframework.util.Assert;
+import org.springframework.util.CollectionUtils;
 import org.springframework.util.ObjectUtils;
 import org.springframework.util.ReflectionUtils;
 
@@ -836,7 +837,7 @@ protected void registerListeners() {
 		// Publish early application events now that we finally have a multicaster...
 		Set<ApplicationEvent> earlyEventsToProcess = this.earlyApplicationEvents;
 		this.earlyApplicationEvents = null;
-		if (earlyEventsToProcess != null) {
+		if (!CollectionUtils.isEmpty(earlyEventsToProcess)) {
 			for (ApplicationEvent earlyEvent : earlyEventsToProcess) {
 				getApplicationEventMulticaster().multicastEvent(earlyEvent);
 			}

spring-core/src/main/java/org/springframework/core/env/AbstractPropertyResolver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2018 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -223,6 +223,9 @@ public String resolveRequiredPlaceholders(String text) throws IllegalArgumentExc
 	 * @see #setIgnoreUnresolvableNestedPlaceholders
 	 */
 	protected String resolveNestedPlaceholders(String value) {
+		if (value.isEmpty()) {
+			return value;
+		}
 		return (this.ignoreUnresolvableNestedPlaceholders ?
 				resolvePlaceholders(value) : resolveRequiredPlaceholders(value));
 	}

spring-core/src/main/java/org/springframework/core/env/EnumerablePropertySource.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2018 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -44,10 +44,20 @@
  */
 public abstract class EnumerablePropertySource<T> extends PropertySource<T> {
 
+	/**
+	 * Create a new {@code EnumerablePropertySource} with the given name and source object.
+	 * @param name the associated name
+	 * @param source the source object
+	 */
 	public EnumerablePropertySource(String name, T source) {
 		super(name, source);
 	}
 
+	/**
+	 * Create a new {@code EnumerablePropertySource} with the given name and with a new
+	 * {@code Object} instance as the underlying source.
+	 * @param name the associated name
+	 */
 	protected EnumerablePropertySource(String name) {
 		super(name);
 	}

spring-core/src/main/java/org/springframework/core/env/MapPropertySource.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2014 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -23,6 +23,8 @@
 
 /**
  * {@link PropertySource} that reads keys and values from a {@code Map} object.
+ * The underlying map should not contain any {@code null} values in order to
+ * comply with {@link #getProperty} and {@link #containsProperty} semantics.
  *
  * @author Chris Beams
  * @author Juergen Hoeller
@@ -31,6 +33,12 @@
  */
 public class MapPropertySource extends EnumerablePropertySource<Map<String, Object>> {
 
+	/**
+	 * Create a new {@code MapPropertySource} with the given name and {@code Map}.
+	 * @param name the associated name
+	 * @param source the Map source (without {@code null} values in order to get
+	 * consistent {@link #getProperty} and {@link #containsProperty} behavior)
+	 */
 	public MapPropertySource(String name, Map<String, Object> source) {
 		super(name, source);
 	}

spring-core/src/main/java/org/springframework/core/env/PropertyResolver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2016 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -98,7 +98,6 @@ public interface PropertyResolver {
 	 * @return the resolved String (never {@code null})
 	 * @throws IllegalArgumentException if given text is {@code null}
 	 * @see #resolveRequiredPlaceholders
-	 * @see org.springframework.util.SystemPropertyUtils#resolvePlaceholders(String)
 	 */
 	String resolvePlaceholders(String text);
 
@@ -109,7 +108,6 @@ public interface PropertyResolver {
 	 * @return the resolved String (never {@code null})
 	 * @throws IllegalArgumentException if given text is {@code null}
 	 * or if any placeholders are unresolvable
-	 * @see org.springframework.util.SystemPropertyUtils#resolvePlaceholders(String, boolean)
 	 */
 	String resolveRequiredPlaceholders(String text) throws IllegalArgumentException;
 

spring-core/src/main/java/org/springframework/core/env/PropertySource.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2018 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -68,6 +68,8 @@ public abstract class PropertySource<T> {
 
 	/**
 	 * Create a new {@code PropertySource} with the given name and source object.
+	 * @param name the associated name
+	 * @param source the source object
 	 */
 	public PropertySource(String name, T source) {
 		Assert.hasText(name, "Property source name must contain at least one character");

spring-core/src/main/java/org/springframework/util/CollectionUtils.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2019 the original author or authors.
+ * Copyright 2002-2020 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.
@@ -16,15 +16,13 @@
 
 package org.springframework.util;
 
-import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Enumeration;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import java.util.Properties;
@@ -412,25 +410,28 @@ public static <E> Iterator<E> toIterator(@Nullable Enumeration<E> enumeration) {
 
 	/**
 	 * Adapt a {@code Map<K, List<V>>} to an {@code MultiValueMap<K, V>}.
-	 * @param map the original map
-	 * @return the multi-value map
+	 * @param targetMap the original map
+	 * @return the adapted multi-value map (wrapping the original map)
 	 * @since 3.1
 	 */
-	public static <K, V> MultiValueMap<K, V> toMultiValueMap(Map<K, List<V>> map) {
-		return new MultiValueMapAdapter<>(map);
+	public static <K, V> MultiValueMap<K, V> toMultiValueMap(Map<K, List<V>> targetMap) {
+		Assert.notNull(targetMap, "'targetMap' must not be null");
+		return new MultiValueMapAdapter<>(targetMap);
 	}
 
 	/**
 	 * Return an unmodifiable view of the specified multi-value map.
-	 * @param  map the map for which an unmodifiable view is to be returned.
-	 * @return an unmodifiable view of the specified multi-value map.
+	 * @param targetMap the map for which an unmodifiable view is to be returned.
+	 * @return an unmodifiable view of the specified multi-value map
 	 * @since 3.1
 	 */
 	@SuppressWarnings("unchecked")
-	public static <K, V> MultiValueMap<K, V> unmodifiableMultiValueMap(MultiValueMap<? extends K, ? extends V> map) {
-		Assert.notNull(map, "'map' must not be null");
-		Map<K, List<V>> result = new LinkedHashMap<>(map.size());
-		map.forEach((key, value) -> {
+	public static <K, V> MultiValueMap<K, V> unmodifiableMultiValueMap(
+			MultiValueMap<? extends K, ? extends V> targetMap) {
+
+		Assert.notNull(targetMap, "'targetMap' must not be null");
+		Map<K, List<V>> result = new LinkedHashMap<>(targetMap.size());
+		targetMap.forEach((key, value) -> {
 			List<? extends V> values = Collections.unmodifiableList(value);
 			result.put(key, (List<V>) values);
 		});
@@ -467,141 +468,4 @@ public void remove() throws UnsupportedOperationException {
 	}
 
 
-	/**
-	 * Adapts a Map to the MultiValueMap contract.
-	 */
-	@SuppressWarnings("serial")
-	private static class MultiValueMapAdapter<K, V> implements MultiValueMap<K, V>, Serializable {
-
-		private final Map<K, List<V>> map;
-
-		public MultiValueMapAdapter(Map<K, List<V>> map) {
-			Assert.notNull(map, "'map' must not be null");
-			this.map = map;
-		}
-
-		@Override
-		@Nullable
-		public V getFirst(K key) {
-			List<V> values = this.map.get(key);
-			return (values != null ? values.get(0) : null);
-		}
-
-		@Override
-		public void add(K key, @Nullable V value) {
-			List<V> values = this.map.computeIfAbsent(key, k -> new LinkedList<>());
-			values.add(value);
-		}
-
-		@Override
-		public void addAll(K key, List<? extends V> values) {
-			List<V> currentValues = this.map.computeIfAbsent(key, k -> new LinkedList<>());
-			currentValues.addAll(values);
-		}
-
-		@Override
-		public void addAll(MultiValueMap<K, V> values) {
-			for (Entry<K, List<V>> entry : values.entrySet()) {
-				addAll(entry.getKey(), entry.getValue());
-			}
-		}
-
-		@Override
-		public void set(K key, @Nullable V value) {
-			List<V> values = new LinkedList<>();
-			values.add(value);
-			this.map.put(key, values);
-		}
-
-		@Override
-		public void setAll(Map<K, V> values) {
-			values.forEach(this::set);
-		}
-
-		@Override
-		public Map<K, V> toSingleValueMap() {
-			LinkedHashMap<K, V> singleValueMap = new LinkedHashMap<>(this.map.size());
-			this.map.forEach((key, value) -> singleValueMap.put(key, value.get(0)));
-			return singleValueMap;
-		}
-
-		@Override
-		public int size() {
-			return this.map.size();
-		}
-
-		@Override
-		public boolean isEmpty() {
-			return this.map.isEmpty();
-		}
-
-		@Override
-		public boolean containsKey(Object key) {
-			return this.map.containsKey(key);
-		}
-
-		@Override
-		public boolean containsValue(Object value) {
-			return this.map.containsValue(value);
-		}
-
-		@Override
-		public List<V> get(Object key) {
-			return this.map.get(key);
-		}
-
-		@Override
-		public List<V> put(K key, List<V> value) {
-			return this.map.put(key, value);
-		}
-
-		@Override
-		public List<V> remove(Object key) {
-			return this.map.remove(key);
-		}
-
-		@Override
-		public void putAll(Map<? extends K, ? extends List<V>> map) {
-			this.map.putAll(map);
-		}
-
-		@Override
-		public void clear() {
-			this.map.clear();
-		}
-
-		@Override
-		public Set<K> keySet() {
-			return this.map.keySet();
-		}
-
-		@Override
-		public Collection<List<V>> values() {
-			return this.map.values();
-		}
-
-		@Override
-		public Set<Entry<K, List<V>>> entrySet() {
-			return this.map.entrySet();
-		}
-
-		@Override
-		public boolean equals(@Nullable Object other) {
-			if (this == other) {
-				return true;
-			}
-			return this.map.equals(other);
-		}
-
-		@Override
-		public int hashCode() {
-			return this.map.hashCode();
-		}
-
-		@Override
-		public String toString() {
-			return this.map.toString();
-		}
-	}
-
 }

spring-core/src/main/java/org/springframework/util/LinkedCaseInsensitiveMap.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2019 the original author or authors.
+ * Copyright 2002-2020 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,8 +273,8 @@ public LinkedCaseInsensitiveMap<V> clone() {
 	}
 
 	@Override
-	public boolean equals(@Nullable Object obj) {
-		return this.targetMap.equals(obj);
+	public boolean equals(@Nullable Object other) {
+		return (this == other || this.targetMap.equals(other));
 	}
 
 	@Override