Removed PropertyManager argument from initProperties(), as it is already

available within its FObj object.

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

diff --git a/src/java/org/apache/fop/layoutmgr/AbstractLayoutManager.java b/src/java/org/apache/fop/layoutmgr/AbstractLayoutManager.java
index f6d42ae67d6fb159d82d6f4f6cefc458978824c0..e34fbc69a143fab908fd54393609fd174dfeb1f8 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/AbstractLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/AbstractLayoutManager.java
@@ -83,6 +83,24 @@ public abstract class AbstractLayoutManager implements LayoutManager, Constants
         childLMiter = new LMiter(this, fobj.getChildren());
     }
 
+    /**
+     * This method provides a hook for a LayoutManager to initialize traits
+     * for the areas it will create, based on Properties set on its FO.
+     */
+    public void initialize() {
+        if (fobj != null && bInited == false) {
+            initProperties();
+            bInited = true;
+        }
+    }
+
+    /**
+     * This method is called by initialize() to set any method variables
+     * based on Properties set on its FO.
+     */
+    protected void initProperties() {
+    }
+
     /**
      * Set the user agent.
      *
@@ -224,27 +242,6 @@ public abstract class AbstractLayoutManager implements LayoutManager, Constants
         //  }
     }
 
-
-    /**
-     * This method provides a hook for a LayoutManager to initialize traits
-     * for the areas it will create, based on Properties set on its FO.
-     */
-    public void initialize() {
-        if (fobj != null && bInited == false) {
-            initProperties(fobj.getPropertyManager());
-            bInited = true;
-        }
-    }
-
-    /**
-     * This method provides a hook for a LayoutManager to initialize traits
-     * for the areas it will create, based on Properties set on its FO.
-     */
-    protected void initProperties(PropertyManager pm) {
-        //log.debug("AbstractLayoutManager.initProperties");
-    }
-
-
     /**
      * Tell whether this LayoutManager has handled all of its content.
      * @return True if there are no more break possibilities,

diff --git a/src/java/org/apache/fop/layoutmgr/BlockContainerLayoutManager.java b/src/java/org/apache/fop/layoutmgr/BlockContainerLayoutManager.java
index b2166b8c0f66bc5054944d56a309541ae2b0236a..0bfae0d3e04219cb47779d9cbc66daeffb45fc67 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/BlockContainerLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/BlockContainerLayoutManager.java
@@ -60,12 +60,6 @@ public class BlockContainerLayoutManager extends BlockStackingLayoutManager {
     // When viewport should grow with the content.
     private boolean autoHeight = true; 
 
-    /**
-     * Create a new block container layout manager.
-     */
-    public BlockContainerLayoutManager() {
-    }
-
     /**
      * Create a new block container layout manager.
      */
@@ -73,30 +67,36 @@ public class BlockContainerLayoutManager extends BlockStackingLayoutManager {
         super(node);
     }
 
