Fix bug #47147 - XWPF table cells adding extra paragraph - test from Stefan Stern

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

2 files changed, 28 insertions, 1 deletions

diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java
index ec981cad36..d310b288ae 100644
--- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java
+++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java
@@ -51,7 +51,6 @@ public class XWPFTable implements IBodyElement{
         this(table, part);
         for (int i = 0; i < row; i++) {
             XWPFTableRow tabRow = (getRow(i) == null) ? createRow() : getRow(i);
-            tableRows.add(tabRow);
             for (int k = 0; k < col; k++) {
                 XWPFTableCell tabCell = (tabRow.getCell(k) == null) ? tabRow
                         .createCell() : null;
diff --git a/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFTable.java b/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFTable.java
index 6d9a0e93b3..a8eaf1966c 100644
--- a/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFTable.java
+++ b/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFTable.java
@@ -17,9 +17,11 @@
 package org.apache.poi.xwpf.usermodel;
 
 import java.math.BigInteger;
+import java.util.List;
 
 import junit.framework.TestCase;
 
+import org.apache.poi.xwpf.XWPFTestDataSamples;
 import org.openxmlformats.schemas.wordprocessingml.x2006.main.CTP;
 import org.openxmlformats.schemas.wordprocessingml.x2006.main.CTR;
 import org.openxmlformats.schemas.wordprocessingml.x2006.main.CTRow;
@@ -130,4 +132,30 @@ public class TestXWPFTable extends TestCase {
         assertEquals(20, row.getHeight());
     }
 
+    public void testCreateTable() throws Exception {
+       // open an empty document
+       XWPFDocument doc = XWPFTestDataSamples.openSampleDocument("sample.docx");
+
+       // create a table with 5 rows and 7 coloumns
+       int noRows = 5; 
+       int noCols = 7;
+       XWPFTable table = doc.createTable(noRows,noCols);
+
+       // assert the table is empty
+       List<XWPFTableRow> rows = table.getRows();
+       assertEquals("Table has less rows than requested.", noRows, rows.size());
+       for (XWPFTableRow xwpfRow : rows)
+       {
+          assertNotNull(xwpfRow);
+          for (int i = 0 ; i < 7 ; i++)
+          {
+             XWPFTableCell xwpfCell = xwpfRow.getCell(i);
+             assertNotNull(xwpfCell);
+             assertEquals("Empty cells should not have one paragraph.",1,xwpfCell.getParagraphs().size());
+             xwpfCell = xwpfRow.getCell(i);
+             assertEquals("Calling 'getCell' must not modify cells content.",1,xwpfCell.getParagraphs().size());
+          }
+       }
+       doc.getPackage().revert();
+    }
 }
\ No newline at end of file