[jboss-svn-commits] JBL Code SVN: r35714 - labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table.

[jboss-svn-commits at lists.jboss.org]

Author: ge0ffrey
Date: 2010-10-25 10:51:56 -0400 (Mon, 25 Oct 2010)
New Revision: 35714

Modified:
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeaderGroup.java
Log:
CellTable grid support: fix sorting issues

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeaderGroup.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeaderGroup.java	2010-10-25 14:08:27 UTC (rev 35713)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/table/SortableHeaderGroup.java	2010-10-25 14:51:56 UTC (rev 35714)
@@ -60,7 +60,7 @@
             public int compare(R left, R right) {
                 for (SortableHeader<R, ?> sortableHeader : sortOrderList) {
                     Comparable leftValue = sortableHeader.getColumn().getValue(left);
-                    Comparable rightValue = sortableHeader.getColumn().getValue(left);
+                    Comparable rightValue = sortableHeader.getColumn().getValue(right);
                     int comparison = (leftValue == rightValue) ? 0
                             : (leftValue == null) ? -1
                             : (rightValue == null) ? 1
@@ -71,6 +71,7 @@
                                 break;
                             case DESCENDING:
                                 comparison = -comparison;
+                                break;
                             default:
                                 throw new IllegalStateException("Sorting can only be enabled for ASCENDING or" +
                                         " DESCENDING, not sortDirection (" + sortableHeader.getSortDirection() + ") .");