The following comment has been added to this issue: Author: Created: Thu, 24 Feb 2005 10:06 AM Body: I have applied the patch and it seems to be working correctly, but if I'm not wrong it throws a NullPointerException if you haven't declared a decorator for the table. I have traced the error back to the Row class. This sentence on line 151 is causing the exception: addStyle = this.tableModel.getTableDecorator().addRowClass();
Prepending this line with a null check for tableModel.getTableDecorator() corrects the bug. --------------------------------------------------------------------- View this comment: http://jira.codehaus.org/browse/DISPL-192?page=comments#action_30265 --------------------------------------------------------------------- View the issue: http://jira.codehaus.org/browse/DISPL-192 Here is an overview of the issue: --------------------------------------------------------------------- Key: DISPL-192 Summary: add row style to current row from TableDecorator Type: New Feature Status: Unassigned Priority: Major Original Estimate: 1 hour Time Spent: Unknown Remaining: 1 hour Project: DisplayTag Components: Build/distribution Versions: 1.1 Assignee: Reporter: Justin James Created: Wed, 16 Feb 2005 5:37 PM Updated: Thu, 24 Feb 2005 10:06 AM Description: Add a call back method on TableDecorator called addRowStyle(). This allows a row style to be added based on inspection of data for the current row. --------------------------------------------------------------------- JIRA INFORMATION: 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