Bugs item #814058, was opened at 2003-09-28 10:37
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=814058&group_id=73068

Category: main tag library
Group: v 1.0-b1
Status: Closed
Resolution: None
Priority: 8
Submitted By: Jim Moriarty (jimmy_moriarty)
Assigned to: fabrizio giustina (fgiust)
Summary: Sorting with table decorators fails (fix included)

Initial Comment:
When a table decorator has been declared, sorting fails
for columns that are decorated.

Problem occurs as the row objects are not set before
they are compared in model/RowSorter.java.

Need to change (circa line 110):
if (mTableDecorator != null &&
mTableDecorator.hasGetterFor(mProperty))
  {
    lResult1 =
LookupUtil.getBeanProperty(mTableDecorator, mProperty);

    lResult2 =
LookupUtil.getBeanProperty(mTableDecorator, mProperty);
  }
to

if (mTableDecorator != null &&
mTableDecorator.hasGetterFor(mProperty))
  {
    // jmm 26 sept 03 - need to set the row before
sending to the decorator
    ((TableDecorator)mTableDecorator).initRow ( lObj1,
0, 0 );
    lResult1 =
LookupUtil.getBeanProperty(mTableDecorator, mProperty);

    // jmm 26 sept 03 - need to set the row before
sending to the decorator
    ((TableDecorator)mTableDecorator).initRow ( lObj2,
0, 0 );
    lResult2 =
LookupUtil.getBeanProperty(mTableDecorator, mProperty);
  }

or something similar

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2004-06-21 08:12

Message:
Logged In: NO 

Sorting decorated colums is nonsense. If, for example, I 
decorate a colums to show it in red colour, why should I sort 
also the HTML code used to colour the text?

----------------------------------------------------------------------

Comment By: fabrizio giustina (fgiust)
Date: 2003-10-26 13:13

Message:
Logged In: YES 
user_id=798060

patch added for 1-b2, thanks

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=814058&group_id=73068


-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - 
digital self defense, top technical experts, no vendor pitches, 
unmatched networking opportunities. Visit www.blackhat.com
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to