Inconsistent default value treatment for HeaderCell.group
---------------------------------------------------------

                 Key: DISPL-378
                 URL: http://jira.codehaus.org/browse/DISPL-378
             Project: DisplayTag
          Issue Type: Bug
          Components: HTML Generation
    Affects Versions: 1.1
            Reporter: erwin de ley
            Priority: Minor


HeaderCell.group has a default value of 0. 

Case 1: For a Table containing Columns defined with the Column tag, the 
HeaderCell.group values are set to -1, by the ColumnTag.addHeaderToTable() 
method, snice the ColumnTag.group is -1 by default.

Case 2 : For a Table without Column definitions in the jsp, the 
TableTag.describeEmptyTable() leaves the HeaderCell.group at 0.

The documentation states that the grouping requires a value > 0 in order to be 
activated. So both cases above should result in no grouping.

But the TableWriterTemplate.writeTableBody() method, at line 359, checks on 
value -1! So grouping is active from group==0, and the 2 cases above behave 
differently!

Would be nicer if ColumnTag and HeaderCell have same default values, and 
TableWriterTemplate checks correctly on the common default value...

cheers

erwin

-- 
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

        

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to