Posted by kkaefer on February 20, 2010 at 11:56am
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | theme system |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | Needs tests |
Issue Summary
When calling theme_table with an empty rows array but with an 'empty' text set, it will automatically create a row for that empty text. However, the header is already generated before that row is added and thus doesn't use thead because it assumes there is no tbody. The attached patch just reverses the order so that the header properly uses thead when there is an empty row.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| table_thead.patch | 861 bytes | Idle | PASSED: [[SimpleTest]]: [MySQL] 17,908 pass(es). | View details |
Comments
#1
Makes sense. But let's make sure it stays fixed by having a test.
#2
Wrote a small test for this
#3
Thanks!
#4
Committed to CVS HEAD. Thanks.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.