[ http://jira.codehaus.org/browse/DISPL-243?page=all ]
     
fabrizio giustina closed DISPL-243:
-----------------------------------

    Resolution: Fixed

Thanks, I added a unit test (yes, the problem can be reproduced, sort is wrong 
after the 2nd click) and fixed it in HEAD for 1.1.

Btw, tests in CVS for 1.1 should run without problems since a couple of weeks, 
see: 
http://displaytag.sourceforge.net/11/multiproject/displaytag/junit-report.html

> Default column sort breaks sorting after a few sorts of the column
> ------------------------------------------------------------------
>
>          Key: DISPL-243
>          URL: http://jira.codehaus.org/browse/DISPL-243
>      Project: DisplayTag
>         Type: Bug
>   Components: Paging/Sorting
>     Versions: 1.1
>      Fix For: 1.1
>  Attachments: TEST-org.displaytag.test.TestAll.txt, 
> TEST-org.displaytag.test.TestAll.xml, TableTagPatch
>
>
> When sorting columns that have been configured with a default sort order, 
> after a few sorts the column stops sorting and remains sorted in the opposite 
> direction configured.  So, for example, if the column is configured to first 
> sort descending, after a few sorts it stops sorting and remains sorted in the 
> ascending direction.  This bug only occurs with columns that have been 
> configured with a default sort order.
> Attached you'll find a patch that fixes this issue, a couple of line changes 
> in TableTag.java.
> BTW, as I'm sure you know, the current org.displaytag.test.TestAll in cvs 
> fails.   

-- 
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
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to