From 39491e6ea46e59e5ddaec81120295785d480220e Mon Sep 17 00:00:00 2001
From: Glen Mazza <gmazza@apache.org>
Date: Sat, 2 Aug 2003 21:26:53 +0000
Subject: [PATCH] PrinterJob setup and construction moved from Driver to
 AWTPrintRenderer class.  AWTPrintRenderer now be instantiated without a
 PrinterJob object.  (Pending: for embedded programming, may still wish to
 have a PrinterJob-based constructor for AWTPrintRenderer, or otherwise
 provide an accessor method to its PrinterJob object.)

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@196770 13f79535-47bb-0310-9956-ffa450edef68
---
 src/java/org/apache/fop/apps/Driver.java      | 15 ++-------
 .../fop/render/awt/AWTPrintRenderer.java      | 31 ++++++++++---------
 .../apache/fop/servlet/FopPrintServlet.java   | 10 ++----
 3 files changed, 21 insertions(+), 35 deletions(-)

diff --git a/src/java/org/apache/fop/apps/Driver.java b/src/java/org/apache/fop/apps/Driver.java
index acf863a54..8ea305751 100644
--- a/src/java/org/apache/fop/apps/Driver.java
+++ b/src/java/org/apache/fop/apps/Driver.java
@@ -50,9 +50,6 @@
  */
 package org.apache.fop.apps;
 
-// Java
-import java.awt.print.PrinterJob;
-
 // FOP
 import org.apache.fop.fo.ElementMapping;
 import org.apache.fop.fo.FOTreeBuilder;
@@ -391,6 +388,7 @@ public class Driver implements LogEnabled {
      * <ul>
      * <li>RENDER_PDF</li>
      * <li>RENDER_AWT</li>
+     * <li>RENDER_PRINT</li>
      * <li>RENDER_MIF</li>
      * <li>RENDER_XML</li>
      * <li>RENDER_PCL</li>
@@ -411,16 +409,7 @@ public class Driver implements LogEnabled {
         case RENDER_AWT:
             throw new IllegalArgumentException("Use renderer form of setRenderer() for AWT");
         case RENDER_PRINT:
-            // a PrinterJob object is needed to create this renderer
-            PrinterJob pj = PrinterJob.getPrinterJob();
-            int copies = AWTPrintRenderer.getIntProperty("copies", 1);
-            pj.setCopies(copies);
-            if (System.getProperty("dialog") != null) {
-                if (!pj.printDialog()) {
-                    throw new IllegalArgumentException("Printing cancelled by operator");
-                }
-            }
-            setRenderer(new AWTPrintRenderer(pj));
+            setRenderer("org.apache.fop.render.awt.AWTPrintRenderer");
             break;
         case RENDER_PCL:
             setRenderer("org.apache.fop.render.pcl.PCLRenderer");
diff --git a/src/java/org/apache/fop/render/awt/AWTPrintRenderer.java b/src/java/org/apache/fop/render/awt/AWTPrintRenderer.java
index f205bdf31..189ec1481 100644
--- a/src/java/org/apache/fop/render/awt/AWTPrintRenderer.java
+++ b/src/java/org/apache/fop/render/awt/AWTPrintRenderer.java
@@ -67,18 +67,30 @@ public class AWTPrintRenderer extends AWTRenderer {
     private int copies = 1;
     private PrinterJob printerJob;
 
-    public AWTPrintRenderer(PrinterJob printerJob) {
+    public AWTPrintRenderer() {
         super(null);
-        this.printerJob = printerJob;
+        initialize();
+    }
+
+    private void initialize() throws IllegalArgumentException {
+        // read from command-line options
+        copies = getIntProperty("copies", 1);
         startNumber = getIntProperty("start", 1) - 1;
         endNumber = getIntProperty("end", -1);
-        printerJob.setPageable(this);
-        mode = EVEN_AND_ALL;
         String str = System.getProperty("even");
         if (str != null) {
             mode = Boolean.valueOf(str).booleanValue() ? EVEN : ODD;
         }
-    }
+        
+        printerJob = PrinterJob.getPrinterJob();
+        printerJob.setCopies(copies);
+        if (System.getProperty("dialog") != null) {
+            if (!printerJob.printDialog()) {
+                throw new IllegalArgumentException("Printing cancelled by operator");
+            }
+        }
+        printerJob.setPageable(this);
+    }   
 
     public void stopRenderer() throws IOException {
         super.stopRenderer();
@@ -137,14 +149,5 @@ public class AWTPrintRenderer extends AWTRenderer {
         }
         return vec;
     }
-
-    /* TODO: I'm not totally sure that this is necessary -Mark
-	void setCopies(int val) {
-		copies = val;
-		Vector copie = tree.getPages();
-		for (int i = 1; i < copies; i++) {
-			tree.getPages().addAll(copie);
-		}
-    } */
 } // class AWTPrintRenderer
 
diff --git a/src/java/org/apache/fop/servlet/FopPrintServlet.java b/src/java/org/apache/fop/servlet/FopPrintServlet.java
index 1b030f51e..fc4f3c3c8 100644
--- a/src/java/org/apache/fop/servlet/FopPrintServlet.java
+++ b/src/java/org/apache/fop/servlet/FopPrintServlet.java
@@ -56,8 +56,6 @@ import java.io.InputStream;
 import java.io.PrintWriter;
 import java.util.List;
 
-import java.awt.print.PrinterJob;
-
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServlet;
 import javax.servlet.http.HttpServletRequest;
@@ -171,8 +169,7 @@ public class FopPrintServlet extends HttpServlet {
                          HttpServletResponse response) throws ServletException {
         try {
             Driver driver = new Driver(foFile, null);
-            PrinterJob pj = PrinterJob.getPrinterJob();
-            AWTPrintRenderer renderer = new AWTPrintRenderer(pj);
+            AWTPrintRenderer renderer = new AWTPrintRenderer();
 
             driver.enableLogging(log);
             driver.setRenderer(renderer);
@@ -194,10 +191,7 @@ public class FopPrintServlet extends HttpServlet {
                           HttpServletResponse response) throws ServletException {
         try {
             Driver driver = new Driver();
-            PrinterJob pj = PrinterJob.getPrinterJob();
-            AWTPrintRenderer renderer = new AWTPrintRenderer(pj);
-
-            pj.setCopies(1);
+            AWTPrintRenderer renderer = new AWTPrintRenderer();
 
             driver.enableLogging(log);
             driver.setRenderer(renderer);
-- 
2.39.5