mirror of
https://github.com/Frooodle/Stirling-PDF.git
synced 2025-11-16 01:21:16 +01:00
refactor(pdf-conversion): enhance font embedding and normalization process
- Updated font embedding settings to avoid incomplete glyphs and allow substitutions - Added high-quality prepress settings for better content preservation - Introduced Type1 font CharSet fixing to handle missing or invalid definitions - Implemented QPDF-based PDF normalization to address font program issues - Enhanced cleanup logic to manage temporary files from normalization steps Signed-off-by: Balázs Szücs <bszucs1209@gmail.com>
This commit is contained in:
Balázs Szücs
parent
796f739fde
commit
e7a15579ea
@ -172,8 +172,10 @@ public class ConvertPDFToPDFA {
|
|||||||
command.add("-sDefaultRGBProfile=" + colorProfiles.rgb().toAbsolutePath());
|
command.add("-sDefaultRGBProfile=" + colorProfiles.rgb().toAbsolutePath());
|
||||||
command.add("-sDefaultGrayProfile=" + colorProfiles.gray().toAbsolutePath());
|
command.add("-sDefaultGrayProfile=" + colorProfiles.gray().toAbsolutePath());
|
||||||
command.add("-dEmbedAllFonts=true");
|
command.add("-dEmbedAllFonts=true");
|
||||||
command.add("-dSubsetFonts=true");
|
command.add("-dSubsetFonts=false"); // Embed complete fonts to avoid incomplete glyphs
|
||||||
command.add("-dCompressFonts=true");
|
command.add("-dCompressFonts=true");
|
||||||
|
command.add("-dNOSUBSTFONTS=false"); // Allow font substitution for problematic fonts
|
||||||
|
command.add("-dPDFSETTINGS=/prepress"); // High quality, preserves all content
|
||||||
command.add("-dNOPAUSE");
|
command.add("-dNOPAUSE");
|
||||||
command.add("-dBATCH");
|
command.add("-dBATCH");
|
||||||
command.add("-dNOOUTERSAVE");
|
command.add("-dNOOUTERSAVE");
|
||||||
@ -337,8 +339,10 @@ public class ConvertPDFToPDFA {
|
|||||||
command.add("-sDefaultRGBProfile=" + colorProfiles.rgb().toAbsolutePath());
|
command.add("-sDefaultRGBProfile=" + colorProfiles.rgb().toAbsolutePath());
|
||||||
command.add("-sDefaultGrayProfile=" + colorProfiles.gray().toAbsolutePath());
|
command.add("-sDefaultGrayProfile=" + colorProfiles.gray().toAbsolutePath());
|
||||||
command.add("-dEmbedAllFonts=true");
|
command.add("-dEmbedAllFonts=true");
|
||||||
command.add("-dSubsetFonts=true");
|
command.add("-dSubsetFonts=false"); // Embed complete fonts to avoid incomplete glyphs
|
||||||
command.add("-dCompressFonts=true");
|
command.add("-dCompressFonts=true");
|
||||||
|
command.add("-dNOSUBSTFONTS=false"); // Allow font substitution for problematic fonts
|
||||||
|
command.add("-dPDFSETTINGS=/prepress");
|
||||||
command.add("-dNOPAUSE");
|
command.add("-dNOPAUSE");
|
||||||
command.add("-dBATCH");
|
command.add("-dBATCH");
|
||||||
command.add("-dNOOUTERSAVE");
|
command.add("-dNOOUTERSAVE");
|
||||||
@ -627,6 +631,260 @@ public class ConvertPDFToPDFA {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static void fixType1FontCharSet(PDDocument document) throws IOException {
|
||||||
|
for (PDPage page : document.getPages()) {
|
||||||
|
PDResources resources = page.getResources();
|
||||||
|
if (resources == null) continue;
|
||||||
|
|
||||||
|
for (COSName fontName : resources.getFontNames()) {
|
||||||
|
try {
|
||||||
|
PDFont font = resources.getFont(fontName);
|
||||||
|
if (font == null) continue;
|
||||||
|
|
||||||
|
String fontNameStr = font.getName();
|
||||||
|
if (fontNameStr == null) continue;
|
||||||
|
|
||||||
|
PDFontDescriptor descriptor = font.getFontDescriptor();
|
||||||
|
if (descriptor == null) continue;
|
||||||
|
|
||||||
|
// Check if this is a Type1 font
|
||||||
|
if (fontNameStr.contains("Type1")
|
||||||
|
|| descriptor.getFontFile() != null
|
||||||
|
|| (descriptor.getFontFile2() == null
|
||||||
|
&& descriptor.getFontFile3() == null)) {
|
||||||
|
|
||||||
|
// Check if CharSet is missing or suspicious
|
||||||
|
String existingCharSet =
|
||||||
|
descriptor.getCOSObject().getString(COSName.CHAR_SET);
|
||||||
|
if (existingCharSet == null || existingCharSet.trim().isEmpty()) {
|
||||||
|
|
||||||
|
// Build a CharSet from commonly used glyphs
|
||||||
|
// For Type1 fonts, include standard PDF glyphs
|
||||||
|
String glyphSet = buildStandardType1GlyphSet();
|
||||||
|
if (!glyphSet.isEmpty()) {
|
||||||
|
descriptor.getCOSObject().setString(COSName.CHAR_SET, glyphSet);
|
||||||
|
log.debug(
|
||||||
|
"Fixed CharSet for Type1 font {} with {} glyphs",
|
||||||
|
fontNameStr,
|
||||||
|
glyphSet.split(" ").length);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
log.warn(
|
||||||
|
"Error processing font descriptor for page resource: {}",
|
||||||
|
e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private static String buildStandardType1GlyphSet() {
|
||||||
|
// Standard PDF glyph names for Type1 fonts
|
||||||
|
Set<String> glyphNames = new LinkedHashSet<>();
|
||||||
|
|
||||||
|
// Add common Type1 glyphs from standard encoding
|
||||||
|
String[] standardGlyphs = {
|
||||||
|
".notdef",
|
||||||
|
".null",
|
||||||
|
"nonmarkingreturn",
|
||||||
|
"space",
|
||||||
|
"exclam",
|
||||||
|
"quotedbl",
|
||||||
|
"numbersign",
|
||||||
|
"dollar",
|
||||||
|
"percent",
|
||||||
|
"ampersand",
|
||||||
|
"quoteright",
|
||||||
|
"parenleft",
|
||||||
|
"parenright",
|
||||||
|
"asterisk",
|
||||||
|
"plus",
|
||||||
|
"comma",
|
||||||
|
"hyphen",
|
||||||
|
"period",
|
||||||
|
"slash",
|
||||||
|
"zero",
|
||||||
|
"one",
|
||||||
|
"two",
|
||||||
|
"three",
|
||||||
|
"four",
|
||||||
|
"five",
|
||||||
|
"six",
|
||||||
|
"seven",
|
||||||
|
"eight",
|
||||||
|
"nine",
|
||||||
|
"colon",
|
||||||
|
"semicolon",
|
||||||
|
"less",
|
||||||
|
"equal",
|
||||||
|
"greater",
|
||||||
|
"question",
|
||||||
|
"at",
|
||||||
|
"A",
|
||||||
|
"B",
|
||||||
|
"C",
|
||||||
|
"D",
|
||||||
|
"E",
|
||||||
|
"F",
|
||||||
|
"G",
|
||||||
|
"H",
|
||||||
|
"I",
|
||||||
|
"J",
|
||||||
|
"K",
|
||||||
|
"L",
|
||||||
|
"M",
|
||||||
|
"N",
|
||||||
|
"O",
|
||||||
|
"P",
|
||||||
|
"Q",
|
||||||
|
"R",
|
||||||
|
"S",
|
||||||
|
"T",
|
||||||
|
"U",
|
||||||
|
"V",
|
||||||
|
"W",
|
||||||
|
"X",
|
||||||
|
"Y",
|
||||||
|
"Z",
|
||||||
|
"bracketleft",
|
||||||
|
"backslash",
|
||||||
|
"bracketright",
|
||||||
|
"asciicircum",
|
||||||
|
"underscore",
|
||||||
|
"quoteleft",
|
||||||
|
"a",
|
||||||
|
"b",
|
||||||
|
"c",
|
||||||
|
"d",
|
||||||
|
"e",
|
||||||
|
"f",
|
||||||
|
"g",
|
||||||
|
"h",
|
||||||
|
"i",
|
||||||
|
"j",
|
||||||
|
"k",
|
||||||
|
"l",
|
||||||
|
"m",
|
||||||
|
"n",
|
||||||
|
"o",
|
||||||
|
"p",
|
||||||
|
"q",
|
||||||
|
"r",
|
||||||
|
"s",
|
||||||
|
"t",
|
||||||
|
"u",
|
||||||
|
"v",
|
||||||
|
"w",
|
||||||
|
"x",
|
||||||
|
"y",
|
||||||
|
"z",
|
||||||
|
"braceleft",
|
||||||
|
"bar",
|
||||||
|
"braceright",
|
||||||
|
"asciitilde",
|
||||||
|
"exclamdown",
|
||||||
|
"cent",
|
||||||
|
"sterling",
|
||||||
|
"currency",
|
||||||
|
"yen",
|
||||||
|
"brokenbar",
|
||||||
|
"section",
|
||||||
|
"dieresis",
|
||||||
|
"copyright",
|
||||||
|
"ordfeminine",
|
||||||
|
"guillemotleft",
|
||||||
|
"logicalnot",
|
||||||
|
"uni00AD",
|
||||||
|
"registered",
|
||||||
|
"macron",
|
||||||
|
"degree",
|
||||||
|
"plusminus",
|
||||||
|
"twosuperior",
|
||||||
|
"threesuperior",
|
||||||
|
"acute",
|
||||||
|
"mu",
|
||||||
|
"paragraph",
|
||||||
|
"periodcentered",
|
||||||
|
"cedilla",
|
||||||
|
"onesuperior",
|
||||||
|
"ordmasculine",
|
||||||
|
"guillemotright",
|
||||||
|
"onequarter",
|
||||||
|
"onehalf",
|
||||||
|
"threequarters",
|
||||||
|
"questiondown",
|
||||||
|
"Agrave",
|
||||||
|
"Aacute",
|
||||||
|
"Acircumflex",
|
||||||
|
"Atilde",
|
||||||
|
"Adieresis",
|
||||||
|
"Aring",
|
||||||
|
"AE",
|
||||||
|
"Ccedilla",
|
||||||
|
"Egrave",
|
||||||
|
"Eacute",
|
||||||
|
"Ecircumflex",
|
||||||
|
"Edieresis",
|
||||||
|
"Igrave",
|
||||||
|
"Iacute",
|
||||||
|
"Icircumflex",
|
||||||
|
"Idieresis",
|
||||||
|
"Eth",
|
||||||
|
"Ntilde",
|
||||||
|
"Ograve",
|
||||||
|
"Oacute",
|
||||||
|
"Ocircumflex",
|
||||||
|
"Otilde",
|
||||||
|
"Odieresis",
|
||||||
|
"multiply",
|
||||||
|
"Oslash",
|
||||||
|
"Ugrave",
|
||||||
|
"Uacute",
|
||||||
|
"Ucircumflex",
|
||||||
|
"Udieresis",
|
||||||
|
"Yacute",
|
||||||
|
"Thorn",
|
||||||
|
"germandbls",
|
||||||
|
"agrave",
|
||||||
|
"aacute",
|
||||||
|
"acircumflex",
|
||||||
|
"atilde",
|
||||||
|
"adieresis",
|
||||||
|
"aring",
|
||||||
|
"ae",
|
||||||
|
"ccedilla",
|
||||||
|
"egrave",
|
||||||
|
"eacute",
|
||||||
|
"ecircumflex",
|
||||||
|
"edieresis",
|
||||||
|
"igrave",
|
||||||
|
"iacute",
|
||||||
|
"icircumflex",
|
||||||
|
"idieresis",
|
||||||
|
"eth",
|
||||||
|
"ntilde",
|
||||||
|
"ograve",
|
||||||
|
"oacute",
|
||||||
|
"ocircumflex",
|
||||||
|
"otilde",
|
||||||
|
"odieresis",
|
||||||
|
"divide",
|
||||||
|
"oslash",
|
||||||
|
"ugrave",
|
||||||
|
"uacute",
|
||||||
|
"ucircumflex",
|
||||||
|
"udieresis",
|
||||||
|
"yacute",
|
||||||
|
"thorn",
|
||||||
|
"ydieresis"
|
||||||
|
};
|
||||||
|
|
||||||
|
Collections.addAll(glyphNames, standardGlyphs);
|
||||||
|
|
||||||
|
return String.join(" ", glyphNames);
|
||||||
|
}
|
||||||
|
|
||||||
private byte[] processWithPDFBox(PDDocument document, int pdfaPart) throws Exception {
|
private byte[] processWithPDFBox(PDDocument document, int pdfaPart) throws Exception {
|
||||||
|
|
||||||
removeElementsForPdfA(document, pdfaPart);
|
removeElementsForPdfA(document, pdfaPart);
|
||||||
@ -635,6 +893,8 @@ public class ConvertPDFToPDFA {
|
|||||||
|
|
||||||
addICCProfileIfNotPresent(document);
|
addICCProfileIfNotPresent(document);
|
||||||
|
|
||||||
|
fixType1FontCharSet(document);
|
||||||
|
|
||||||
// Mark the document as PDF/A
|
// Mark the document as PDF/A
|
||||||
PDDocumentCatalog catalog = document.getDocumentCatalog();
|
PDDocumentCatalog catalog = document.getDocumentCatalog();
|
||||||
catalog.setMetadata(
|
catalog.setMetadata(
|
||||||
@ -1099,16 +1359,60 @@ public class ConvertPDFToPDFA {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private Path normalizePdfWithQpdf(Path inputPdf) {
|
||||||
|
try {
|
||||||
|
ProcessExecutorResult checkResult =
|
||||||
|
ProcessExecutor.getInstance(ProcessExecutor.Processes.GHOSTSCRIPT)
|
||||||
|
.runCommandWithOutputHandling(Arrays.asList("qpdf", "--version"));
|
||||||
|
|
||||||
|
if (checkResult.getRc() != 0) {
|
||||||
|
log.debug("QPDF not available");
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
Path normalizedPdf =
|
||||||
|
inputPdf.getParent().resolve("normalized_" + inputPdf.getFileName().toString());
|
||||||
|
|
||||||
|
List<String> command =
|
||||||
|
Arrays.asList(
|
||||||
|
"qpdf",
|
||||||
|
"--normalize-content=y",
|
||||||
|
"--object-streams=preserve",
|
||||||
|
inputPdf.toAbsolutePath().toString(),
|
||||||
|
normalizedPdf.toAbsolutePath().toString());
|
||||||
|
|
||||||
|
ProcessExecutorResult result =
|
||||||
|
ProcessExecutor.getInstance(ProcessExecutor.Processes.GHOSTSCRIPT)
|
||||||
|
.runCommandWithOutputHandling(command);
|
||||||
|
|
||||||
|
if (result.getRc() == 0 && Files.exists(normalizedPdf)) {
|
||||||
|
log.info("PDF normalized with QPDF to fix font programs");
|
||||||
|
return normalizedPdf;
|
||||||
|
}
|
||||||
|
return null;
|
||||||
|
|
||||||
|
} catch (Exception e) {
|
||||||
|
log.debug("QPDF normalization error: {}", e.getMessage());
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
private byte[] convertWithPdfBoxMethod(Path inputPath, PdfaProfile profile) throws Exception {
|
private byte[] convertWithPdfBoxMethod(Path inputPath, PdfaProfile profile) throws Exception {
|
||||||
Path tempInputFile = null;
|
Path tempInputFile = null;
|
||||||
byte[] fileBytes;
|
byte[] fileBytes;
|
||||||
Path loPdfPath = null;
|
Path loPdfPath = null;
|
||||||
File preProcessedFile = null;
|
File preProcessedFile = null;
|
||||||
int pdfaPart = profile.getPart();
|
int pdfaPart = profile.getPart();
|
||||||
|
Path normalizedPath = null;
|
||||||
|
|
||||||
try {
|
try {
|
||||||
tempInputFile = inputPath;
|
tempInputFile = inputPath;
|
||||||
|
|
||||||
|
normalizedPath = normalizePdfWithQpdf(tempInputFile);
|
||||||
|
if (normalizedPath != null) {
|
||||||
|
tempInputFile = normalizedPath;
|
||||||
|
}
|
||||||
|
|
||||||
if (pdfaPart == 2 || pdfaPart == 3) {
|
if (pdfaPart == 2 || pdfaPart == 3) {
|
||||||
preProcessedFile = tempInputFile.toFile();
|
preProcessedFile = tempInputFile.toFile();
|
||||||
} else {
|
} else {
|
||||||
@ -1137,6 +1441,9 @@ public class ConvertPDFToPDFA {
|
|||||||
if (preProcessedFile != null && !preProcessedFile.equals(tempInputFile.toFile())) {
|
if (preProcessedFile != null && !preProcessedFile.equals(tempInputFile.toFile())) {
|
||||||
Files.deleteIfExists(preProcessedFile.toPath());
|
Files.deleteIfExists(preProcessedFile.toPath());
|
||||||
}
|
}
|
||||||
|
if (normalizedPath != null && !normalizedPath.equals(inputPath)) {
|
||||||
|
Files.deleteIfExists(normalizedPath);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user