svn commit: r744479 - /incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDDocument.java

[le...@apache.org]

Author: lehmi
Date: Sat Feb 14 10:51:11 2009
New Revision: 744479

URL: http://svn.apache.org/viewvc?rev=744479&view=rev
Log:
PDFBOX425: use the given printJob instead of creating a new one wihtin silentPrint. Thanks to Matthew Pangaro (mattp at lokitech dot com)

Modified:
    incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDDocument.java

Modified: incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDDocument.java
URL: http://svn.apache.org/viewvc/incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDDocument.java?rev=744479&r1=744478&r2=744479&view=diff
==============================================================================
--- incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDDocument.java (original)
+++ incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDDocument.java Sat Feb 14 10:51:11 2009
@@ -814,6 +814,10 @@
      */
     public void print(PrinterJob printJob) throws PrinterException
     {
+        if(printJob == null)
+        {
+            throw new PrinterException( "The delivered printJob is null." );
+        }
         AccessPermission currentPermissions = this.getCurrentAccessPermission();
 
         if(!currentPermissions.canPrint())
@@ -866,15 +870,18 @@
      *
      * @throws PrinterException If there is an error while printing.
      */
-    public void silentPrint( PrinterJob job ) throws PrinterException
+    public void silentPrint( PrinterJob printJob ) throws PrinterException
     {
+        if(printJob == null)
+        {
+            throw new PrinterException( "The delivered printJob is null." );
+        }
         AccessPermission currentPermissions = this.getCurrentAccessPermission();
 
         if(!currentPermissions.canPrint())
         {
             throw new PrinterException( "You do not have permission to print this document." );
         }
-        PrinterJob printJob = PrinterJob.getPrinterJob();
         printJob.setPageable(this);
         printJob.print();
     }