[ 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel
