Community
Participate
Working Groups
Build Identifier: M20110210-1200 Our product uses Remote System Details view and blank cells may be included on the view in our case. When I sort resources on this view by the "Created" header that seems to contain Date class object, blank cells and other cells are partially sorted in both ascending and descending case, instead of sorting all records by the date labels. Other header, for example "Size", causes same issue, if some blank cells are included. The sort logic of this view is included in SystemTableViewSorter of RSE. In this class, the "compare" method compares two arguments those are treated as the instances of the same class. In our case that the one object is a instance of Date class and the other is null, NPE occurs in this "compare" and zero is returned so that sorting result is not correct. Please add null check into the "compare" method. Reproducible: Always Steps to Reproduce: 1. show resources in Remote System Details view. Some resources should have empty cells those belong to Integer, Date or some other non-String classes. 2. sort by non-String columns header that includes some blank cells.
Created attachment 202536 [details] patch to check for null
Kevin and Xuan, could you please review this patch?
Review +.
The fix looks good. Thanks.
Thanks for the reviews. I've committed the change to the HEAD stream.