Bug 31353 - Datagrid: Hyperlinks not re-ordered when values re-ordered
Summary: Datagrid: Hyperlinks not re-ordered when values re-ordered
Status: CLOSED FIXED
Alias: None
Product: Taglibs
Classification: Unclassified
Component: Sandbox Taglibs (show other bugs)
Version: 1.0
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-09-22 00:06 UTC by Bruce Yu
Modified: 2005-03-20 17:06 UTC (History)
0 users


Attachments
Patch for bug 31353 (134.33 KB, patch)
2004-09-23 19:03 UTC, Nestor Boscan 		Details | Diff
New Patch (124.18 KB, patch)
2004-09-24 05:47 UTC, Nestor Boscan 		Details | Diff
New Source File (441 bytes, text/plain)
2004-09-24 05:47 UTC, Nestor Boscan 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Bruce Yu 2004-09-22 00:06:52 UTC 
When the datagrid is sorted by one of the columns and redisplayed, the column 
values are sorted correctly, but hyperlinks remain in the same order as first 
displayed.

To repro use example6.jsp --

1. load example6.jsp of examples war into browser
2. Note current order is (Peter Jackson, John Woo, George Lucas, Michael Kain) 
and the links go to the correct page in the IMDB.com site
3. Reorder page by clicking the Name header. 
4. Order is now (Charlize Theron, Edward Norton, Gary Oldman, George Lucas); 
however, hyperlinks still go to pages in order (Peter Jackson, John Woo, George 
Lucas, Michael Kain)
Comment 1 Nestor Boscan 2004-09-23 19:03:33 UTC 
Created attachment 12850 [details]
Patch for bug 31353
Comment 2 Felipe Leme 2004-09-24 00:18:17 UTC 
Nestor,

The patch didn't get through, I thing you did something wrong.

Please try the following in your machine:

- go to the sandbox/datagrid directory
- run:

 cvs diff -u > patch.31353

- then add patch.31353 as attachment here

-- Felipe

PS: notice that if you added a new file, it won't be in the patch
Comment 3 Nestor Boscan 2004-09-24 05:47:00 UTC 
Created attachment 12853 [details]
New Patch
Comment 4 Nestor Boscan 2004-09-24 05:47:24 UTC 
Created attachment 12854 [details]
New Source File
Comment 5 Nestor Boscan 2004-09-24 05:48:00 UTC 
Uploaded new patch and a new source file.
Comment 6 Felipe Leme 2004-09-26 00:07:15 UTC 
Applied Nestor patch, so I'm marking it as fixed.
Comment 7 Bruce Yu 2004-09-27 18:16:41 UTC 
Thanks for the quick fix!