From: PJ Fanning <fanningpj@apache.org>
Date: Thu, 12 Sep 2024 21:16:34 +0000 (+0000)
Subject: [bug-69209] default ignoreMissingFontSystem to true
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=5fd1f28b4b886cc7c3024a22c1ecd516ece584e5;p=poi.git

[bug-69209] default ignoreMissingFontSystem to true

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

diff --git a/poi/src/main/java/org/apache/poi/ss/util/SheetUtil.java b/poi/src/main/java/org/apache/poi/ss/util/SheetUtil.java
index f90d229895..a6ea07893a 100644
--- a/poi/src/main/java/org/apache/poi/ss/util/SheetUtil.java
+++ b/poi/src/main/java/org/apache/poi/ss/util/SheetUtil.java
@@ -100,10 +100,10 @@ public class SheetUtil {
 
     /**
      * A system property which can be enabled to not fail when the
-     * font-system is not available on the current machine
+     * font-system is not available on the current machine.
+     * Since POI 5.3.1, this flag is enabled by default.
      */
-    private static boolean ignoreMissingFontSystem =
-            Boolean.parseBoolean(System.getProperty("org.apache.poi.ss.ignoreMissingFontSystem"));
+    private static boolean ignoreMissingFontSystem = initIgnoreMissingFontSystemFlag();
 
     /**
      * Which default char-width to use if the font-system is unavailable.
@@ -507,4 +507,12 @@ public class SheetUtil {
     protected static void setFontRenderContext(FontRenderContext fontRenderContext) {
         SheetUtil.fontRenderContext = fontRenderContext;
     }
+
+    private static boolean initIgnoreMissingFontSystemFlag() {
+        final String flag = System.getProperty("org.apache.poi.ss.ignoreMissingFontSystem");
+        if (flag != null) {
+            return !flag.trim().equalsIgnoreCase("false");
+        }
+        return true;
+    }
 }