You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pdfbox.apache.org by "Tilman Hausherr (JIRA)" <ji...@apache.org> on 2014/05/29 11:50:01 UTC

[jira] [Updated] (PDFBOX-1596) OverlayPDF logic should be moved into a library class

     [ https://issues.apache.org/jira/browse/PDFBOX-1596?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tilman Hausherr updated PDFBOX-1596:
------------------------------------

    Description: 
We're finding that the OverlayPDF command line utility fixes problems observed when using the Overlay class. However, with the Overlay class we can operate it as a library call, but the OverlayPDF command line utility can't be, particularly given we work with PDDocuments.

I've just had someone copy the entire OverlayPDF class into one of our own and modify it to accept pre-loaded PDDocuments instead of filenames. It would be far more constructive to extract the core logic into a library class and have the CLI class drive it.

If we had a proper clone method it would be good to create a completely new PDDocument to serve as the new returned document instead of mutating the inputs too...


  was:
We're finding that OverlayPDF fixes problems observed when using Overlay. However, with Overlay we can operate it as a library call but OverlayPDF can't be particularly given we work with PDDocuments.

I've just had someone copy the entire OverlayPDF class into one of our own and modify it to accept pre-loaded PDDocuments instead of filenames. It would be far more constructive to extract the core logic into a library class and have the CLI class drive it.

If we had a proper clone method it would be good to create a completely new PDDocument to serve as the new returned document instead of mutating the inputs too...



> OverlayPDF logic should be moved into a library class
> -----------------------------------------------------
>
>                 Key: PDFBOX-1596
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-1596
>             Project: PDFBox
>          Issue Type: Improvement
>          Components: Utilities
>    Affects Versions: 1.8.1
>            Reporter: James Green
>            Priority: Minor
>         Attachments: PDFBOX-1596.patch
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> We're finding that the OverlayPDF command line utility fixes problems observed when using the Overlay class. However, with the Overlay class we can operate it as a library call, but the OverlayPDF command line utility can't be, particularly given we work with PDDocuments.
> I've just had someone copy the entire OverlayPDF class into one of our own and modify it to accept pre-loaded PDDocuments instead of filenames. It would be far more constructive to extract the core logic into a library class and have the CLI class drive it.
> If we had a proper clone method it would be good to create a completely new PDDocument to serve as the new returned document instead of mutating the inputs too...



--
This message was sent by Atlassian JIRA
(v6.2#6252)