[ 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

Reply via email to