officecfg/registry/data/org/openoffice/Office/Math.xcu |  198 -----------------
 starmath/source/cfgitem.cxx                            |    2 
 starmath/source/node.cxx                               |    5 
 3 files changed, 4 insertions(+), 201 deletions(-)

New commits:
commit 32d2e001bb056e7d98aa143b222f2721967337dc
Author:     Khaled Hosny <kha...@libreoffice.org>
AuthorDate: Tue Sep 12 18:35:46 2023 +0300
Commit:     خالد حسني <kha...@libreoffice.org>
CommitDate: Wed Sep 13 12:54:40 2023 +0200

    tdf#101174: Don’t hard-code OpenSymbol font for predefined special symbols
    
    Leave the format empty and use the variables font instead.
    
    Change-Id: I3e3cdf40dc285dee7375054e85b4a2bed352eb66
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156855
    Tested-by: Jenkins
    Reviewed-by: خالد حسني <kha...@libreoffice.org>

diff --git a/officecfg/registry/data/org/openoffice/Office/Math.xcu 
b/officecfg/registry/data/org/openoffice/Office/Math.xcu
index 96e6653615d7..02e5e94acfcb 100644
--- a/officecfg/registry/data/org/openoffice/Office/Math.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/Math.xcu
@@ -72,9 +72,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="ALPHA" oor:op="replace">
       <prop oor:name="Char">
@@ -86,9 +83,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="beta" oor:op="replace">
       <prop oor:name="Char">
@@ -100,9 +94,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="BETA" oor:op="replace">
       <prop oor:name="Char">
@@ -114,9 +105,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="gamma" oor:op="replace">
       <prop oor:name="Char">
@@ -128,9 +116,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="GAMMA" oor:op="replace">
       <prop oor:name="Char">
@@ -142,9 +127,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="delta" oor:op="replace">
       <prop oor:name="Char">
@@ -156,9 +138,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="DELTA" oor:op="replace">
       <prop oor:name="Char">
@@ -170,9 +149,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="epsilon" oor:op="replace">
       <prop oor:name="Char">
@@ -184,9 +160,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="EPSILON" oor:op="replace">
       <prop oor:name="Char">
@@ -198,9 +171,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="zeta" oor:op="replace">
       <prop oor:name="Char">
@@ -212,9 +182,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="ZETA" oor:op="replace">
       <prop oor:name="Char">
@@ -226,9 +193,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="eta" oor:op="replace">
       <prop oor:name="Char">
@@ -240,9 +204,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="ETA" oor:op="replace">
       <prop oor:name="Char">
@@ -254,9 +215,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="theta" oor:op="replace">
       <prop oor:name="Char">
@@ -268,9 +226,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="THETA" oor:op="replace">
       <prop oor:name="Char">
@@ -282,9 +237,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="iota" oor:op="replace">
       <prop oor:name="Char">
@@ -296,9 +248,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="IOTA" oor:op="replace">
       <prop oor:name="Char">
@@ -310,9 +259,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="kappa" oor:op="replace">
       <prop oor:name="Char">
@@ -324,9 +270,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="KAPPA" oor:op="replace">
       <prop oor:name="Char">
@@ -338,9 +281,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="lambda" oor:op="replace">
       <prop oor:name="Char">
@@ -352,9 +292,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="LAMBDA" oor:op="replace">
       <prop oor:name="Char">
@@ -366,9 +303,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="mu" oor:op="replace">
       <prop oor:name="Char">
@@ -380,9 +314,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="MU" oor:op="replace">
       <prop oor:name="Char">
@@ -394,9 +325,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="nu" oor:op="replace">
       <prop oor:name="Char">
@@ -408,9 +336,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="NU" oor:op="replace">
       <prop oor:name="Char">
@@ -422,9 +347,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="xi" oor:op="replace">
       <prop oor:name="Char">
@@ -436,9 +358,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="XI" oor:op="replace">
       <prop oor:name="Char">
@@ -450,9 +369,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="omicron" oor:op="replace">
       <prop oor:name="Char">
@@ -464,9 +380,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="OMICRON" oor:op="replace">
       <prop oor:name="Char">
@@ -478,9 +391,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="pi" oor:op="replace">
       <prop oor:name="Char">
@@ -492,9 +402,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="PI" oor:op="replace">
       <prop oor:name="Char">
@@ -506,9 +413,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="rho" oor:op="replace">
       <prop oor:name="Char">
@@ -520,9 +424,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="RHO" oor:op="replace">
       <prop oor:name="Char">
@@ -534,9 +435,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="sigma" oor:op="replace">
       <prop oor:name="Char">
@@ -548,9 +446,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="SIGMA" oor:op="replace">
       <prop oor:name="Char">
