SWF-370 - Add support for automatically adapting a returned list for display in a DataTable to a JSF DataModel.

spring-faces/src/main/java/org/springframework/faces/model/SerializableListDataModel.java

@@ -1,6 +1,7 @@
 package org.springframework.faces.model;
 
 import java.io.Serializable;
+import java.util.ArrayList;
 import java.util.List;
 
 import javax.faces.model.DataModel;
@@ -18,11 +19,18 @@ public class SerializableListDataModel extends DataModel implements Serializable
 
 	private List data;
 
+	public SerializableListDataModel() {
+		this(new ArrayList());
+	}
+
 	/**
 	 * Adapt the list to a data model;
 	 * @param list the list
 	 */
 	public SerializableListDataModel(List list) {
+		if (list == null) {
+			list = new ArrayList();
+		}
 		setWrappedData(list);
 	}
 
@@ -65,7 +73,9 @@ public class SerializableListDataModel extends DataModel implements Serializable
 	}
 
 	public void setWrappedData(Object data) {
-		Assert.notNull(data, "A non-null List is required for " + getClass());
+		if (data == null) {
+			data = new ArrayList();
+		}
 		Assert.isInstanceOf(List.class, data, "The data object for " + getClass() + " must be a List");
 		this.data = (List) data;
 		int newRowIndex = 0;

spring-faces/src/main/java/org/springframework/faces/model/converter/DataModelConverter.java

@@ -0,0 +1,29 @@
+package org.springframework.faces.model.converter;
+
+import java.lang.reflect.Constructor;
+import java.util.List;
+
+import javax.faces.model.DataModel;
+
+import org.springframework.binding.convert.ConversionContext;
+import org.springframework.binding.convert.support.AbstractConverter;
+import org.springframework.util.ClassUtils;
+
+public class DataModelConverter extends AbstractConverter {
+
+	protected Object doConvert(Object source, Class targetClass, ConversionContext context) throws Exception {
+		Constructor emptyConstructor = ClassUtils.getConstructorIfAvailable(targetClass, new Class[] {});
+		DataModel model = (DataModel) emptyConstructor.newInstance(new Object[] {});
+		model.setWrappedData(source);
+		return model;
+	}
+
+	public Class[] getSourceClasses() {
+		return new Class[] { Object[].class, List.class, Object.class };
+	}
+
+	public Class[] getTargetClasses() {
+		return new Class[] { DataModel.class };
+	}
+
+}