-    public void setOverflow(int of) {
-        overflow = of;
-    }
-
-    protected void initProperties(PropertyManager pm) {
-        propManager = pm;
+    /**
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
+     */
+    protected void initProperties() {
+        propManager = fobj.getPropertyManager();
 
-        abProps = pm.getAbsolutePositionProps();
+        abProps = propManager.getAbsolutePositionProps();
         if (abProps.absolutePosition == AbsolutePosition.ABSOLUTE) {
             Rectangle2D rect = new Rectangle2D.Double(abProps.left,
                                 abProps.top, abProps.right - abProps.left,
                                 abProps.bottom - abProps.top);
             relDims = new FODimension(0, 0);
-            absoluteCTM = CTM.getCTMandRelDims(pm.getAbsRefOrient(),
-                pm.getWritingMode(), rect, relDims);
+            absoluteCTM = CTM.getCTMandRelDims(propManager.getAbsRefOrient(),
+                propManager.getWritingMode(), rect, relDims);
         }
-        marginProps = pm.getMarginProps();
-        borderProps = pm.getBorderAndPadding();
-        height = pm.getPropertyList().get(PR_BLOCK_PROGRESSION_DIMENSION | CP_OPTIMUM).getLength();
-        width = pm.getPropertyList().get(PR_INLINE_PROGRESSION_DIMENSION | CP_OPTIMUM).getLength();
+        
+        marginProps = propManager.getMarginProps();
+        borderProps = propManager.getBorderAndPadding();
+        height = propManager.getPropertyList().get(
+            PR_BLOCK_PROGRESSION_DIMENSION | CP_OPTIMUM).getLength();
+        width = propManager.getPropertyList().get(
+            PR_INLINE_PROGRESSION_DIMENSION | CP_OPTIMUM).getLength();
+    }
+
+    public void setOverflow(int of) {
+        overflow = of;
     }
 
     protected int getRotatedIPD() {
-        PropertyList props = propManager.getPropertyList();
+        PropertyList props = fobj.getPropertyList();
         int height = props.get(PR_HEIGHT).getLength().getValue();
         height = props.get(PR_INLINE_PROGRESSION_DIMENSION | CP_OPTIMUM).getLength().getValue();
 

diff --git a/src/java/org/apache/fop/layoutmgr/BlockLayoutManager.java b/src/java/org/apache/fop/layoutmgr/BlockLayoutManager.java
index f2339c519398444a34ca16d804095476a9f6d393..ec9fdcf05c6a9540c198dfb89e2c714e4411b555 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/BlockLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/BlockLayoutManager.java
@@ -83,6 +83,19 @@ public class BlockLayoutManager extends BlockStackingLayoutManager {
         lineHeight = ti.lineHeight;
     }
 
+    /**
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
+     */
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        layoutProps = pm.getLayoutProps();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
+        marginProps = pm.getMarginProps();
+        foBlockSpaceBefore = layoutProps.spaceBefore.getSpace();
+        prevFoBlockSpaceAfter = foBlockSpaceAfter;
+    }
+
     /**
      * Iterator for Block layout.
      * This iterator combines consecutive inline areas and
@@ -139,19 +152,6 @@ public class BlockLayoutManager extends BlockStackingLayoutManager {
         }
     }
 
-    /**
-     * This method provides a hook for a LayoutManager to intialize traits
-     * for the areas it will create, based on Properties set on its FO.
-     */
-    protected void initProperties(PropertyManager pm) {
-        layoutProps = pm.getLayoutProps();
-        borderProps = pm.getBorderAndPadding();
-        backgroundProps = pm.getBackgroundProps();
-        marginProps = pm.getMarginProps();
-        foBlockSpaceBefore = layoutProps.spaceBefore.getSpace();
-        prevFoBlockSpaceAfter = foBlockSpaceAfter;
-    }
-
     public BreakPoss getNextBreakPoss(LayoutContext context) {
         LayoutManager curLM; // currently active LM
 

diff --git a/src/java/org/apache/fop/layoutmgr/InlineStackingLayoutManager.java b/src/java/org/apache/fop/layoutmgr/InlineStackingLayoutManager.java
index 3cb4f3b960fe043cf5d0c229a4ee73ecd2a25d78..78bf35d0aa1d5ad24381b661cf42a3290d8024b1 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/InlineStackingLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/InlineStackingLayoutManager.java
@@ -83,14 +83,6 @@ public class InlineStackingLayoutManager extends AbstractLayoutManager {
     /** Used to store previous content IPD for each child LM. */
     private HashMap hmPrevIPD = new HashMap();
 
-    /**
-     * Create an inline stacking layout manager.
-     * This is used for fo's that create areas that
-     * contain inline areas.
-     */
-    public InlineStackingLayoutManager() {
-    }
-
     /**
      * Create an inline stacking layout manager.
      * This is used for fo's that create areas that
@@ -113,6 +105,24 @@ public class InlineStackingLayoutManager extends AbstractLayoutManager {
         childLMiter = null;
     }
 
+    /**
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
+     */
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        inlineProps = pm.getInlineProps();
+        borderProps = pm.getBorderAndPadding();
+        // Calculdate border and padding size in BPD
+        int iPad = borderProps.getPadding(CommonBorderAndPadding.BEFORE, false);
+        iPad += borderProps.getBorderWidth(CommonBorderAndPadding.BEFORE,
+                                             false);
+        iPad += borderProps.getPadding(CommonBorderAndPadding.AFTER, false);
+        iPad += borderProps.getBorderWidth(CommonBorderAndPadding.AFTER, false);
+        extraBPD = new MinOptMax(iPad);
+
+        backgroundProps = pm.getBackgroundProps();
+    }
+
     /**
      * Set the iterator.
      *
@@ -132,26 +142,6 @@ public class InlineStackingLayoutManager extends AbstractLayoutManager {
         return true;
     }
 
-    /**
-     * Initialize properties for this layout manager.
-     *
-     * @param propMgr the property manager from the fo that created this manager
-     */
-    protected void initProperties(PropertyManager propMgr) {
-        // super.initProperties(propMgr);
-        inlineProps = propMgr.getInlineProps();
-        borderProps = propMgr.getBorderAndPadding();
-        // Calculdate border and padding size in BPD
-        int iPad = borderProps.getPadding(CommonBorderAndPadding.BEFORE, false);
-        iPad += borderProps.getBorderWidth(CommonBorderAndPadding.BEFORE,
-                                             false);
-        iPad += borderProps.getPadding(CommonBorderAndPadding.AFTER, false);
-        iPad += borderProps.getBorderWidth(CommonBorderAndPadding.AFTER, false);
-        extraBPD = new MinOptMax(iPad);
-
-        backgroundProps = propMgr.getBackgroundProps();
-    }
-
     private MinOptMax getExtraIPD(boolean bNotFirst, boolean bNotLast) {
         int iBP = borderProps.getPadding(CommonBorderAndPadding.START,
                                            bNotFirst);

diff --git a/src/java/org/apache/fop/layoutmgr/LineLayoutManager.java b/src/java/org/apache/fop/layoutmgr/LineLayoutManager.java
index a5ad82a8b8054cb64711f61aed6885ec3f32edbd..3fec261501764f0f18e8711c51cd0a2c348ea106 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/LineLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/LineLayoutManager.java
@@ -48,6 +48,19 @@ import org.apache.fop.traits.MinOptMax;
  */
 public class LineLayoutManager extends InlineStackingLayoutManager {
 
+    /**
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
+     */
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        CommonMarginBlock marginProps = pm.getMarginProps();
+        iIndents = marginProps.startIndent + marginProps.endIndent;
+        BlockProps blockProps = pm.getBlockProps();
+        bTextAlignment = blockProps.textAlign;
+        textIndent = blockProps.firstIndent;
+        hyphProps = pm.getHyphenationProps();
+    }
+
     /**
      * Private class to store information about inline breaks.
      * Each value holds the start and end indexes into a List of
@@ -107,20 +120,6 @@ public class LineLayoutManager extends InlineStackingLayoutManager {
         initialize(); // Normally done when started by parent!
     }
 
-    /**
-     * Initialize the properties for this layout manager.
-     * The properties are from the block area.
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
-     */
-    protected void initProperties(PropertyManager propMgr) {
-        CommonMarginBlock marginProps = propMgr.getMarginProps();
-        iIndents = marginProps.startIndent + marginProps.endIndent;
-        BlockProps blockProps = propMgr.getBlockProps();
-        bTextAlignment = blockProps.textAlign;
-        textIndent = blockProps.firstIndent;
-        hyphProps = propMgr.getHyphenationProps();
-    }
-
     /**
      * Call child layout managers to generate content.
      * This gets the next break which is a full line.

diff --git a/src/java/org/apache/fop/layoutmgr/list/Item.java b/src/java/org/apache/fop/layoutmgr/list/Item.java
index 9ba9d9f566923189e1d29aabf5f17459c518487d..dabb267a94f4880bdabc89a78199766ffd4ed2b1 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/list/Item.java
+++ b/src/java/org/apache/fop/layoutmgr/list/Item.java
@@ -62,11 +62,12 @@ public class Item extends BlockStackingLayoutManager {
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/list/ListBlockLayoutManager.java b/src/java/org/apache/fop/layoutmgr/list/ListBlockLayoutManager.java
index 92b46dcce1e9917c373dabdf05f552f232cef584..a2f519c9ada3a1522b2054377bb0a5a06fb50a5b 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/list/ListBlockLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/list/ListBlockLayoutManager.java
@@ -68,11 +68,12 @@ public class ListBlockLayoutManager extends BlockStackingLayoutManager {
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/list/ListItemLayoutManager.java b/src/java/org/apache/fop/layoutmgr/list/ListItemLayoutManager.java
index 3f10d7526a830157508976a213b321ef58fade2f..8c58514bc779bec2441ed0244ffe6745ff789ea3 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/list/ListItemLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/list/ListItemLayoutManager.java
@@ -70,11 +70,12 @@ public class ListItemLayoutManager extends BlockStackingLayoutManager {
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/table/Body.java b/src/java/org/apache/fop/layoutmgr/table/Body.java
index 59a0ec0ce851b1ffeee0cc1c29775eabdcac3d17..c2c781eb7b30edb26a8a0c47c5d15299416588c8 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/table/Body.java
+++ b/src/java/org/apache/fop/layoutmgr/table/Body.java
@@ -65,13 +65,12 @@ public class Body extends BlockStackingLayoutManager {
     }
 
     /**
-     * Initialize properties for this layout manager.
-     *
-     * @param propMgr the property manager from the fo object
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/table/Cell.java b/src/java/org/apache/fop/layoutmgr/table/Cell.java
index 75aa16ed972974b4b0a1dd21f5adcb3794ea256b..c2446b689a965c7e1f8cbb4dce655e4992d9a390 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/table/Cell.java
+++ b/src/java/org/apache/fop/layoutmgr/table/Cell.java
@@ -64,11 +64,12 @@ public class Cell extends BlockStackingLayoutManager {
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/table/Column.java b/src/java/org/apache/fop/layoutmgr/table/Column.java
index 662394e8fcdcc8d1315ff9a40f782ce33d20a0c4..b8d214f217b67188c36e4a4098b9770d2764e0db 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/table/Column.java
+++ b/src/java/org/apache/fop/layoutmgr/table/Column.java
@@ -50,11 +50,12 @@ public class Column extends AbstractLayoutManager {
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/table/Row.java b/src/java/org/apache/fop/layoutmgr/table/Row.java
index 5ae91879fb90e69bac9b4fcf3e2c14feae75e223..e0c9cf3fc6b1d5d7ecc190f011f6107061ce9e6f 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/table/Row.java
+++ b/src/java/org/apache/fop/layoutmgr/table/Row.java
@@ -72,13 +72,12 @@ public class Row extends BlockStackingLayoutManager {
     }
 
     /**
-     * Initialize properties for this layout manager.
-     *
-     * @param propMgr the property manager for the fo
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**

diff --git a/src/java/org/apache/fop/layoutmgr/table/TableLayoutManager.java b/src/java/org/apache/fop/layoutmgr/table/TableLayoutManager.java
index fa464aea7881dc267896c67a39a7276efbdb9dda..c11fd90d71b2aab11199d3ce18226797cd748abc 100644 (file)
--- a/src/java/org/apache/fop/layoutmgr/table/TableLayoutManager.java
+++ b/src/java/org/apache/fop/layoutmgr/table/TableLayoutManager.java
@@ -81,11 +81,12 @@ public class TableLayoutManager extends BlockStackingLayoutManager {
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties(PropertyManager)
+     * @see org.apache.fop.layoutmgr.AbstractLayoutManager#initProperties()
      */
-    protected void initProperties(PropertyManager propMgr) {
-        borderProps = propMgr.getBorderAndPadding();
-        backgroundProps = propMgr.getBackgroundProps();
+    protected void initProperties() {
+        PropertyManager pm = fobj.getPropertyManager();
+        borderProps = pm.getBorderAndPadding();
+        backgroundProps = pm.getBackgroundProps();
     }
 
     /**