@@ -562,9 +457,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="tau" oor:op="replace">
       <prop oor:name="Char">
@@ -576,9 +468,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="TAU" oor:op="replace">
       <prop oor:name="Char">
@@ -590,9 +479,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="upsilon" oor:op="replace">
       <prop oor:name="Char">
@@ -604,9 +490,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="UPSILON" oor:op="replace">
       <prop oor:name="Char">
@@ -618,9 +501,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="phi" oor:op="replace">
       <prop oor:name="Char">
@@ -632,9 +512,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="PHI" oor:op="replace">
       <prop oor:name="Char">
@@ -646,9 +523,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="chi" oor:op="replace">
       <prop oor:name="Char">
@@ -660,9 +534,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="CHI" oor:op="replace">
       <prop oor:name="Char">
@@ -674,9 +545,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="psi" oor:op="replace">
       <prop oor:name="Char">
@@ -688,9 +556,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="PSI" oor:op="replace">
       <prop oor:name="Char">
@@ -702,9 +567,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="omega" oor:op="replace">
       <prop oor:name="Char">
@@ -716,9 +578,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="OMEGA" oor:op="replace">
       <prop oor:name="Char">
@@ -730,9 +589,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="varepsilon" oor:op="replace">
       <prop oor:name="Char">
@@ -744,9 +600,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="vartheta" oor:op="replace">
       <prop oor:name="Char">
@@ -758,9 +611,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="varpi" oor:op="replace">
       <prop oor:name="Char">
@@ -772,9 +622,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="varrho" oor:op="replace">
       <prop oor:name="Char">
@@ -786,9 +633,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="varsigma" oor:op="replace">
       <prop oor:name="Char">
@@ -800,9 +644,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="varphi" oor:op="replace">
       <prop oor:name="Char">
@@ -814,9 +655,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id2</value>
-      </prop>
     </node>
     <node oor:name="element" oor:op="replace">
       <prop oor:name="Char">
@@ -828,9 +666,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="noelement" oor:op="replace">
       <prop oor:name="Char">
@@ -842,9 +677,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="strictlylessthan" oor:op="replace">
       <prop oor:name="Char">
@@ -856,9 +688,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="strictlygreaterthan" oor:op="replace">
       <prop oor:name="Char">
@@ -870,9 +699,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="notequal" oor:op="replace">
       <prop oor:name="Char">
@@ -884,9 +710,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="identical" oor:op="replace">
       <prop oor:name="Char">
@@ -898,9 +721,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="tendto" oor:op="replace">
       <prop oor:name="Char">
@@ -912,9 +732,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="and" oor:op="replace">
       <prop oor:name="Char">
@@ -926,9 +743,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="or" oor:op="replace">
       <prop oor:name="Char">
@@ -940,9 +754,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="infinite" oor:op="replace">
       <prop oor:name="Char">
@@ -954,9 +765,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="angle" oor:op="replace">
       <prop oor:name="Char">
@@ -968,9 +776,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
     <node oor:name="perthousand" oor:op="replace">
       <prop oor:name="Char">
@@ -982,9 +787,6 @@
       <prop oor:name="Predefined">
         <value>true</value>
       </prop>
-      <prop oor:name="FontFormatId">
-        <value>Id1</value>
-      </prop>
     </node>
   </node>
 </oor:component-data>
diff --git a/starmath/source/cfgitem.cxx b/starmath/source/cfgitem.cxx
index 6e27a0168e8b..8906474c9f35 100644
--- a/starmath/source/cfgitem.cxx
+++ b/starmath/source/cfgitem.cxx
@@ -434,8 +434,6 @@ void SmMathConfig::ReadSymbol( SmSym &rSymbol,
         if (pFntFmt)
             aFont = pFntFmt->GetFont();
     }
-    else
-        bOK = false;
     ++pValue;
 
     if (bOK)
diff --git a/starmath/source/node.cxx b/starmath/source/node.cxx
index 7a9d42bd4fe1..1572d5f8e67e 100644
--- a/starmath/source/node.cxx
+++ b/starmath/source/node.cxx
@@ -2186,7 +2186,10 @@ void SmSpecialNode::Prepare(const SmFormat &rFormat, 
const SmDocShell &rDocShell
         sal_UCS4 cChar = pSym->GetCharacter();
         OUString aTmp( &cChar, 1 );
         SetText( aTmp );
-        GetFont() = pSym->GetFace();
+        if (pSym->GetFace().GetFamilyName().isEmpty())
+            GetFont() = rFormat.GetFont(FNT_VARIABLE);
+        else
+            GetFont() = pSym->GetFace();
     }
     else
     {

Reply via email to