You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by ni...@apache.org on 2007/01/20 12:50:38 UTC
svn commit: r498105 -
/jakarta/commons/proper/beanutils/trunk/optional/bean-collections/src/java/org/apache/commons/beanutils/BeanComparator.java
Author: niallp
Date: Sat Jan 20 03:50:37 2007
New Revision: 498105
URL: http://svn.apache.org/viewvc?view=rev&rev=498105
Log:
BEANUTILS-267 - BeanComparator constructor should check the comparator for null and default to ComparableComparator - thanks to Jacob Kjome
Modified:
jakarta/commons/proper/beanutils/trunk/optional/bean-collections/src/java/org/apache/commons/beanutils/BeanComparator.java
Modified: jakarta/commons/proper/beanutils/trunk/optional/bean-collections/src/java/org/apache/commons/beanutils/BeanComparator.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/beanutils/trunk/optional/bean-collections/src/java/org/apache/commons/beanutils/BeanComparator.java?view=diff&rev=498105&r1=498104&r2=498105
==============================================================================
--- jakarta/commons/proper/beanutils/trunk/optional/bean-collections/src/java/org/apache/commons/beanutils/BeanComparator.java (original)
+++ jakarta/commons/proper/beanutils/trunk/optional/bean-collections/src/java/org/apache/commons/beanutils/BeanComparator.java Sat Jan 20 03:50:37 2007
@@ -100,7 +100,11 @@
*/
public BeanComparator( String property, Comparator comparator ) {
setProperty( property );
- this.comparator = comparator;
+ if (comparator != null) {
+ this.comparator = comparator;
+ } else {
+ this.comparator = ComparableComparator.getInstance();
+ }
}
/**
---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org