Bugs item #814058, was opened at 2003-09-28 19:37
Message generated for change (Comment added) made by fgiust
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: fabrizio giustina (fgiust)
Date: 2003-10-26 22: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 is sponsored by: The SF.net Donation Program.
Do you like what SourceForge.net is doing for the Open
Source Community?  Make a contribution, and help us add new
features and functionality. Click here: http://sourceforge.net/donate/
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to