ICEpdf
  1. ICEpdf
  2. PDF-496

Error processing tilling pattern that are smaller then one pixel

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.3.3, 4.3.4
    • Fix Version/s: 4.4, 5.0.0 alpha1, 5.0
    • Component/s: Core/Parsing
    • Labels:
      None
    • Environment:
      any

      Description

      A potential client sent in a PDF that had missing content on a large vector diagram. After closer inspection it turned out to be an issue with our tiling pattern. Putting sample on support drive.

        Issue Links

          Activity

          Patrick Corless made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Patrick Corless made changes -
          Fix Version/s 4.4 [ 10670 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #33832 Thu Mar 07 13:03:13 MST 2013 patrick.corless PDF-496 added check to make sure the tile isn't painted if it has now shapes.
          Files Changed
          Commit graph MODIFY /icepdf/branches/icepdf-4.3.3/icepdf/core/src/org/icepdf/core/pobjects/graphics/TilingPattern.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #33817 Thu Mar 07 07:35:07 MST 2013 patrick.corless PDF-496 updated TilingPattern class to return the bbox transformed into user space and updated the content parser to avoid painted tiles that have a dimension less then 1x1.
          Files Changed
          Commit graph MODIFY /icepdf/branches/icepdf-4.3.3/icepdf/core/src/org/icepdf/core/util/ContentParser.java
          Commit graph MODIFY /icepdf/branches/icepdf-4.3.3/icepdf/core/src/org/icepdf/core/pobjects/graphics/TilingPattern.java
          Patrick Corless made changes -
          Fix Version/s 5.0.0 alpha1 [ 10676 ]
          Patrick Corless made changes -
          Link This issue depends on PDF-439 [ PDF-439 ]
          Hide
          Patrick Corless added a comment -

          Similar case, valuable for regression tests.

          Show
          Patrick Corless added a comment - Similar case, valuable for regression tests.
          Patrick Corless made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 5.0 [ 10314 ]
          Resolution Fixed [ 1 ]
          Hide
          Patrick Corless added a comment -

          Updated the TilingPattern class so that the bbox converted to users-space is calculated during to the init method and available to the content parser during parsing. We know only try to use a tilled paint if the bounds are larger the 1x1 pixel.

          Show
          Patrick Corless added a comment - Updated the TilingPattern class so that the bbox converted to users-space is calculated during to the init method and available to the content parser during parsing. We know only try to use a tilled paint if the bounds are larger the 1x1 pixel.
          Patrick Corless created issue -
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #32743 Tue Dec 11 13:54:06 MST 2012 patrick.corless PDF-496 updated TilingPattern class to return the bbox transformed into user space and updated the content parser to avoid painted tiles that have a dimension less then 1x1.
          Files Changed
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/commands/ColorDrawCmd.java
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/ContentParser.java
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/TilingPattern.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #29229 Tue May 29 07:51:15 MDT 2012 patrick.corless PDF-496 removed JAI reference that broke the build.
          Files Changed
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #29216 Mon May 28 14:01:43 MDT 2012 patrick.corless PDF-496 removed transparency tint operand from content parser because of regressions. Replaced logic with adding alpha value to new colour creation which gives a mores consistent effect.
          Files Changed
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/ContentParser.java
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #29212 Mon May 28 11:00:10 MDT 2012 patrick.corless PDF-496 reworked Separation colour space to use any valid named colour before trying the alternative colour space. Also added a hook in the content parser to apply a tint value if specified in the Seperation colour space.
          Files Changed
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/ContentParser.java
          Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java

            People

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

              Dates

              • Created:
                Updated:
                Resolved: