Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement [pdffile]_updates.plist in pdf parser #141

Closed
libreliodev opened this issue Mar 19, 2015 · 7 comments
Closed

Implement [pdffile]_updates.plist in pdf parser #141

libreliodev opened this issue Mar 19, 2015 · 7 comments
Labels

Comments

@libreliodev
Copy link
Owner

This is related to #140.

@AlirezaAlgo
Copy link
Collaborator

@libreliodev
This update plist file refers to existing annotations using their obj ID.
I reviewed WAPDFParser.m. Mainly CGPDF is used for accessing pdf document.

There's no method for accessing annotation object id with CGPDF.

  • Maybe i didn't found the api yet.
  • We should use another api for accessing obj ID.

Look an annotation example.

486 0 obj
<< /A 507 0 R /Border [ 0 0 0 ] /Type /Annot /Subtype /Link /Rect [ 85.3199
41.8132 574.255 438.555 ] >>
endobj
507 0 obj
<< /S /URI /URI (buy://localhost/wind_376/wind_376_.pdf) >>
endobj

@libreliodev
Copy link
Owner Author

There's no method for accessing annotation object id with CGPDF.

This seems to be exact: http://stackoverflow.com/questions/13861672/cgpdfobject-get-id

Let's think about it.

@libreliodev
Copy link
Owner Author

@serverfire I think the best way to solve this problem would be to use the annotation number on a given page, instead of object id. This would require some changes in libreliodev/javascript#128 . What do you think?

@AlirezaAlgo
Copy link
Collaborator

@libreliodev If you mean annotation index in a page by Annotation number.
We have to test it in javascript project too. Not all systems may have the same order.

@libreliodev
Copy link
Owner Author

@serverfire

We have to test it in javascript project too. Not all systems may not have the same order.

I agree.

@AlirezaAlgo
Copy link
Collaborator

@libreliodev Done with 324f514 & b9005d2

Tested with developer/wind/wind_384/wind_384.pdf.
And I've changed pdf-annotation-editor file. I'll commit changes shortly.

@libreliodev
Copy link
Owner Author

Successfully tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants