- Patch #153535 by traxer: according to the XHTML1 Strict specification, it is not allowed to generate empty <tr></tr> tags.

1 files changed, 17 insertions, 16 deletions

diff --git a/includes/theme.inc b/includes/theme.inc
index 367befade..950d8f293 100644
--- a/includes/theme.inc
+++ b/includes/theme.inc
@@ -1153,24 +1153,25 @@ function theme_table($header, $rows, $attributes = array(), $caption = NULL) {
       else {
         $cells = $row;
       }
+      if (count($cells)) {
+        // Add odd/even class
+        $class = $flip[$class];
+        if (isset($attributes['class'])) {
+          $attributes['class'] .= ' '. $class;
+        }
+        else {
+          $attributes['class'] = $class;
+        }
 
-      // Add odd/even class
-      $class = $flip[$class];
-      if (isset($attributes['class'])) {
-        $attributes['class'] .= ' '. $class;
-      }
-      else {
-        $attributes['class'] = $class;
-      }
-
-      // Build row
-      $output .= ' <tr'. drupal_attributes($attributes) .'>';
-      $i = 0;
-      foreach ($cells as $cell) {
-        $cell = tablesort_cell($cell, $header, $ts, $i++);
-        $output .= _theme_table_cell($cell);
+        // Build row
+        $output .= ' <tr'. drupal_attributes($attributes) .'>';
+        $i = 0;
+        foreach ($cells as $cell) {
+          $cell = tablesort_cell($cell, $header, $ts, $i++);
+          $output .= _theme_table_cell($cell);
+        }
+        $output .= " </tr>\n";
       }
-      $output .= " </tr>\n";
     }
   }