From 7951774f1754564fb27f775ce3eccbdf71e3e062 Mon Sep 17 00:00:00 2001
From: Simon Steiner <ssteiner@apache.org>
Date: Thu, 11 Oct 2018 15:28:23 +0000
Subject: [PATCH] FOP-2822: Use correct ipd for table inside float

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@1843563 13f79535-47bb-0310-9956-ffa450edef68
---
 .../layoutmgr/FloatContentLayoutManager.java  | 23 ++++++-
 .../standard-testcases/float_8.xml            | 67 +++++++++++++++++++
 2 files changed, 89 insertions(+), 1 deletion(-)
 create mode 100644 fop/test/layoutengine/standard-testcases/float_8.xml

diff --git a/fop-core/src/main/java/org/apache/fop/layoutmgr/FloatContentLayoutManager.java b/fop-core/src/main/java/org/apache/fop/layoutmgr/FloatContentLayoutManager.java
index dd65be248..b489760ce 100644
--- a/fop-core/src/main/java/org/apache/fop/layoutmgr/FloatContentLayoutManager.java
+++ b/fop-core/src/main/java/org/apache/fop/layoutmgr/FloatContentLayoutManager.java
@@ -31,6 +31,7 @@ import org.apache.fop.fo.flow.Float;
 import org.apache.fop.fo.properties.CommonBorderPaddingBackground;
 import org.apache.fop.layoutmgr.inline.FloatLayoutManager;
 import org.apache.fop.layoutmgr.inline.KnuthInlineBox;
+import org.apache.fop.layoutmgr.table.TableLayoutManager;
 
 public class FloatContentLayoutManager extends SpacedBorderedPaddedBlockLayoutManager {
 
@@ -75,7 +76,7 @@ public class FloatContentLayoutManager extends SpacedBorderedPaddedBlockLayoutMa
     public void addChildArea(Area childArea) {
         floatContentArea.addChildArea(childArea);
         floatContentArea.setBPD(childArea.getAllocBPD());
-        int effectiveContentIPD = childArea.getEffectiveAllocIPD();
+        int effectiveContentIPD = getContentAreaIPD(childLMs, childArea);
         int contentIPD = childArea.getIPD();
         int xOffset = childArea.getBorderAndPaddingWidthStart();
         floatContentArea.setIPD(effectiveContentIPD);
@@ -99,6 +100,26 @@ public class FloatContentLayoutManager extends SpacedBorderedPaddedBlockLayoutMa
         }
     }
 
+    private int getContentAreaIPD(List<LayoutManager> childLMs, Area childArea) {
+        int ipd = getContentAreaIPD(childLMs);
+        if (ipd == 0) {
+            return childArea.getEffectiveAllocIPD();
+        }
+        return ipd;
+    }
+
+    private int getContentAreaIPD(List<LayoutManager> childLMs) {
+        int ipd = 0;
+        for (LayoutManager childLM : childLMs) {
+            if (childLM instanceof TableLayoutManager) {
+                ipd += childLM.getContentAreaIPD();
+            } else {
+                ipd += getContentAreaIPD(childLM.getChildLMs());
+            }
+        }
+        return ipd;
+    }
+
     /**
      * {Add info}
      *
diff --git a/fop/test/layoutengine/standard-testcases/float_8.xml b/fop/test/layoutengine/standard-testcases/float_8.xml
new file mode 100644
index 000000000..5c1d986f7
--- /dev/null
+++ b/fop/test/layoutengine/standard-testcases/float_8.xml
@@ -0,0 +1,67 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  Licensed to the Apache Software Foundation (ASF) under one or more
+  contributor license agreements.  See the NOTICE file distributed with
+  this work for additional information regarding copyright ownership.
+  The ASF licenses this file to You under the Apache License, Version 2.0
+  (the "License"); you may not use this file except in compliance with
+  the License.  You may obtain a copy of the License at
+
+       http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+-->
+<!-- $Id$ -->
+<testcase>
+  <info>
+    <p>
+      This test checks floats.
+    </p>
+  </info>
+  <fo>
+<fo:root xmlns:fo="http://www.w3.org/1999/XSL/Format" xmlns:fox="http://xmlgraphics.apache.org/fop/extensions">
+  <fo:layout-master-set>
+    <fo:simple-page-master master-name="Page" page-height="11in" page-width="8.5in" margin="1in">
+      <fo:region-body region-name="Body"/>      
+    </fo:simple-page-master>
+    <fo:page-sequence-master master-name="PageSequence">
+      <fo:repeatable-page-master-reference master-reference="Page"/>
+    </fo:page-sequence-master>
+  </fo:layout-master-set>
+  <fo:page-sequence format="1" force-page-count="auto" initial-page-number="auto" master-reference="PageSequence" id="TH_LastPage">
+    <fo:flow flow-name="Body">
+      <fo:block>
+        <fo:float float="right">
+          <fo:block id="infloat">
+            <fo:table table-layout="fixed" width="3.25in" id="table">
+              <fo:table-column column-width="1in" column-number="1"/>
+              <fo:table-column column-width="2.25in" column-number="2"/>
+              <fo:table-body>
+                <fo:table-row>
+                  <fo:table-cell number-columns-spanned="2" border-style="solid">        
+                      <fo:block>YOUR POLICY INFORMATION</fo:block>       
+                  </fo:table-cell>
+                </fo:table-row>      
+              </fo:table-body>
+            </fo:table>
+          </fo:block>
+        </fo:float>
+      </fo:block>
+      <fo:block id="outfloat">
+        <fo:inline font-size="12pt">
+          <fo:inline>Your bank has declined your payment</fo:inline>
+        </fo:inline>
+      </fo:block>  
+    </fo:flow>
+  </fo:page-sequence>
+</fo:root>
+  </fo>
+  <checks>
+    <eval expected="234000" xpath="//pageViewport[1]/page/regionViewport[1]//block[2]/@ipd" />
+    <eval expected="234000" xpath="//pageViewport[1]/page/regionViewport[1]//block[4]/@ipd" />
+  </checks>
+</testcase>
-- 
2.39.5