Font alias names can be used case-insensitive (#1576) (#1578)

eclipse-birt · Mar 4, 2024 · 03dc43e · 03dc43e
1 parent b1d4f7d
commit 03dc43e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/...ngine.tests/test/org/eclipse/birt/report/engine/layout/pdf/font/FontConfigReaderTest.java b/...ngine.tests/test/org/eclipse/birt/report/engine/layout/pdf/font/FontConfigReaderTest.java
@@ -77,7 +77,8 @@ public void testFontMapWhenAllFontsNotDefined()
 		// alias: defined; composite-font: not; character: defined by the
 		// logical font.
 		assertTrue(isMappedTo('1', "alias2", "Helvetica"));
-
+		// The Mapping should be case-insensitive
+		assertTrue(isMappedTo('1', "Alias2", "Helvetica"));
 		// alias: not; composite-font: defined; block: defined; character:
 		// defined by the block font.
 		assertTrue(isMappedTo('1', "Symbol", "Courier"));

diff --git a/....report.engine/src/org/eclipse/birt/report/engine/layout/pdf/font/FontMappingManager.java b/....report.engine/src/org/eclipse/birt/report/engine/layout/pdf/font/FontMappingManager.java
@@ -122,7 +122,7 @@ public String getDefaultPhysicalFont(char c) {
 	}
 
 	public String getAliasedFont(String fontAlias) {
-		String alias = (String) fontAliases.get(fontAlias);
+		String alias = (String) fontAliases.get(fontAlias.toLowerCase());
 		if (alias != null) {
 			return alias;
 		}