ICEpdf
  1. ICEpdf
  2. PDF-176

Total Number Of Pages to print incorrect

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.1
    • Fix Version/s: 4.1
    • Component/s: Viewer RI
    • Labels:
      None
    • Environment:
      Windows XP, JDK1.6.0_20

      Description

      When i select "all" as print range in the print dialog the print progress dialog shows a total number of Integer.MAX_VALUE

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Patrick Corless added a comment -

        Thanks for the bug report. I updated the page range iteration to check each range for valid page values and correct accordingly. Seams to work well enough.

        Show
        Patrick Corless added a comment - Thanks for the bug report. I updated the page range iteration to check each range for valid page values and correct accordingly. Seams to work well enough.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21786 Tue Jun 22 11:56:39 MDT 2010 patrick.corless PDF-176 updated print helper to correctly handle incorrect page range values as well as fixed an issue where we didn't show the max number of pages when printing "all" pages.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/viewer/src/org/icepdf/ri/common/PrintHelper.java
        Patrick Corless made changes -
        Fix Version/s 4.1 [ 10227 ]
        Fix Version/s 4.0.2 [ 10234 ]
        Tyler Johnson made changes -
        Salesforce Case [5007000000C2Pr5]
        Patrick Corless made changes -
        Salesforce Case []
        Fix Version/s 4.0.2 [ 10234 ]
        Dirk Maass made changes -
        Field Original Value New Value
        Attachment icepdf4.0.1-PrintHelper-fixTotalPagesToPrint.patch [ 12320 ]
        Hide
        Dirk Maass added a comment -

        This patch circumvents the problem, but wouldn't it be better to check this during the iteration over page ranges?

        Show
        Dirk Maass added a comment - This patch circumvents the problem, but wouldn't it be better to check this during the iteration over page ranges?
        Dirk Maass created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved: