search

pcorless / icepdf

PDF Rendering and Viewing API in Java
Apache License 2.0
84 stars 21 forks source link

Error display font #340

Closed didiermaurel closed 9 months ago

didiermaurel commented 9 months ago

Good morning

I use icepdf 7.1.4 but the fonts are displayed incorrectly on one of my documents this document has the font ArialMT.ttf, Arial-BoldMT.ttf, Helvetica.ttf, Helvetica-Bold.ttf, Helvetica-Oblique.ttf and Helvetica-BoldOblique.ttf, and it displays well if I open it with the windows pdf reader

i have use readSystemFonts, readFontPackage, readFonts, FontPropertiesManager.getInstance().loadOrReadSystemFonts(); method; but it doesn't work

We must copy the fonts to c:/windows/Font (for me it's ttf) isn't it ? but even doing this it doesn't work. what I would have liked is to be able to load the font for icepdf from ttf files on the hard drive.

if i use readSystemFonts on my fonts I get this error

org.apache.fontbox.ttf.TTFParser parse AVERTISSEMENT: Skip table '' which goes past the file size; offset: 1146308935, size: 8580, font size: 13674828 févr. 13, 2024 10:48:42 AM org.icepdf.core.pobjects.fonts.zfont.fontFiles.ZFontTrueType AVERTISSEMENT: Error reading font file with java.io.IOException: 'head' table is mandatory at org.apache.fontbox.ttf.TTFParser.parseTables(TTFParser.java:198) at org.apache.fontbox.ttf.TTFParser.parse(TTFParser.java:165)

Thanks for your help

Sincerely

pcorless commented 9 months ago

Your call to FontPropertiesManager.getInstance().loadOrReadSystemFonts() will cache all the system fonts. But since you have tried a lot of different ways to load fonts there might be something else going on when decoding the file.

Do you have a sample file you could share publicly?

didiermaurel commented 9 months ago

I asked the laboratory for an identical but anonymized file I'll keep you informed

pcorless commented 9 months ago

7.2.0 was released last nigth and it has redaction support. It might be helpful for stripping any phi.

didiermaurel commented 9 months ago

thank you

At first glance it doesn't work yet with 7.2.0, I'm on vacation but my colleague will look into it further. As soon as I have the anonymized file I will send it to you.

Best regards

[cid:cf6cfa01-ebe5-489f-bc15-c0639e0f1f12]

De : Patrick Corless @.> Envoyé : vendredi 16 février 2024 17:06 À : pcorless/icepdf @.> Cc : Didier Maurel @.>; Author @.> Objet : Re: [pcorless/icepdf] Error display font (Issue #340)

Vous ne recevez pas souvent de courriers de la part de @.*** Découvrez pourquoi cela est importanthttps://aka.ms/LearnAboutSenderIdentification

7.2.0 was released last nigth and it has redaction support. It might be helpful for stripping any phi.

— Reply to this email directly, view it on GitHubhttps://github.com/pcorless/icepdf/issues/340#issuecomment-1948750032, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANLDAYFGDTRIMGB4HHDKGVDYT57ZTAVCNFSM6AAAAABDH2BLVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBYG42TAMBTGI. You are receiving this because you authored the thread.Message ID: @.***>

didiermaurel commented 9 months ago

Hello The laboratory sent me a new file but it was read correctly by ice pdf. I think I managed to anonymize the problem file but I wouldn't really like to send it to the github forum. What personal email address can I send it to?

pcorless commented 9 months ago

Pretty confident that I have this figured out. Turns out I have a few similar examples in my test suite that use the same cmap encoding flag to specify one byte character codes. The issue was without processing the flag the string parsers will treat string as multi-byte and hence the garbled text rendering. There are no regression with this fix and I I have 4 test files that now render correctly.

I can get this out in a 7.2.1 release but that likely won't be for another week or so as I'd like to wait a bit longer for any other tickets to come in.

didiermaurel commented 9 months ago

ok super we will wait for this 7.2.1 version, tell us when it will be available

pcorless commented 9 months ago

Merged into master. Closing for now.

didiermaurel commented 3 months ago

Hello Patrick

in February on this post

https://github.com/pcorless/icepdf/issues/340

you told me that the new version 7.2.1 would arrive soon but it is still not there can you tell me more ? Otherwise on version 7.2.0 I load a pdf but the getpagetree() is null whereas on older versions of icepdf it works where can this come from?

Thanking you in advance

Kind regards

Didier Maurel

[https://opengraph.githubassets.com/4b4971da0339614b875f398d8c0b05646b0c8fe9543c0f65cacc781900092110/pcorless/icepdf/issues/340]https://github.com/pcorless/icepdf/issues/340 Error display font · Issue #340 · pcorless/icepdfhttps://github.com/pcorless/icepdf/issues/340 Good morning I use icepdf 7.1.4 but the fonts are displayed incorrectly on one of my documents this document has the font ArialMT.ttf, Arial-BoldMT.ttf, Helvetica.ttf, Helvetica-Bold.ttf, Helvetica... github.com

[cid:0d2f466a-7d44-4626-b846-1289f5740b1c]

De : Patrick Corless @.> Envoyé : vendredi 23 février 2024 05:58 À : pcorless/icepdf @.> Cc : Didier Maurel @.>; Author @.> Objet : Re: [pcorless/icepdf] Error display font (Issue #340)

Vous ne recevez pas souvent de courriers de la part de @.*** Découvrez pourquoi cela est importanthttps://aka.ms/LearnAboutSenderIdentification

Merged into master. Closing for now.

— Reply to this email directly, view it on GitHubhttps://github.com/pcorless/icepdf/issues/340#issuecomment-1960733588, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANLDAYEYZG6PZGYCHIR24S3YVAOW3AVCNFSM6AAAAABDH2BLVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRQG4ZTGNJYHA. You are receiving this because you authored the thread.Message ID: @.***>

ctoabidmaqbool1 commented 2 months ago

@didiermaurel ICEPdf new version was released: