Merge pull request #96 from mminella/BATCH-1938

BATCH-1938: Updated 2.2 branch to support Spring 3.2.0.RELEASE

archetypes/simple-cli/pom.xml

@@ -13,7 +13,7 @@
 		the project jar on the classpath and run the
 		CommandLineJobRunner directly or with "java -jar launch-context.xml job1".</description>
 	<properties>
-		<spring.framework.version>3.1.2.RELEASE</spring.framework.version>
+		<spring.framework.version>3.2.0.RELEASE</spring.framework.version>
 		<spring.batch.version>2.2.0.BUILD-SNAPSHOT</spring.batch.version>
 		<dependency.locations.enabled>false</dependency.locations.enabled>
 		<junit.version>4.10</junit.version>

spring-batch-core/pom.xml

@@ -41,7 +41,9 @@
 			<artifactId>hsqldb</artifactId>
 		</dependency>
 		<dependency>
-			<groupId>com.h2database</groupId><artifactId>h2</artifactId><version>1.2.132</version>
+			<groupId>com.h2database</groupId>
+			<artifactId>h2</artifactId>
+			<version>1.2.132</version>
 		</dependency>
 		<dependency>
 			<groupId>commons-io</groupId>

spring-batch-core/src/main/java/org/springframework/batch/core/scope/util/PlaceholderTargetSource.java

@@ -16,6 +16,7 @@
 package org.springframework.batch.core.scope.util;
 
 import java.beans.PropertyEditor;
+import java.lang.reflect.Field;
 import java.util.Date;
 import java.util.List;
 import java.util.Map;
@@ -89,6 +90,7 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 	 * @see
 	 * org.springframework.beans.factory.InitializingBean#afterPropertiesSet()
 	 */
+	@Override
 	public void afterPropertiesSet() {
 		Assert.notNull(contextFactory, "The ContextFactory must be set.");
 		beanName = getTargetBeanName() + "#" + contextFactory.getContextId();
@@ -116,9 +118,16 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 		beanFactory.copyConfigurationFrom(listableBeanFactory);
 
 		final TypeConverter contextTypeConverter = new TypeConverter() {
+			@Override
 			@SuppressWarnings({ "unchecked", "rawtypes" })
 			public Object convertIfNecessary(Object value, Class requiredType, MethodParameter methodParam)
 					throws TypeMismatchException {
+				return doConvertIfNecessary(value, requiredType,
+						methodParam);
+			}
+
+			private Object doConvertIfNecessary(Object value,
+					Class requiredType, Object param) {
 				Object result = null;
 				if (value instanceof String) {
 					String key = (String) value;
@@ -155,12 +164,30 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 						result = value.toString();
 					}
 				}
-				return result != null ? result : typeConverter.convertIfNecessary(value, requiredType, methodParam);
+
+				if(result != null) {
+					return result;
+				}
+				else if(param instanceof Field) {
+					return typeConverter.convertIfNecessary(value, requiredType, (Field) param);
+				}
+				else {
+					return typeConverter.convertIfNecessary(value, requiredType, (MethodParameter) param);
+				}
 			}
 
+			@Override
 			@SuppressWarnings("rawtypes")
 			public Object convertIfNecessary(Object value, Class requiredType) throws TypeMismatchException {
-				return convertIfNecessary(value, requiredType, null);
+				return doConvertIfNecessary(value, requiredType, null);
+			}
+
+			@Override
+			public Object convertIfNecessary(Object value,
+					Class requiredType, Field field)
+					throws TypeMismatchException {
+				return doConvertIfNecessary(value, requiredType,
+						field);
 			}
 		};
 		beanFactory.setTypeConverter(contextTypeConverter);
@@ -220,7 +247,7 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 		try {
 			// Give it one chance to convert - this forces the default editors
 			// to be registered
-			result = (String) typeConverter.convertIfNecessary(value, String.class);
+			result = typeConverter.convertIfNecessary(value, String.class);
 		}
 		catch (TypeMismatchException e) {
 			// ignore
@@ -309,6 +336,7 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 			super(new PlaceholderStringValueResolver(typeConverter));
 		}
 
+		@Override
 		@SuppressWarnings({ "unchecked", "rawtypes" })
 		protected Object resolveValue(Object value) {
 
@@ -367,14 +395,14 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 			else if (value instanceof BeanDefinition) {
 
 				BeanDefinition newValue = new GenericBeanDefinition((BeanDefinition) value);
-				visitBeanDefinition((BeanDefinition) newValue);
+				visitBeanDefinition(newValue);
 				value = newValue;
 
 			}
 			else if (value instanceof BeanDefinitionHolder) {
 
 				BeanDefinition newValue = new GenericBeanDefinition(((BeanDefinitionHolder) value).getBeanDefinition());
-				visitBeanDefinition((BeanDefinition) newValue);
+				visitBeanDefinition(newValue);
 				value = newValue;
 
 			}
@@ -398,6 +426,7 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 			this.typeConverter = typeConverter;
 		}
 
+		@Override
 		public String resolveStringValue(String strVal) {
 			if (!strVal.contains(PLACEHOLDER_PREFIX)) {
 				return strVal;
@@ -454,7 +483,7 @@ public class PlaceholderTargetSource extends SimpleBeanTargetSource implements I
 				Class<?> requiredType, TypeConverter typeConverter) {
 			Object property = convertFromContext(key, requiredType);
 			if (property != null) {
-				result.replace(first, next + 1, (String) typeConverter.convertIfNecessary(property, String.class));
+				result.replace(first, next + 1, typeConverter.convertIfNecessary(property, String.class));
 				return true;
 			}
 			return false;

spring-batch-parent/pom.xml

@@ -29,7 +29,7 @@
 		</developer>
 	</developers>
 	<properties>
-		<spring.framework.version>3.1.2.RELEASE</spring.framework.version>
+		<spring.framework.version>3.2.0.RELEASE</spring.framework.version>
 		<spring.amqp.version>1.1.2.RELEASE</spring.amqp.version>
 		<junit.version>4.10</junit.version>
 		<bundlor.version>1.0.0.RELEASE</bundlor.version>