Part of the matching is via equivalents specified in FontSubstitutions.xml.
Here's an extract referenced below.
- Code: Select all
I've encountered the following three bugs in this process:
(1) FontSubstutitions.xml uses the lowercase whitespace and punctuation removed name of the font. If the document contains "Times New Roman" it is not matched to the equivalent replace element for "timesnewroman". Similarly "Arial" is not matched to "arial".
(2) When matched, the method searching PhysicalFonts for the substitution font also uses the short key, not the proper name used by PhysicalFonts. For example, if matching "arial" to a substitute it tries to find "freesans" in PhysicalFont's map instead of "Free Sans".
(3) On the system tested, the SubsFonts value is inclusive of the leading whitespace (eg. in the line above, the first token is "\n\t\tarial' instead of "arial" (seems odd that whitespace is included after unmarshalling). This means the first substitution always fails to match a font. As, by convention, the first token is usually the name of the font, this effectively means on systems where msttcorefonts are installed, the BestMatchingMapper fails to match the exact font. ie. it can't match "arial" to "arial" because the substitution is named "\n\t\tarial".
Tested with v2.7.1 on Linux 3.0.0-12-generic #20-Ubuntu SMP x86_64 GNU/Linux
I've worked around this but it's not elegant. I created a map in BestMatchingMapper of the PhysicalFonts' map keyed using the similar convention as in FontSubstitutions.xml and modified BestMatchingMapper to use this key when dealing with values in or from FontSubstitutions.xml. The patch for the work-around is attached.
When it works this is the result (on a Linux system with msttfcorefonts package):
- Code: Select all
[BestMatchingMapper] Mapped Wingdings --> OpenSymbol( file:/usr/share/fonts/truetype/openoffice/opens___.ttf
[BestMatchingMapper] Mapped Times New Roman --> Times New Roman( file:/usr/share/fonts/truetype/msttcorefonts/times.ttf
[BestMatchingMapper] Mapped Courier New --> Courier New( file:/usr/share/fonts/truetype/msttcorefonts/Courier_New.ttf
[BestMatchingMapper] Mapped Symbol --> Symbol( file:/usr/share/fonts/X11/Type1/Symbol.pfb
[BestMatchingMapper] Mapped Arial --> Arial( file:/usr/share/fonts/truetype/msttcorefonts/arial.ttf
PS. Patches are provided in accordance in accordance with the CLA http://dev.plutext.org/docx4j/docx4j_In ... butor.docx