Bug 25809 (Patch by Finn Bock) NPE for incorrectly created tables fixed.

Also changed text of warning message.

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@197149 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/java/org/apache/fop/layoutmgr/AddLMVisitor.java b/src/java/org/apache/fop/layoutmgr/AddLMVisitor.java
index c502d71b7fe05e7223d80d1dbce52ff05f928356..be4567ce00a7a3692c3b7cc48cf1dac46a269047 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/AddLMVisitor.java
+++ b/src/java/org/apache/fop/layoutmgr/AddLMVisitor.java
@@ -842,12 +842,15 @@ public class AddLMVisitor implements FOTreeVisitor {
          TableLayoutManager tlm = new TableLayoutManager();
          tlm.setUserAgent(node.getUserAgent());
          tlm.setFObj(node);
-         ArrayList columnLMs = new ArrayList();
-         ListIterator iter = node.getColumns().listIterator();
-         while (iter.hasNext()) {
-             columnLMs.add(getTableColumnLayoutManager((TableColumn)iter.next()));
+         ArrayList columns = node.getColumns();
+         if (columns != null) {
+             ArrayList columnLMs = new ArrayList();
+             ListIterator iter = columns.listIterator();
+             while (iter.hasNext()) {
+                 columnLMs.add(getTableColumnLayoutManager((TableColumn)iter.next()));
+             }
+             tlm.setColumns(columnLMs);
          }
-         tlm.setColumns(columnLMs);
          if (node.getTableHeader() != null) {
              tlm.setTableHeader(getTableBodyLayoutManager(node.getTableHeader()));
          }

diff --git a/src/java/org/apache/fop/layoutmgr/table/Body.java b/src/java/org/apache/fop/layoutmgr/table/Body.java
index 5efbd7514e10f0091d929d7e2742d2fce2576a56..622664ffe6802d94763ec819aa2a6e9cfd9357de 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/table/Body.java
+++ b/src/java/org/apache/fop/layoutmgr/table/Body.java
@@ -128,7 +128,7 @@ public class Body extends BlockStackingLayoutManager {
 
         if (columns == null) {
             setFinished(true);
-            getLogger().warn("ignoring table body with undefined columns");
+            getLogger().warn("ignoring fo:table-body with undefined fo:table-columns");
             return null;
         }