[ http://jira.codehaus.org/browse/DISPL-77?page=history ] andy pruitt closed DISPL-77: ----------------------------
Fix Version: 1.1 Resolution: Fixed Thanks for the comments, this is now fixed. > Account for extended ascii strings when sorting > ----------------------------------------------- > > Key: DISPL-77 > URL: http://jira.codehaus.org/browse/DISPL-77 > Project: DisplayTag > Type: Bug > Components: Paging/Sorting > Versions: 1.0 RC1 > Reporter: fabrizio giustina > Assignee: fabrizio giustina > Fix For: 1.1 > > > ==== > imported from sf tracker > id 949485 > submitted by karlhungus - karlhungus > http://sourceforge.net/support/tracker.php?aid=949485 > ==== > Feature: take extended ascii into account when sorting > strings > Solutions: > -Option 1 > in RowSorter.checkNullsAndCompare > change the last else condition: > <code> // if object are not null and don't > implement comparable, compare using string values > returnValue = > object1.toString().compareTo(object2.toString());</code> > to: > <code>Collator.getInstance().compare(object1.toString(), > object2.toString());</code> > It's simple and it basicly solves the problem > -Option 2 > When creating the TableModel in TableTag pass in the > locale and make it an attribute of RowSorter, then > <code>Collator.getInstance(locale).compare(object1.toString(), > object2.toString());</code> > It's a little more impacting but it sovles the problem > more generally -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - If you want more information on JIRA, or have a bug to report see: http://www.atlassian.com/software/jira ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click _______________________________________________ displaytag-devel mailing list displaytag-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/displaytag-devel