ICEpdf
  1. ICEpdf
  2. PDF-372

Portions of PDF/A document do not display

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.2.2
    • Fix Version/s: 4.3.4
    • Component/s: Core/Parsing
    • Labels:
      None
    • Environment:
      -

      Description

      The attached PDF is missing text.
      1. sf-10790.pdf
        125 kB
        Tyler Johnson
      1. screenshot-1.jpg
        111 kB

        Activity

        Tyler Johnson created issue -
        Tyler Johnson made changes -
        Field Original Value New Value
        Attachment screenshot-1.jpg [ 13940 ]
        Tyler Johnson made changes -
        Attachment sf-10790.pdf [ 13941 ]
        Tyler Johnson made changes -
        Summary Portions of PDF do not display Portions of PDF/A document do not display
        Salesforce Case [5007000000JOy4K]
        Hide
        Patrick Corless added a comment -

        There doesn't appear to be any obvious issues with the parsing of the file and the shapes array is ready to paint with properly mapped characters. The font is question is a CID font and there appears to be a lot of white space used to control layout. My best guess is that we aren't handling the white space correctly and throwing the layout off.

        Show
        Patrick Corless added a comment - There doesn't appear to be any obvious issues with the parsing of the file and the shapes array is ready to paint with properly mapped characters. The font is question is a CID font and there appears to be a lot of white space used to control layout. My best guess is that we aren't handling the white space correctly and throwing the layout off.
        Patrick Corless made changes -
        Fix Version/s 4.3 [ 10266 ]
        Patrick Corless made changes -
        Fix Version/s 5.0 [ 10314 ]
        Fix Version/s 4.3 [ 10266 ]
        Hide
        Patrick Corless added a comment -

        Some improvements in rendering for 4.3.3 but still some more work to complete.

        Show
        Patrick Corless added a comment - Some improvements in rendering for 4.3.3 but still some more work to complete.
        Patrick Corless made changes -
        Fix Version/s 4.3.4 [ 10341 ]
        Fix Version/s 5.0 [ 10314 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #31385 Wed Oct 10 11:25:20 MDT 2012 patrick.corless PDF-372 update how the content parser handles text scaling, new approach seems to address some corner cases that have been problematic for quite some time.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-4.3.3/icepdf/core/src/org/icepdf/core/util/ContentParser.java
        Hide
        Patrick Corless added a comment - - edited

        After quite a lot of debuging and false positives I might have finally addressed a long running issue with how we apply text scaling to text. The new approach only applies the horizontal scaling when at text showing operator is applied, TJ, Tj, ' and ". And once the string is drawn the CTM is restored to the pre horizontal scalling affine transform. This technique avoid numerous calls to applyTextScalling() in the previous implementation.
        Still a round of QA work to do but feel pretty confident with the fix.

        Show
        Patrick Corless added a comment - - edited After quite a lot of debuging and false positives I might have finally addressed a long running issue with how we apply text scaling to text. The new approach only applies the horizontal scaling when at text showing operator is applied, TJ, Tj, ' and ". And once the string is drawn the CTM is restored to the pre horizontal scalling affine transform. This technique avoid numerous calls to applyTextScalling() in the previous implementation. Still a round of QA work to do but feel pretty confident with the fix.
        Hide
        Patrick Corless added a comment - - edited

        Marking as resoloved. Will open again if QA proves otherwise.

        Show
        Patrick Corless added a comment - - edited Marking as resoloved. Will open again if QA proves otherwise.
        Migration made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Patrick Corless made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: