diff --git a/spring-faces/src/main/java/org/springframework/faces/model/ManySelectionTrackingListDataModel.java b/spring-faces/src/main/java/org/springframework/faces/model/ManySelectionTrackingListDataModel.java index edc9a212..05595c77 100644 --- a/spring-faces/src/main/java/org/springframework/faces/model/ManySelectionTrackingListDataModel.java +++ b/spring-faces/src/main/java/org/springframework/faces/model/ManySelectionTrackingListDataModel.java @@ -8,7 +8,7 @@ import javax.faces.model.DataModel; import org.springframework.util.Assert; /** - * A {@link DataModel} implementation that tracks the currently selected rows, allowing any number of rows to be + * A {@link DataModel} implementation that tracks the currently selected rows, allowing any number of rows to be * selected at one time. * * @author Jeremy Grelle @@ -17,10 +17,6 @@ public class ManySelectionTrackingListDataModel extends SerializableListDataMode private List selections = new ArrayList(); - public ManySelectionTrackingListDataModel(List list) { - super(list); - } - public List getSelections() { return selections; } diff --git a/spring-faces/src/main/java/org/springframework/faces/model/OneSelectionTrackingListDataModel.java b/spring-faces/src/main/java/org/springframework/faces/model/OneSelectionTrackingListDataModel.java index 44e217b0..0c287a21 100644 --- a/spring-faces/src/main/java/org/springframework/faces/model/OneSelectionTrackingListDataModel.java +++ b/spring-faces/src/main/java/org/springframework/faces/model/OneSelectionTrackingListDataModel.java @@ -16,10 +16,6 @@ public class OneSelectionTrackingListDataModel extends SerializableListDataModel private List selections = new ArrayList(); - public OneSelectionTrackingListDataModel(List list) { - super(list); - } - public List getSelections() { return selections; } diff --git a/spring-faces/src/main/java/org/springframework/faces/model/converter/FacesConversionService.java b/spring-faces/src/main/java/org/springframework/faces/model/converter/FacesConversionService.java index ec985b44..daa314fa 100644 --- a/spring-faces/src/main/java/org/springframework/faces/model/converter/FacesConversionService.java +++ b/spring-faces/src/main/java/org/springframework/faces/model/converter/FacesConversionService.java @@ -1,9 +1,10 @@ package org.springframework.faces.model.converter; import org.springframework.binding.convert.ConversionService; -import org.springframework.binding.convert.support.CompositeConversionService; import org.springframework.binding.convert.support.DefaultConversionService; -import org.springframework.binding.convert.support.GenericConversionService; +import org.springframework.faces.model.ManySelectionTrackingListDataModel; +import org.springframework.faces.model.OneSelectionTrackingListDataModel; +import org.springframework.faces.model.SerializableListDataModel; /** * Convenient {@link ConversionService} implementation for JSF that composes JSF-specific converters with the standard @@ -11,29 +12,18 @@ import org.springframework.binding.convert.support.GenericConversionService; * * @author Jeremy Grelle */ -public class FacesConversionService extends CompositeConversionService { - - private static ConversionService[] conversionServices = new ConversionService[] { new DefaultConversionService(), - new InternalFacesConversionService() }; +public class FacesConversionService extends DefaultConversionService { public FacesConversionService() { - super(conversionServices); + super(); + addFacesConverters(); } - private static class InternalFacesConversionService extends GenericConversionService { + protected void addFacesConverters() { + addConverter(new DataModelConverter()); - /** - * Creates a new default conversion service, installing the default converters. - */ - public InternalFacesConversionService() { - addFacesConverters(); - } - - /** - * Add all default converters to the conversion service. - */ - protected void addFacesConverters() { - addConverter(new DataModelConverter()); - } + addAlias("listDataModel", SerializableListDataModel.class); + addAlias("selectOneDataModel", OneSelectionTrackingListDataModel.class); + addAlias("selectManyDataModel", ManySelectionTrackingListDataModel.class); } }