getListIndex() does not return the real list index

Tom Tibbetts added a comment - 01/Jul/05 9:15 AM I found that the Row object contains the original index of the unsorted list in the form of rowNumber. When it is passed to the table decorator via table decorator's init function, rowNumber gets reset to the sorted index. I've done a search on clients to Row.getRowNumber and found no users, so I went ahead and removed the line that resets the rowNumber to the sorted index and that seems to work.

Tom Tibbetts added a comment - 01/Jul/05 10:35 AM in RowIterator.java replace the code for the next() method with the following code:     public Row next()     {         int currentRowNumber = this.rowNumber++;         if (log.isDebugEnabled())         {             log.debug("[" + this.id + "] rowIterator.next() row number=" + currentRowNumber);         }         Object object = this.iterator.next();         Row row = (Row) object;         int listIndex = row.getRowNumber() - 1;         row.setRowNumber(currentRowNumber);         if (this.decorator != null)         {             this.decorator.initRow(row.getObject(), currentRowNumber, listIndex);         }         return row;     }

fabrizio giustina added a comment - 29/Dec/05 5:36 PM I reviewed the patch supplied, but the value returned by getListIndex() will not be the correct one. Actually, there is no way to get the correct index in the original, unsorter list: doing this will require iterating throug all the list for each object to get its original position (and still it will not work for duplicate objects). getListIndex() returns the list in the FULL list, but after sorting (so it's the same as the view index + page offset). I added a note about this to the TableDecorator javadocs. Closing since it can't be fixed.