#64981 - Upgrade OOXML schema to 5th edition (transitional)

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

diff --git a/maven/poi-ooxml-full.pom b/maven/poi-ooxml-full.pom
index d35d8f7ddc0d3b00fab077bbd098ad1434185e1d..76d38e799ec5779424c0e691842819f5bde285e8 100644 (file)
--- a/maven/poi-ooxml-full.pom
+++ b/maven/poi-ooxml-full.pom
@@ -28,7 +28,7 @@
   <version>@VERSION@</version>
   <packaging>jar</packaging>
   <name>Apache POI - OOXML schemas (full)</name>
-  <description>XmlBeans generated from the Ecma supplied xsds (since POI 5.0.0, the 3rd edition is used):
+  <description>XmlBeans generated from the Ecma supplied xsds (since POI 5.0.0, the 5th edition is used):
     https://www.ecma-international.org/publications/standards/Ecma-376.htm</description>
   <url>https://poi.apache.org/</url>
 

diff --git a/src/examples/src/org/apache/poi/examples/xslf/LinkVideoToPptx.java b/src/examples/src/org/apache/poi/examples/xslf/LinkVideoToPptx.java
index db7f45301993fdb929c8fa150317a935e7558a47..d2988478d289b248eb86dc409d2919100d9e04df 100644 (file)
--- a/src/examples/src/org/apache/poi/examples/xslf/LinkVideoToPptx.java
+++ b/src/examples/src/org/apache/poi/examples/xslf/LinkVideoToPptx.java
@@ -107,7 +107,7 @@ public class LinkVideoToPptx {
         ctn.setFill(STTLTimeNodeFillType.HOLD);
         ctn.setDisplay(false);
         ctn.addNewStCondLst().addNewCond().setDelay(STTLTimeIndefinite.INDEFINITE);
-        cmedia.addNewTgtEl().addNewSpTgt().setSpid(""+pic1.getShapeId());
+        cmedia.addNewTgtEl().addNewSpTgt().setSpid(pic1.getShapeId());
 
 
         try (FileOutputStream fos = new FileOutputStream("mp4test/mp4test-poi.pptx")) {

diff --git a/src/examples/src/org/apache/poi/examples/xssf/usermodel/BarChart.java b/src/examples/src/org/apache/poi/examples/xssf/usermodel/BarChart.java
index 2a3c6a761bc920cba5c43f1620e8bc126202b478..8c68fe87a71e3cca0177ad0cc15b1c0f14c49664 100644 (file)
--- a/src/examples/src/org/apache/poi/examples/xssf/usermodel/BarChart.java
+++ b/src/examples/src/org/apache/poi/examples/xssf/usermodel/BarChart.java
@@ -113,7 +113,7 @@ public final class BarChart {
 
     private static void solidFillSeries(XDDFChartData data, int index, PresetColor color) {
         XDDFSolidFillProperties fill = new XDDFSolidFillProperties(XDDFColor.from(color));
-        XDDFChartData.Series series = data.getSeries().get(index);
+        XDDFChartData.Series series = data.getSeries(index);
         XDDFShapeProperties properties = series.getShapeProperties();
         if (properties == null) {
             properties = new XDDFShapeProperties();

diff --git a/src/examples/src/org/apache/poi/examples/xssf/usermodel/LineChart.java b/src/examples/src/org/apache/poi/examples/xssf/usermodel/LineChart.java
index 268d632a0411a22ec708201ea570ab7298f2b575..724d3e26089a3f9c5e23a105e933446112d5a4b9 100644 (file)
--- a/src/examples/src/org/apache/poi/examples/xssf/usermodel/LineChart.java
+++ b/src/examples/src/org/apache/poi/examples/xssf/usermodel/LineChart.java
@@ -117,7 +117,7 @@ public final class LineChart {
         XDDFSolidFillProperties fill = new XDDFSolidFillProperties(XDDFColor.from(color));
         XDDFLineProperties line = new XDDFLineProperties();
         line.setFillProperties(fill);
-        XDDFChartData.Series series = data.getSeries().get(index);
+        XDDFChartData.Series series = data.getSeries(index);
         XDDFShapeProperties properties = series.getShapeProperties();
         if (properties == null) {
             properties = new XDDFShapeProperties();

diff --git a/src/examples/src/org/apache/poi/examples/xssf/usermodel/ScatterChart.java b/src/examples/src/org/apache/poi/examples/xssf/usermodel/ScatterChart.java
index 6946126a39ed41f6cea99f72fc4fa41743aab3d6..a5ce3c84ae9bf58d45dfdfa024c15d4d99a59ad3 100644 (file)
--- a/src/examples/src/org/apache/poi/examples/xssf/usermodel/ScatterChart.java
+++ b/src/examples/src/org/apache/poi/examples/xssf/usermodel/ScatterChart.java
@@ -110,7 +110,7 @@ public final class ScatterChart {
         XDDFSolidFillProperties fill = new XDDFSolidFillProperties(XDDFColor.from(color));
         XDDFLineProperties line = new XDDFLineProperties();
         line.setFillProperties(fill);
-        XDDFChartData.Series series = data.getSeries().get(index);
+        XDDFChartData.Series series = data.getSeries(index);
         XDDFShapeProperties properties = series.getShapeProperties();
         if (properties == null) {
             properties = new XDDFShapeProperties();

diff --git a/src/multimodule/ooxml-full/java9/module-info.class b/src/multimodule/ooxml-full/java9/module-info.class
index 2bbf9820254072b1f27df4dc892fbca7c7bba2a0..4c1707b28d91d5f3b5651e921f243505919e7900 100644 (file)
Binary files a/src/multimodule/ooxml-full/java9/module-info.class and b/src/multimodule/ooxml-full/java9/module-info.class differ

diff --git a/src/multimodule/ooxml-full/java9/module-info.java b/src/multimodule/ooxml-full/java9/module-info.java
index 102ab6ad69486f3813dbc9e435a953c8780f5b05..4587ea5e5dd85c3be064448d3a5bccf2924b3253 100644 (file)
--- a/src/multimodule/ooxml-full/java9/module-info.java
+++ b/src/multimodule/ooxml-full/java9/module-info.java
@@ -30,7 +30,6 @@ open module org.apache.poi.ooxml.schemas {
     exports com.microsoft.schemas.vml;
     exports org.openxmlformats.schemas.drawingml.x2006.chart;
     exports org.openxmlformats.schemas.drawingml.x2006.chartDrawing;
-    exports org.openxmlformats.schemas.drawingml.x2006.compatibility;
     exports org.openxmlformats.schemas.drawingml.x2006.diagram;
     exports org.openxmlformats.schemas.drawingml.x2006.lockedCanvas;
     exports org.openxmlformats.schemas.drawingml.x2006.main;

diff --git a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java
index af5fa32e248ad93d36de13d972f9b5d7551fdccc..871b1a933c005e6aeb2432878f74431a78e20d17 100644 (file)
--- a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java
+++ b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCellStyle.java
@@ -592,7 +592,7 @@ public class XSSFCellStyle implements CellStyle, Duplicatable {
     @Override
     public short getRotation() {
         CTCellAlignment align = _cellXf.getAlignment();
-        return (short)(align == null ? 0 : align.getTextRotation());
+        return align == null ? 0 : align.getTextRotation().shortValue();
     }
 
     @Override

diff --git a/src/ooxml/java/org/apache/poi/xssf/usermodel/extensions/XSSFCellAlignment.java b/src/ooxml/java/org/apache/poi/xssf/usermodel/extensions/XSSFCellAlignment.java
index 9fed1a8cf0b81a4deecec75fae16322a3285975d..a291336493e9ae4f4720af1f56044c9a15e1e1a6 100644 (file)
--- a/src/ooxml/java/org/apache/poi/xssf/usermodel/extensions/XSSFCellAlignment.java
+++ b/src/ooxml/java/org/apache/poi/xssf/usermodel/extensions/XSSFCellAlignment.java
@@ -16,6 +16,8 @@
 ==================================================================== */
 package org.apache.poi.xssf.usermodel.extensions;
 
+import java.math.BigInteger;
+
 import org.apache.poi.ss.usermodel.HorizontalAlignment;
 import org.apache.poi.ss.usermodel.ReadingOrder;
 import org.apache.poi.ss.usermodel.VerticalAlignment;
@@ -29,7 +31,7 @@ import org.openxmlformats.schemas.spreadsheetml.x2006.main.STVerticalAlignment;
  * Cell settings available in the Format/Alignment tab
  */
 public class XSSFCellAlignment {
-    private CTCellAlignment cellAlignement;
+    private final CTCellAlignment cellAlignement;
 
     /**
      * Creates a Cell Alignment from the supplied XML definition
@@ -85,7 +87,7 @@ public class XSSFCellAlignment {
     public void setHorizontal(HorizontalAlignment align) {
         cellAlignement.setHorizontal(STHorizontalAlignment.Enum.forInt(align.ordinal() + 1));
     }
-    
+
     /**
      * Set the type of reading order for the cell
      *
@@ -95,7 +97,7 @@ public class XSSFCellAlignment {
     public void setReadingOrder(ReadingOrder order) {
         cellAlignement.setReadingOrder(order.getCode());
     }
-    
+
     /**
      * Get the reading order for the cell
      *
@@ -108,7 +110,7 @@ public class XSSFCellAlignment {
         }
         return ReadingOrder.CONTEXT;
     }
-    
+
     /**
      * Get the number of spaces to indent the text in the cell
      *
@@ -142,7 +144,7 @@ public class XSSFCellAlignment {
      * @return rotation degrees (between 0 and 180 degrees)
      */
     public long getTextRotation() {
-        return cellAlignement.getTextRotation();
+        return cellAlignement.isSetTextRotation() ? cellAlignement.getTextRotation().longValue() : 0;
     }
 
     /**
@@ -157,8 +159,8 @@ public class XSSFCellAlignment {
      * <code>[degrees below horizon] = 90 - textRotation.</code>
      * </p>
      *
-     * Note: HSSF uses values from -90 to 90 degrees, whereas XSSF 
-     * uses values from 0 to 180 degrees. The implementations of this method will map between these two value-ranges 
+     * Note: HSSF uses values from -90 to 90 degrees, whereas XSSF
+     * uses values from 0 to 180 degrees. The implementations of this method will map between these two value-ranges
      * accordingly, however the corresponding getter is returning values in the range mandated by the current type
      * of Excel file-format that this CellStyle is applied to.
      *
@@ -168,7 +170,7 @@ public class XSSFCellAlignment {
         if(rotation < 0 && rotation >= -90) {
             rotation = 90 + ((-1)*rotation);
         }
-        cellAlignement.setTextRotation(rotation);
+        cellAlignement.setTextRotation(BigInteger.valueOf(rotation));
     }
 
     /**
@@ -192,11 +194,11 @@ public class XSSFCellAlignment {
     public boolean getShrinkToFit() {
        return cellAlignement.getShrinkToFit();
     }
-    
+
     public void setShrinkToFit(boolean shrink) {
        cellAlignement.setShrinkToFit(shrink);
     }
-    
+
     /**
      * Access to low-level data
      */

diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFAbstractSDT.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFAbstractSDT.java
index 93b2cdba8a940a507b6c15310ee6b07c81570554..434d857f2cb2fc3ed65553a2015f5b583bf4ae21 100644 (file)
--- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFAbstractSDT.java
+++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFAbstractSDT.java
@@ -18,7 +18,6 @@ package org.apache.poi.xwpf.usermodel;
 
 import org.apache.poi.ooxml.POIXMLDocumentPart;
 import org.openxmlformats.schemas.wordprocessingml.x2006.main.CTSdtPr;
-import org.openxmlformats.schemas.wordprocessingml.x2006.main.CTString;
 
 /**
  * Experimental abstract class that is a base for XWPFSDT and XWPFSDTCell
@@ -33,23 +32,8 @@ public abstract class XWPFAbstractSDT implements ISDTContents {
     private final IBody part;
 
     public XWPFAbstractSDT(CTSdtPr pr, IBody part) {
-        if (pr == null) {
-            title = "";
-            tag = "";
-        } else {
-            CTString[] aliases = pr.getAliasArray();
-            if (aliases != null && aliases.length > 0) {
-                title = aliases[0].getVal();
-            } else {
-                title = "";
-            }
-            CTString[] tags = pr.getTagArray();
-            if (tags != null && tags.length > 0) {
-                tag = tags[0].getVal();
-            } else {
-                tag = "";
-            }
-        }
+        title = (pr != null && pr.isSetAlias()) ? pr.getAlias().getVal() : "";
+        tag = (pr != null && pr.isSetTag()) ? pr.getTag().getVal() : "";
         this.part = part;
 
     }

diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDefaultRunStyle.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDefaultRunStyle.java
index 8eb8f6c1e0077a564208afc3cfd0cf2f9d6600f8..24c235d0e0aea3e03e7b7c3f3bd63316e742bb3a 100644 (file)
--- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDefaultRunStyle.java
+++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDefaultRunStyle.java
@@ -66,8 +66,8 @@ public class XWPFDefaultRunStyle {
     }
 
     private BigDecimal getFontSizeAsBigDecimal(int scale) {
-        return (rpr != null && rpr.isSetSz())
-            ? BigDecimal.valueOf(Units.toPoints(POIXMLUnits.parseLength(rpr.getSz().xgetVal()))).divide(BigDecimal.valueOf(4), scale, RoundingMode.HALF_UP)
+        return (rpr != null && rpr.sizeOfSzArray() > 0)
+            ? BigDecimal.valueOf(Units.toPoints(POIXMLUnits.parseLength(rpr.getSzArray(0).xgetVal()))).divide(BigDecimal.valueOf(4), scale, RoundingMode.HALF_UP)
             : null;
     }
 }

diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java
index c7ba363119504abb5c738abf1e95ea6806a0f0b0..06df99be128131ca9abeec931da7375ade318d56 100644 (file)
--- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java
+++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java
@@ -76,10 +76,10 @@ import org.xml.sax.SAXException;
  * XWPFRun object defines a region of text with a common set of properties
  */
 public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
-    private CTR run;
-    private String pictureText;
-    private IRunBody parent;
-    private List<XWPFPicture> pictures;
+    private final CTR run;
+    private final String pictureText;
+    private final IRunBody parent;
+    private final List<XWPFPicture> pictures;
 
     /**
      * @param r the CTR bean which holds the run attributes
@@ -238,7 +238,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public String getLang() {
         CTRPr pr = getRunProperties(false);
-        Object lang = pr == null || !pr.isSetLang() ? null : pr.getLang().getVal();
+        Object lang = (pr == null || pr.sizeOfLangArray() == 0) ? null : pr.getLangArray(0).getVal();
         return (String) lang;
     }
 
@@ -250,7 +250,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setLang(String lang) {
         CTRPr pr = getRunProperties(true);
-        CTLanguage ctLang = pr.isSetLang() ? pr.getLang() : pr.addNewLang();
+        CTLanguage ctLang = pr.sizeOfLangArray() > 0 ? pr.getLangArray(0) : pr.addNewLang();
         ctLang.setVal(lang);
     }
 
@@ -263,7 +263,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public boolean isBold() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetB() && isCTOnOff(pr.getB());
+        return pr != null && pr.sizeOfBArray() > 0 && isCTOnOff(pr.getBArray(0));
     }
 
     /**
@@ -293,7 +293,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public void setBold(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff bold = pr.isSetB() ? pr.getB() : pr.addNewB();
+        CTOnOff bold = pr.sizeOfBArray() > 0 ? pr.getBArray(0) : pr.addNewB();
         bold.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
@@ -304,8 +304,8 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
         String color = null;
         if (run.isSetRPr()) {
             CTRPr pr = getRunProperties(false);
-            if (pr != null && pr.isSetColor()) {
-                CTColor clr = pr.getColor();
+            if (pr != null && pr.sizeOfColorArray() > 0) {
+                CTColor clr = pr.getColorArray(0);
                 color = clr.xgetVal().getStringValue();
             }
         }
@@ -319,7 +319,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setColor(String rgbStr) {
         CTRPr pr = getRunProperties(true);
-        CTColor color = pr.isSetColor() ? pr.getColor() : pr.addNewColor();
+        CTColor color = pr.sizeOfColorArray() > 0 ? pr.getColorArray(0) : pr.addNewColor();
         color.setVal(rgbStr);
     }
 
@@ -373,7 +373,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public boolean isItalic() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetI() && isCTOnOff(pr.getI());
+        return pr != null && pr.sizeOfIArray() > 0 && isCTOnOff(pr.getIArray(0));
     }
 
     /**
@@ -404,7 +404,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public void setItalic(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff italic = pr.isSetI() ? pr.getI() : pr.addNewI();
+        CTOnOff italic = pr.sizeOfIArray() > 0 ? pr.getIArray(0) : pr.addNewI();
         italic.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
@@ -452,11 +452,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     private CTUnderline getCTUnderline(boolean create) {
         CTRPr pr = getRunProperties(true);
-        CTUnderline underline = pr.getU();
-        if (create && underline == null) {
-            underline = pr.addNewU();
-        }
-        return underline;
+        return pr.sizeOfUArray() > 0 ? pr.getUArray(0) : (create ? pr.addNewU() : null);
     }
 
     /**
@@ -539,7 +535,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public boolean isStrikeThrough() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetStrike() && isCTOnOff(pr.getStrike());
+        return pr != null && pr.sizeOfStrikeArray() > 0 && isCTOnOff(pr.getStrikeArray(0));
     }
 
     /**
@@ -569,7 +565,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public void setStrikeThrough(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff strike = pr.isSetStrike() ? pr.getStrike() : pr.addNewStrike();
+        CTOnOff strike = pr.sizeOfStrikeArray() > 0 ? pr.getStrikeArray(0) : pr.addNewStrike();
         strike.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
@@ -592,7 +588,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public boolean isDoubleStrikeThrough() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetDstrike() && isCTOnOff(pr.getDstrike());
+        return pr != null && pr.sizeOfDstrikeArray() > 0 && isCTOnOff(pr.getDstrikeArray(0));
     }
 
     /**
@@ -604,72 +600,72 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public void setDoubleStrikethrough(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff dstrike = pr.isSetDstrike() ? pr.getDstrike() : pr.addNewDstrike();
+        CTOnOff dstrike = pr.sizeOfDstrikeArray() > 0 ? pr.getDstrikeArray(0) : pr.addNewDstrike();
         dstrike.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
     @Override
     public boolean isSmallCaps() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetSmallCaps() && isCTOnOff(pr.getSmallCaps());
+        return pr != null && pr.sizeOfSmallCapsArray() > 0 && isCTOnOff(pr.getSmallCapsArray(0));
     }
 
     @Override
     public void setSmallCaps(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff caps = pr.isSetSmallCaps() ? pr.getSmallCaps() : pr.addNewSmallCaps();
+        CTOnOff caps = pr.sizeOfSmallCapsArray() > 0 ? pr.getSmallCapsArray(0) : pr.addNewSmallCaps();
         caps.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
     @Override
     public boolean isCapitalized() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetCaps() && isCTOnOff(pr.getCaps());
+        return pr != null && pr.sizeOfCapsArray() > 0 && isCTOnOff(pr.getCapsArray(0));
     }
 
     @Override
     public void setCapitalized(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff caps = pr.isSetCaps() ? pr.getCaps() : pr.addNewCaps();
+        CTOnOff caps = pr.sizeOfCapsArray() > 0 ? pr.getCapsArray(0) : pr.addNewCaps();
         caps.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
     @Override
     public boolean isShadowed() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetShadow() && isCTOnOff(pr.getShadow());
+        return pr != null && pr.sizeOfShadowArray() > 0 && isCTOnOff(pr.getShadowArray(0));
     }
 
     @Override
     public void setShadow(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff shadow = pr.isSetShadow() ? pr.getShadow() : pr.addNewShadow();
+        CTOnOff shadow = pr.sizeOfShadowArray() > 0 ? pr.getShadowArray(0) : pr.addNewShadow();
         shadow.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
     @Override
     public boolean isImprinted() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetImprint() && isCTOnOff(pr.getImprint());
+        return pr != null && pr.sizeOfImprintArray() > 0 && isCTOnOff(pr.getImprintArray(0));
     }
 
     @Override
     public void setImprinted(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff imprinted = pr.isSetImprint() ? pr.getImprint() : pr.addNewImprint();
+        CTOnOff imprinted = pr.sizeOfImprintArray() > 0 ? pr.getImprintArray(0) : pr.addNewImprint();
         imprinted.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
     @Override
     public boolean isEmbossed() {
         CTRPr pr = getRunProperties(false);
-        return pr != null && pr.isSetEmboss() && isCTOnOff(pr.getEmboss());
+        return pr != null && pr.sizeOfEmbossArray() > 0 && isCTOnOff(pr.getEmbossArray(0));
     }
 
     @Override
     public void setEmbossed(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff emboss = pr.isSetEmboss() ? pr.getEmboss() : pr.addNewEmboss();
+        CTOnOff emboss = pr.sizeOfEmbossArray() > 0 ? pr.getEmbossArray(0) : pr.addNewEmboss();
         emboss.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
@@ -691,33 +687,33 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setSubscript(VerticalAlign valign) {
         CTRPr pr = getRunProperties(true);
-        CTVerticalAlignRun ctValign = pr.isSetVertAlign() ? pr.getVertAlign() : pr.addNewVertAlign();
+        CTVerticalAlignRun ctValign = pr.sizeOfVertAlignArray() > 0 ? pr.getVertAlignArray(0) : pr.addNewVertAlign();
         ctValign.setVal(STVerticalAlignRun.Enum.forInt(valign.getValue()));
     }
 
     @Override
     public int getKerning() {
         CTRPr pr = getRunProperties(false);
-        if (pr == null || !pr.isSetKern()) {
+        if (pr == null || pr.sizeOfKernArray() == 0) {
             return 0;
         }
-        return (int)POIXMLUnits.parseLength(pr.getKern().xgetVal());
+        return (int)POIXMLUnits.parseLength(pr.getKernArray(0).xgetVal());
     }
 
     @Override
     public void setKerning(int kern) {
         CTRPr pr = getRunProperties(true);
-        CTHpsMeasure kernmes = pr.isSetKern() ? pr.getKern() : pr.addNewKern();
+        CTHpsMeasure kernmes = pr.sizeOfKernArray() > 0 ? pr.getKernArray(0) : pr.addNewKern();
         kernmes.setVal(BigInteger.valueOf(kern));
     }
 
     @Override
     public boolean isHighlighted() {
         CTRPr pr = getRunProperties(false);
-        if (pr == null || !pr.isSetHighlight()) {
+        if (pr == null || pr.sizeOfHighlightArray() == 0) {
             return false;
         }
-        STHighlightColor.Enum val = pr.getHighlight().getVal();
+        STHighlightColor.Enum val = pr.getHighlightArray(0).getVal();
         if (val == null || val == STHighlightColor.NONE) {
             return false;
         }
@@ -729,16 +725,16 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     @Override
     public int getCharacterSpacing() {
         CTRPr pr = getRunProperties(false);
-        if (pr == null || !pr.isSetSpacing()) {
+        if (pr == null || pr.sizeOfSpacingArray() == 0) {
             return 0;
         }
-        return (int)Units.toDXA(POIXMLUnits.parseLength(pr.getSpacing().xgetVal()));
+        return (int)Units.toDXA(POIXMLUnits.parseLength(pr.getSpacingArray(0).xgetVal()));
     }
 
     @Override
     public void setCharacterSpacing(int twips) {
         CTRPr pr = getRunProperties(true);
-        CTSignedTwipsMeasure spc = pr.isSetSpacing() ? pr.getSpacing() : pr.addNewSpacing();
+        CTSignedTwipsMeasure spc = pr.sizeOfSpacingArray() > 0 ? pr.getSpacingArray(0) : pr.addNewSpacing();
         spc.setVal(BigInteger.valueOf(twips));
     }
 
@@ -784,11 +780,11 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public String getFontFamily(FontCharRange fcr) {
         CTRPr pr = getRunProperties(false);
-        if (pr == null || !pr.isSetRFonts()) {
+        if (pr == null || pr.sizeOfRFontsArray() == 0) {
             return null;
         }
 
-        CTFonts fonts = pr.getRFonts();
+        CTFonts fonts = pr.getRFontsArray(0);
         switch (fcr == null ? FontCharRange.ascii : fcr) {
             default:
             case ascii:
@@ -813,7 +809,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setFontFamily(String fontFamily, FontCharRange fcr) {
         CTRPr pr = getRunProperties(true);
-        CTFonts fonts = pr.isSetRFonts() ? pr.getRFonts() : pr.addNewRFonts();
+        CTFonts fonts = pr.sizeOfRFontsArray() > 0 ? pr.getRFontsArray(0) : pr.addNewRFonts();
 
         if (fcr == null) {
             fonts.setAscii(fontFamily);
@@ -875,8 +871,8 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
 
     private BigDecimal getFontSizeAsBigDecimal(int scale) {
         CTRPr pr = getRunProperties(false);
-        return (pr != null && pr.isSetSz())
-            ? BigDecimal.valueOf(Units.toPoints(POIXMLUnits.parseLength(pr.getSz().xgetVal()))).divide(BigDecimal.valueOf(4), scale, RoundingMode.HALF_UP)
+        return (pr != null && pr.sizeOfSzArray() > 0)
+            ? BigDecimal.valueOf(Units.toPoints(POIXMLUnits.parseLength(pr.getSzArray(0).xgetVal()))).divide(BigDecimal.valueOf(4), scale, RoundingMode.HALF_UP)
             : null;
     }
 
@@ -897,7 +893,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     public void setFontSize(int size) {
         BigInteger bint = BigInteger.valueOf(size);
         CTRPr pr = getRunProperties(true);
-        CTHpsMeasure ctSize = pr.isSetSz() ? pr.getSz() : pr.addNewSz();
+        CTHpsMeasure ctSize = pr.sizeOfSzArray() > 0 ? pr.getSzArray(0) : pr.addNewSz();
         ctSize.setVal(bint.multiply(BigInteger.valueOf(2)));
     }
 
@@ -919,7 +915,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     public void setFontSize(double size) {
         BigDecimal bd = BigDecimal.valueOf(size);
         CTRPr pr = getRunProperties(true);
-        CTHpsMeasure ctSize = pr.isSetSz() ? pr.getSz() : pr.addNewSz();
+        CTHpsMeasure ctSize = pr.sizeOfSzArray() > 0 ? pr.getSzArray(0) : pr.addNewSz();
         ctSize.setVal(bd.multiply(BigDecimal.valueOf(2)).setScale(0, RoundingMode.HALF_UP).toBigInteger());
     }
 
@@ -933,7 +929,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public int getTextPosition() {
         CTRPr pr = getRunProperties(false);
-        return (pr != null && pr.isSetPosition()) ? (int)(Units.toPoints(POIXMLUnits.parseLength(pr.getPosition().xgetVal())) / 2.)
+        return (pr != null && pr.sizeOfPositionArray() > 0) ? (int)(Units.toPoints(POIXMLUnits.parseLength(pr.getPositionArray(0).xgetVal())) / 2.)
                 : -1;
     }
 
@@ -963,7 +959,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
     public void setTextPosition(int val) {
         BigInteger bint = new BigInteger(Integer.toString(val));
         CTRPr pr = getRunProperties(true);
-        CTSignedHpsMeasure position = pr.isSetPosition() ? pr.getPosition() : pr.addNewPosition();
+        CTSignedHpsMeasure position = pr.sizeOfPositionArray() > 0 ? pr.getPositionArray(0) : pr.addNewPosition();
         position.setVal(bint);
     }
 
@@ -1241,7 +1237,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
         if (null == pr) {
            pr = getCTR().addNewRPr();
         }
-        CTString style = pr.getRStyle() != null ? pr.getRStyle() : pr.addNewRStyle();
+        CTString style = pr.sizeOfRStyleArray() > 0 ? pr.getRStyleArray(0) : pr.addNewRStyle();
         style.setVal(styleId);
     }
 
@@ -1257,7 +1253,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
             return "";
         }
 
-        CTString style = pr.getRStyle();
+        CTString style = pr.getRStyleArray(0);
         if (style == null) {
             return "";
         }
@@ -1437,7 +1433,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setTextScale(int percentage) {
         CTRPr pr = getRunProperties(true);
-        CTTextScale scale = pr.isSetW() ? pr.getW() : pr.addNewW();
+        CTTextScale scale = pr.sizeOfWArray() > 0 ? pr.getWArray(0) : pr.addNewW();
         scale.setVal(percentage);
     }
 
@@ -1449,11 +1445,11 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public int getTextScale() {
         CTRPr pr = getRunProperties(false);
-        if (pr == null || !pr.isSetW()) {
+        if (pr == null || pr.sizeOfWArray() == 0) {
             return 100;
         }
 
-        int value = POIXMLUnits.parsePercent(pr.getW().xgetVal());
+        int value = POIXMLUnits.parsePercent(pr.getWArray(0).xgetVal());
         // 100% scaling, that is, no change. See 17.3.2.43 w (Expanded/Compressed Text)
         return value == 0 ? 100 : value / 1000;
     }
@@ -1466,7 +1462,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setTextHighlightColor(String colorName) {
         CTRPr pr = getRunProperties(true);
-        CTHighlight highlight = pr.isSetHighlight() ? pr.getHighlight() : pr.addNewHighlight();
+        CTHighlight highlight = pr.sizeOfHighlightArray() > 0 ? pr.getHighlightArray(0) : pr.addNewHighlight();
         STHighlightColor color = highlight.xgetVal();
         if (color == null) {
             color = STHighlightColor.Factory.newInstance();
@@ -1487,7 +1483,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public STHighlightColor.Enum getTextHightlightColor() {
         CTRPr pr = getRunProperties(true);
-        CTHighlight highlight = pr.isSetHighlight() ? pr.getHighlight() : pr.addNewHighlight();
+        CTHighlight highlight = pr.sizeOfHighlightArray() > 0 ? pr.getHighlightArray(0) : pr.addNewHighlight();
         STHighlightColor color = highlight.xgetVal();
         if (color == null) {
             color = STHighlightColor.Factory.newInstance();
@@ -1504,7 +1500,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public boolean isVanish() {
         CTRPr pr = getRunProperties(true);
-        return pr != null && pr.isSetVanish() && isCTOnOff(pr.getVanish());
+        return pr != null && pr.sizeOfVanishArray() > 0 && isCTOnOff(pr.getVanishArray(0));
     }
 
     /**
@@ -1515,7 +1511,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setVanish(boolean value) {
         CTRPr pr = getRunProperties(true);
-        CTOnOff vanish = pr.isSetVanish() ? pr.getVanish() : pr.addNewVanish();
+        CTOnOff vanish = pr.sizeOfVanishArray() > 0 ? pr.getVanishArray(0) : pr.addNewVanish();
         vanish.setVal(value ? STOnOff1.ON : STOnOff1.OFF);
     }
 
@@ -1527,7 +1523,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public STVerticalAlignRun.Enum getVerticalAlignment() {
         CTRPr pr = getRunProperties(true);
-        CTVerticalAlignRun vertAlign = pr.isSetVertAlign() ? pr.getVertAlign() : pr.addNewVertAlign();
+        CTVerticalAlignRun vertAlign = pr.sizeOfVertAlignArray() > 0 ? pr.getVertAlignArray(0) : pr.addNewVertAlign();
         STVerticalAlignRun.Enum val = vertAlign.getVal();
         if (val == null) {
             val = STVerticalAlignRun.BASELINE;
@@ -1543,7 +1539,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setVerticalAlignment(String verticalAlignment) {
         CTRPr pr = getRunProperties(true);
-        CTVerticalAlignRun vertAlign = pr.isSetVertAlign() ? pr.getVertAlign() : pr.addNewVertAlign();
+        CTVerticalAlignRun vertAlign = pr.sizeOfVertAlignArray() > 0 ? pr.getVertAlignArray(0) : pr.addNewVertAlign();
         STVerticalAlignRun align = vertAlign.xgetVal();
         if (align == null) {
             align = STVerticalAlignRun.Factory.newInstance();
@@ -1565,7 +1561,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public STEm.Enum getEmphasisMark() {
         CTRPr pr = getRunProperties(true);
-        CTEm emphasis = pr.isSetEm() ? pr.getEm() : pr.addNewEm();
+        CTEm emphasis = pr.sizeOfEmArray() > 0 ? pr.getEmArray(0) : pr.addNewEm();
 
         STEm.Enum val = emphasis.getVal();
         if (val == null) {
@@ -1583,7 +1579,7 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
      */
     public void setEmphasisMark(String markType) {
         CTRPr pr = getRunProperties(true);
-        CTEm emphasisMark = pr.isSetEm() ? pr.getEm() : pr.addNewEm();
+        CTEm emphasisMark = pr.sizeOfEmArray() > 0 ? pr.getEmArray(0) : pr.addNewEm();
         STEm mark = emphasisMark.xgetVal();
         if (mark == null) {
             mark = STEm.Factory.newInstance();

diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFStyles.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFStyles.java
index 75d3acc96dbc5536f1dbd36d8279dd791e6eb81b..1b1cf861547b9c78bcccfa2a469e05f12000c57e 100644 (file)
--- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFStyles.java
+++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFStyles.java
@@ -50,7 +50,7 @@ import org.openxmlformats.schemas.wordprocessingml.x2006.main.StylesDocument;
  */
 public class XWPFStyles extends POIXMLDocumentPart {
     private CTStyles ctStyles;
-    private List<XWPFStyle> listStyle = new ArrayList<>();
+    private final List<XWPFStyle> listStyle = new ArrayList<>();
 
     private XWPFLatentStyles latentStyles;
     private XWPFDefaultRunStyle defaultRunStyle;
@@ -234,8 +234,8 @@ public class XWPFStyles extends POIXMLDocumentPart {
         ensureDocDefaults();
 
         CTLanguage lang = null;
-        if (defaultRunStyle.getRPr().isSetLang()) {
-            lang = defaultRunStyle.getRPr().getLang();
+        if (defaultRunStyle.getRPr().sizeOfLangArray() > 0) {
+            lang = defaultRunStyle.getRPr().getLangArray(0);
         } else {
             lang = defaultRunStyle.getRPr().addNewLang();
         }
@@ -273,7 +273,10 @@ public class XWPFStyles extends POIXMLDocumentPart {
         ensureDocDefaults();
 
         CTRPr runProps = defaultRunStyle.getRPr();
-        runProps.setRFonts(fonts);
+        if (runProps.sizeOfRFontsArray() == 0) {
+            runProps.addNewRFonts();
+        }
+        runProps.setRFontsArray(0, fonts);
     }
 
     /**

diff --git a/src/ooxml/resources/org/apache/poi/schemas/OfficeOpenXML-XMLSchema-Transitional.zip b/src/ooxml/resources/org/apache/poi/schemas/OfficeOpenXML-XMLSchema-Transitional.zip
index 5a2e31dd99eebbc0a20fd1fbf8876d839b747368..d6dcf4fbb9571392738afde4164925379995e839 100644 (file)
Binary files a/src/ooxml/resources/org/apache/poi/schemas/OfficeOpenXML-XMLSchema-Transitional.zip and b/src/ooxml/resources/org/apache/poi/schemas/OfficeOpenXML-XMLSchema-Transitional.zip differ

diff --git a/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFExamples.java b/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFExamples.java
index 87d73c96ea89dd871ec5e5e6e467bcde2e51c096..dc951a540475d40d970cae1dc5e352fa179089db 100644 (file)
--- a/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFExamples.java
+++ b/src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFExamples.java
@@ -109,7 +109,7 @@ public class TestXSLFExamples {
             ctn.setFill(STTLTimeNodeFillType.HOLD);
             ctn.setDisplay(false);
             ctn.addNewStCondLst().addNewCond().setDelay(STTLTimeIndefinite.INDEFINITE);
-            cmedia.addNewTgtEl().addNewSpTgt().setSpid(""+pic1.getShapeId());
+            cmedia.addNewTgtEl().addNewSpTgt().setSpid(pic1.getShapeId());
 
 
             // write to file - use FileOutputStream instead

diff --git a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java
index 00e9ed802436fd021a05051fa09b119ab3d7a090..1b2bbdf8992a747ec503b1b4c6819cf7d4bab9b1 100644 (file)
--- a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java
+++ b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFCellStyle.java
@@ -30,7 +30,18 @@ import java.io.IOException;
 
 import org.apache.poi.hssf.usermodel.HSSFCellStyle;
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
-import org.apache.poi.ss.usermodel.*;
+import org.apache.poi.ss.usermodel.BorderStyle;
+import org.apache.poi.ss.usermodel.Cell;
+import org.apache.poi.ss.usermodel.CellStyle;
+import org.apache.poi.ss.usermodel.DataFormat;
+import org.apache.poi.ss.usermodel.FillPatternType;
+import org.apache.poi.ss.usermodel.HorizontalAlignment;
+import org.apache.poi.ss.usermodel.IndexedColors;
+import org.apache.poi.ss.usermodel.ReadingOrder;
+import org.apache.poi.ss.usermodel.Row;
+import org.apache.poi.ss.usermodel.Sheet;
+import org.apache.poi.ss.usermodel.VerticalAlignment;
+import org.apache.poi.ss.usermodel.Workbook;
 import org.apache.poi.xssf.XSSFTestDataSamples;
 import org.apache.poi.xssf.model.StylesTable;
 import org.apache.poi.xssf.usermodel.extensions.XSSFCellBorder;
@@ -716,7 +727,8 @@ public class TestXSSFCellStyle {
     @Test
     public void testGetSetVerticalAlignment() {
                assertEquals(VerticalAlignment.BOTTOM, cellStyle.getVerticalAlignment());
-               assertNull(cellStyle.getCellAlignment().getCTCellAlignment().getVertical());
+               assertFalse(cellStyle.getCellAlignment().getCTCellAlignment().isSetVertical());
+               assertEquals(STVerticalAlignment.BOTTOM, cellStyle.getCellAlignment().getCTCellAlignment().getVertical());
 
                cellStyle.setVerticalAlignment(VerticalAlignment.CENTER);
                assertEquals(VerticalAlignment.CENTER, cellStyle.getVerticalAlignment());

diff --git a/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFRun.java b/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFRun.java
index c5db3cbb134003efdaf0944e413cd19c0e36cfdf..3bb802e3442e5faba499ab165ea820e394bb5548 100644 (file)
--- a/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFRun.java
+++ b/src/ooxml/testcases/org/apache/poi/xwpf/usermodel/TestXWPFRun.java
@@ -139,7 +139,7 @@ public class TestXWPFRun {
         run.setBold(false);
         // Implementation detail: POI natively prefers <w:b w:val="false"/>,
         // but should correctly read val="0" and val="off"
-        assertEquals("off", rpr.getB().getVal());
+        assertEquals("off", rpr.getBArray(0).getVal());
     }
 
     @Test
@@ -151,7 +151,7 @@ public class TestXWPFRun {
         assertTrue(run.isItalic());
 
         run.setItalic(false);
-        assertEquals("off", rpr.getI().getVal());
+        assertEquals("off", rpr.getIArray(0).getVal());
     }
 
     @Test
@@ -163,7 +163,7 @@ public class TestXWPFRun {
         assertTrue(run.isStrikeThrough());
 
         run.setStrikeThrough(false);
-        assertEquals("off", rpr.getStrike().getVal());
+        assertEquals("off", rpr.getStrikeArray(0).getVal());
     }
 
     @Test
@@ -176,7 +176,7 @@ public class TestXWPFRun {
                 .getValue());
 
         run.setUnderline(UnderlinePatterns.NONE);
-        assertEquals(STUnderline.NONE.intValue(), rpr.getU().getVal()
+        assertEquals(STUnderline.NONE.intValue(), rpr.getUArray(0).getVal()
                 .intValue());
     }
 
@@ -189,7 +189,7 @@ public class TestXWPFRun {
         assertEquals("Times New Roman", run.getFontFamily());
 
         run.setFontFamily("Verdana");
-        assertEquals("Verdana", rpr.getRFonts().getAscii());
+        assertEquals("Verdana", rpr.getRFontsArray(0).getAscii());
     }
 
     @Test
@@ -202,10 +202,10 @@ public class TestXWPFRun {
         assertEquals(7.0, run.getFontSizeAsDouble(), 0.01);
 
         run.setFontSize(24);
-        assertEquals("48", rpr.getSz().getVal().toString());
+        assertEquals("48", rpr.getSzArray(0).getVal().toString());
 
         run.setFontSize(24.5f);
-        assertEquals("49", rpr.getSz().getVal().toString());
+        assertEquals("49", rpr.getSzArray(0).getVal().toString());
         assertEquals(25, run.getFontSize());
         assertEquals(24.5, run.getFontSizeAsDouble(), 0.01);
     }
@@ -219,7 +219,7 @@ public class TestXWPFRun {
         assertEquals(4000, run.getTextPosition());
 
         run.setTextPosition(2400);
-        assertEquals("2400", rpr.getPosition().getVal().toString());
+        assertEquals("2400", rpr.getPositionArray(0).getVal().toString());
     }
 
     @Test
@@ -331,8 +331,8 @@ public class TestXWPFRun {
         assertTrue(run.isBold());
         assertTrue(run.isItalic());
         assertFalse(run.isStrikeThrough());
-        assertTrue(run.getCTR().getRPr().isSetB());
-        assertFalse(run.getCTR().getRPr().getB().isSetVal());
+        assertTrue(run.getCTR().getRPr().sizeOfBArray() > 0);
+        assertFalse(run.getCTR().getRPr().getBArray(0).isSetVal());
 
 
         // Back to normal
@@ -477,10 +477,10 @@ public class TestXWPFRun {
         run.getCTR().addNewRPr().addNewLang().setVal("en-CA");
         assertEquals("en-CA", run.getLang());
 
-        run.getCTR().getRPr().getLang().setVal("fr-CA");
+        run.getCTR().getRPr().getLangArray(0).setVal("fr-CA");
         assertEquals("fr-CA", run.getLang());
 
-        run.getCTR().getRPr().getLang().setVal(null);
+        run.getCTR().getRPr().getLangArray(0).setVal(null);
         assertNull(run.getLang());
     }
 
@@ -734,7 +734,7 @@ public class TestXWPFRun {
         XWPFRun run = new XWPFRun(ctRun, irb);
 
         run.setSubscript(VerticalAlign.BASELINE);
-        assertEquals(STVerticalAlignRun.BASELINE, rpr.getVertAlign().getVal());
+        assertEquals(STVerticalAlignRun.BASELINE, rpr.getVertAlignArray(0).getVal());
     }
 
     @Test
@@ -787,7 +787,7 @@ public class TestXWPFRun {
 
         String styleId = "bolditalic";
         run.setStyle(styleId);
-        String candStyleId = run.getCTR().getRPr().getRStyle().getVal();
+        String candStyleId = run.getCTR().getRPr().getRStyleArray(0).getVal();
         assertNotNull("Expected to find a run style ID", candStyleId);
         assertEquals(styleId, candStyleId);