#363817 by thePanz: Empty fields in table could still trigger separators.

diff --git a/CHANGELOG.txt b/CHANGELOG.txt
index b09b701..f7b7e36 100644 (file)
--- a/CHANGELOG.txt
+++ b/CHANGELOG.txt
@@ -1,7 +1,10 @@
 CHANGELOG for Views 2 for Drupal 6
 Views 2.5
+  Bugs fixed:
+    o #363817 by thePanz: Empty fields in table could still trigger separators.
   Other changes:
     o #379382 by neochief: Add option to strip tags during advanced rendering.
+
 Views 2.4
   Bugs fixed:
     o #371466 by dereine: Fix incorrect link to comments.

diff --git a/theme/theme.inc b/theme/theme.inc
index 6e5a9b1..663705c 100644 (file)
--- a/theme/theme.inc
+++ b/theme/theme.inc
@@ -346,12 +346,12 @@ function template_preprocess_views_view_table(&$vars) {
         $field_output = $renders[$num][$field];
 
         // Don't bother with separators and stuff if the field does not show up.
-        if (!isset($field_output) && isset($vars['rows'][$num][$column])) {
+        if (empty($field_output) && !empty($vars['rows'][$num][$column])) {
           continue;
         }
 
         // Place the field into the column, along with an optional separator.
-        if (isset($vars['rows'][$num][$column])) {
+        if (!empty($vars['rows'][$num][$column])) {
           if (!empty($options['info'][$column]['separator'])) {
             $vars['rows'][$num][$column] .= filter_xss_admin($options['info'][$column]['separator']);
           }