ICEpdf
  1. ICEpdf
  2. PDF-1006

Page.getPageBoundary() defaults to MEDIABOX instead of CROPBOX in case of bad usage

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.1.2
    • Fix Version/s: 6.1.3
    • Component/s: Documentation
    • Labels:
      None
    • Environment:
      ICEpdf 6.1.2 Pro
      Java SE 8u92 x64
      Windows 10 x64

      Description

      Comments in org.icepdf.core.pobjects.Page indicate that getPageBoundary(...) is supposed to check CROPBOX boundary in case of bad usage, but it actually always returns MEDIABOX: http://sventon.icesoft.org/svn/repos/repo/show/icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/Page.java?revision=HEAD#L1200

      If the intended behavior was to fall back to CROPBOX, then code and documentation can be adjusted so: https://gist.github.com/fragmer/ba470cb0f523eba63d53a7f9230e2dfa/revisions

      Otherwise, if the intended behavior was to fall back to MEDIABOX, then code and documentation can be adjusted so: https://gist.github.com/fragmer/09a58e5f50906afc88d6d889f2c46d63/revisions

      (I know this is a pretty small problem, but it's also very easy to fix :))

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #49511 Tue Nov 08 12:35:12 MST 2016 patrick.corless PDF-1006 updated page.getBoundary() rules to return BOUNDARY_CROPBOX if no matching type can found as java documentation specifies.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/Page.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #49424 Fri Oct 28 09:23:12 MDT 2016 patrick.corless PDF-1006 updated page.getBoundary() rules to return BOUNDARY_CROPBOX if no matching type can found as java documentation specifies.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-6.1.0/icepdf/core/src/org/icepdf/core/pobjects/Page.java

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Matvei Stefarov
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: