diff --git a/spring-expression/src/main/java/org/springframework/expression/spel/support/StandardEvaluationContext.java b/spring-expression/src/main/java/org/springframework/expression/spel/support/StandardEvaluationContext.java index c8cbac7ef0d..de0a8c781cc 100644 --- a/spring-expression/src/main/java/org/springframework/expression/spel/support/StandardEvaluationContext.java +++ b/spring-expression/src/main/java/org/springframework/expression/spel/support/StandardEvaluationContext.java @@ -18,9 +18,9 @@ package org.springframework.expression.spel.support; import java.lang.reflect.Method; import java.util.ArrayList; -import java.util.HashMap; import java.util.List; import java.util.Map; +import java.util.concurrent.ConcurrentHashMap; import org.springframework.core.convert.TypeDescriptor; import org.springframework.expression.BeanResolver; @@ -88,7 +88,7 @@ public class StandardEvaluationContext implements EvaluationContext { private OperatorOverloader operatorOverloader = new StandardOperatorOverloader(); - private final Map variables = new HashMap<>(); + private final Map variables = new ConcurrentHashMap<>(); /** @@ -203,7 +203,7 @@ public class StandardEvaluationContext implements EvaluationContext { @Override public TypeConverter getTypeConverter() { if (this.typeConverter == null) { - this.typeConverter = new StandardTypeConverter(); + this.typeConverter = new StandardTypeConverter(); } return this.typeConverter; } @@ -230,11 +230,16 @@ public class StandardEvaluationContext implements EvaluationContext { @Override public void setVariable(String name, @Nullable Object value) { - this.variables.put(name, value); + if (value != null) { + this.variables.put(name, value); + } + else { + this.variables.remove(name); + } } - public void setVariables(Map variables) { - this.variables.putAll(variables); + public void setVariables(Map variables) { + variables.forEach(this::setVariable); } public void registerFunction(String name, Method method) {