theme_table() contains the following comment:

    // HTML requires that the thead tag has tr tags in it follwed by tbody
    // tags. Using ternary operator to check and see if we have any rows.

See follwed.

CommentFileSizeAuthor
#2 656078-D6.patch751 bytesjhodgdon

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue tags: +Novice

http://api.drupal.org/api/function/theme_table

Good catch. This is in the function's body, not in the doc header, and it's been fixed already in Drupal 7, by the way.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new 656078-D6.patch751 bytes

Here's a (trivial) patch.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Novice

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.