From: Manuel Mall <manuel@apache.org>
Date: Wed, 9 Nov 2005 11:49:43 +0000 (+0000)
Subject: Fix for Bug 36928 - refresh cached property value if explicitly set twice which can... 
X-Git-Tag: fop-0_90-alpha1~73
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d9aaf8603b6f29c1ad5e273d801ce8202d5b5134;p=xmlgraphics-fop.git

Fix for Bug 36928 - refresh cached property value if explicitly set twice which can happen for font-size

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

diff --git a/src/java/org/apache/fop/fo/StaticPropertyList.java b/src/java/org/apache/fop/fo/StaticPropertyList.java
index 37699847a..a8455ee9e 100755
--- a/src/java/org/apache/fop/fo/StaticPropertyList.java
+++ b/src/java/org/apache/fop/fo/StaticPropertyList.java
@@ -53,6 +53,9 @@ public class StaticPropertyList extends PropertyList {
      */
     public void putExplicit(int propId, Property value) {
         explicit[propId] = value;
+        if (values[propId] != null) { // if the cached value is set overwrite it
+            values[propId] = value;
+        }
     }
 
     /**
diff --git a/test/layoutengine/testcases/block_font-size_relative.xml b/test/layoutengine/testcases/block_font-size_relative.xml
index 8dc3dbf8c..d12ebfe07 100644
--- a/test/layoutengine/testcases/block_font-size_relative.xml
+++ b/test/layoutengine/testcases/block_font-size_relative.xml
@@ -118,6 +118,16 @@
               </fo:block>
             </fo:block>
           </fo:block>
+          <fo:block font-size="36pt">font-size="36pt"
+            <fo:block font-size="0.5em">font-size="0.5em"
+              <fo:block font-size="0.5em">font-size="0.5em"
+                <fo:block font-size="2em">font-size="2em"
+                  <fo:block font-size="2em">font-size="2em"
+                  </fo:block>
+                </fo:block>
+              </fo:block>
+            </fo:block>
+          </fo:block>
         </fo:flow>
       </fo:page-sequence>
     </fo:root>
@@ -163,5 +173,14 @@
     <eval expected="51598" xpath="//flow/block[8]/block/block/lineArea/text/@font-size"/>
     <eval expected="42998" xpath="//flow/block[8]/block/block/block/lineArea/text/@font-size"/>
     <eval expected="35832" xpath="//flow/block[8]/block/block/block/block/lineArea/text/@font-size"/>
+    <eval expected="14400" xpath="//flow/block[9]/lineArea/text/@font-size"/>
+    <eval expected="17280" xpath="//flow/block[9]/block/lineArea/text/@font-size"/>
+    <eval expected="14400" xpath="//flow/block[9]/block/block/lineArea/text/@font-size"/>
+    <eval expected="12000" xpath="//flow/block[9]/block/block/block/lineArea/text/@font-size"/>
+    <eval expected="36000" xpath="//flow/block[10]/lineArea/text/@font-size"/>
+    <eval expected="18000" xpath="//flow/block[10]/block/lineArea/text/@font-size"/>
+    <eval expected="9000" xpath="//flow/block[10]/block/block/lineArea/text/@font-size"/>
+    <eval expected="18000" xpath="//flow/block[10]/block/block/block/lineArea/text/@font-size"/>
+    <eval expected="36000" xpath="//flow/block[10]/block/block/block/block/lineArea/text/@font-size"/>
   </checks>
 </testcase>