[ 
https://issues.apache.org/jira/browse/PDFBOX-2786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14542761#comment-14542761
 ] 

John Hewson edited comment on PDFBOX-2786 at 5/13/15 9:27 PM:
--------------------------------------------------------------

I don't really see it as the counterpart of setPageNumber() anyway. 
setPageNumber() sets the page number. getPageNumber() _might_ return the page 
number, or it might not, so the behaviour isn't complementary. What's the point 
in getPageNumber(), why would anybody ever want to call it rather than 
retrievePageNumber()? Only someone interested in COS would do that... and we 
already have COS for that. What is the problem that getPageNumber() solves?


was (Author: jahewson):
I don't really see it as the counterpart of setPageNumber() anyway. 
setPageNumber() sets the page number. getPageNumber() _might_ return the page 
number, or it might not, so the behaviour isn't complementary. What's the point 
in getPageNumber(), why would anybody ever want to call it rather than 
retrievePageNumber()? Only someone interested in COS would do that... and we 
already have COS for that.

> PDPageDestination page index off by one
> ---------------------------------------
>
>                 Key: PDFBOX-2786
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-2786
>             Project: PDFBox
>          Issue Type: Bug
>          Components: PDModel
>    Affects Versions: 1.8.9, 2.0.0
>            Reporter: Johanneke Lamberink
>            Assignee: Tilman Hausherr
>             Fix For: 1.8.10, 2.0.0
>
>         Attachments: Archive.zip
>
>
> When creating a new bookmark with the same page number as an existing 
> bookmark, the resulting destination is offset by 1 compared to the old 
> destination.
> This results in the bookmark being set for the next page, which could be a 
> non-existing page.
> I've added a class with an example pdf and my own output pdf. Run with 
> argument of a path to where you have the pdf, including a trailing slash.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to