Changeset 1754 for trunk/docx4j/src/main/java/org/docx4j/model

Timestamp:

01/24/12 01:35:16 (4 months ago)

Author:

jharrop

Message:

Improvements to indentation (direct indent values are given priority over values in numbering, which in turn is given priority over values in styles), plus associated unit tests.

Location:

trunk/docx4j/src/main/java/org/docx4j/model

Files:

• ImmutablePropertyResolver.java (modified) (1 diff)
• PropertyResolver.java (modified) (6 diffs)
• properties/PropertyFactory.java (modified) (2 diffs)

trunk/docx4j/src/main/java/org/docx4j/model/ImmutablePropertyResolver.java

@@ -24 +24 @@
         }
 
-        public ImmutablePropertyResolver(StyleDefinitionsPart styleDefinitionsPart,
-                        ThemePart themePart,
-                        NumberingDefinitionsPart numberingDefinitionsPart)
-                        throws Docx4JException {
-                super(styleDefinitionsPart, themePart, numberingDefinitionsPart);
-        }
+//      public ImmutablePropertyResolver(StyleDefinitionsPart styleDefinitionsPart,
+//                      ThemePart themePart,
+//                      NumberingDefinitionsPart numberingDefinitionsPart)
+//                      throws Docx4JException {
+//              super(styleDefinitionsPart, themePart, numberingDefinitionsPart);
+//      }
 
         protected void applyPPr(PPr src, PPr dest) {

trunk/docx4j/src/main/java/org/docx4j/model/PropertyResolver.java

@@ -17 +17 @@
 import org.docx4j.model.properties.PropertyFactory;
 import org.docx4j.model.properties.paragraph.AbstractParagraphProperty;
+import org.docx4j.model.properties.paragraph.Indent;
 import org.docx4j.model.properties.run.AbstractRunProperty;
 import org.docx4j.openpackaging.exceptions.Docx4JException;
@@ -107 +108 @@
         private StyleDefinitionsPart styleDefinitionsPart;
         
+        private WordprocessingMLPackage wordMLPackage;
         
         /**
@@ -133 +135 @@
         public PropertyResolver(WordprocessingMLPackage wordMLPackage) throws Docx4JException {
                 
+                this.wordMLPackage = wordMLPackage;
+                
                 MainDocumentPart mdp = wordMLPackage.getMainDocumentPart();
                 
@@ -141 +145 @@
         }
         
-        public PropertyResolver(StyleDefinitionsPart styleDefinitionsPart,
-                                                        ThemePart themePart,
-                                                        NumberingDefinitionsPart numberingDefinitionsPart) throws Docx4JException {
-                
-                this.styleDefinitionsPart= styleDefinitionsPart;
-                this.themePart = themePart;
-                this.numberingDefinitionsPart = numberingDefinitionsPart;
-                init();
-        }
+//      public PropertyResolver(StyleDefinitionsPart styleDefinitionsPart,
+//                                                      ThemePart themePart,
+//                                                      NumberingDefinitionsPart numberingDefinitionsPart) throws Docx4JException {
+//              
+//              this.styleDefinitionsPart= styleDefinitionsPart;
+//              this.themePart = themePart;
+//              this.numberingDefinitionsPart = numberingDefinitionsPart;
+//              init();
+//      }
         
         String defaultParagraphStyleId;  // "Normal" in English, but ...
@@ -918 +922 @@
         protected void applyPPr(PPr pPrToApply, PPr effectivePPr) {
                 
+                log.debug( "apply " + XmlUtils.marshaltoString(pPrToApply,  true, true)
+                + "\n\r to " + XmlUtils.marshaltoString(effectivePPr,  true, true) );
+                
                 if (pPrToApply==null) {
                         return;
                 }
                 
-        List<Property> properties = PropertyFactory.createProperties(null, pPrToApply); // wmlPackage null
-        
+        List<Property> properties = PropertyFactory.createProperties(wordMLPackage, pPrToApply); 
         for( Property p :  properties ) {
                         if (p!=null) {
@@ -931 +937 @@
         }
 
-
+                log.debug( "result " + XmlUtils.marshaltoString(effectivePPr,  true, true) );
+        
         }
         

trunk/docx4j/src/main/java/org/docx4j/model/properties/PropertyFactory.java

@@ -25 +25 @@
 
 import org.apache.log4j.Logger;
+import org.docx4j.XmlUtils;
 import org.docx4j.model.properties.paragraph.Indent;
 import org.docx4j.model.properties.paragraph.Justification;
@@ -396 +397 @@
 //              if (pPr.getMirrorIndents() != null)
 //                      dest.setMirrorIndents(pPr.getMirrorIndents());
-                Ind ind = null;
-                if (pPr.getNumPr() != null) {
+                
+                /*
+                 * Where a p has indentation specified by both w:numPr and direct w:ind, eg:
+                 *  
+                    <w:p>
+                      <w:pPr>
+                        <w:numPr>
+                          <w:ilvl w:val="0"/>
+                          <w:numId w:val="2"/>
+                        </w:numPr>
+                        <w:ind w:left="0" w:firstLine="0"/>
+                      </w:pPr>
+
+                        we want to ensure that the direct values are given effect.
+                        
+                        ie indent on direct pPr trumps indent in pPr in numbering, which trumps indent
+                        specified in a style.  
+
+                 */
+                Indent indent = null; 
+                boolean numberingIndent = false;
+                if (pPr.getNumPr() == null) {
+                        log.debug("No numPr.. ") ;                                                                      
+                } else {
                         // Numbering is mostly handled directly in the HTML & PDF stylesheets                   
                         properties.add(new NumberingProperty(pPr.getNumPr()));
-                        // but we do want to override w:ind above
-                        if (pPr.getNumPr()!=null) {
-                                if (wmlPackage instanceof WordprocessingMLPackage) {
-                                        NumberingDefinitionsPart ndp = ((WordprocessingMLPackage)wmlPackage).getMainDocumentPart().getNumberingDefinitionsPart();
-                    if (ndp != null) {
-                                        ind = ndp.getInd(pPr.getNumPr());
-                                        if (ind!=null) {
-                                                properties.add(new Indent(ind));                        
+                        // but we do want to get w:ind (in case not set elsewhere)
+                        if (wmlPackage instanceof WordprocessingMLPackage) {
+                                NumberingDefinitionsPart ndp 
+                                        = ((WordprocessingMLPackage)wmlPackage).getMainDocumentPart().getNumberingDefinitionsPart();
+                if (ndp == null) {
+                                        log.debug("No NDP?.. ") ;                                                                       
+                } else {
+                                        Ind ind = ndp.getInd(pPr.getNumPr());
+                                        if (ind==null) {
+                                                log.debug("No Indent in numbering.. ") ;                                                
+                                        } else {
+                                                log.debug("Indent from numbering: " + XmlUtils.marshaltoString(ind,  true, true)) ; 
+                                                indent = new Indent(ind);
+                                                properties.add(indent);                 
                                                 log.debug("Using w:ind from list level");
+                                                numberingIndent = true;
                                         }
-                    }
+                }
+                        } else {
+                                log.info(wmlPackage + " " + wmlPackage.getClass().getName() ) ;
+                        }
+                }
+                
+                // Give effect to the prioritisation
+                        if (pPr.getInd() != null) {
+                                log.debug("Indent from ppr: " + XmlUtils.marshaltoString(pPr.getInd(),  true, true)) ; 
+                                if (indent!=null) {
+                                        properties.remove(indent);
                                 }
+                                properties.add(new Indent(pPr.getInd()));                       
                         }
-                }
-                if (ind==null) {
-                        if (pPr.getInd() != null)
-                                properties.add(new Indent(pPr.getInd()));                       
-                }
+                
 //              if (pPr.getOutlineLvl() != null)
 //                      dest.setOutlineLvl(pPr.getOutlineLvl());