]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Removed the getLogger() and setLogger() from FOUserAgent; replaced with Commons Logging
authorGlen Mazza <gmazza@apache.org>
Wed, 12 May 2004 23:19:53 +0000 (23:19 +0000)
committerGlen Mazza <gmazza@apache.org>
Wed, 12 May 2004 23:19:53 +0000 (23:19 +0000)
equivalents in places where previously used.

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@197591 13f79535-47bb-0310-9956-ffa450edef68

33 files changed:
src/java/org/apache/fop/apps/Driver.java
src/java/org/apache/fop/apps/FOUserAgent.java
src/java/org/apache/fop/fo/FOTreeBuilder.java
src/java/org/apache/fop/fo/flow/ExternalGraphic.java
src/java/org/apache/fop/image/AbstractFopImage.java
src/java/org/apache/fop/image/FopImage.java
src/java/org/apache/fop/image/GifImage.java
src/java/org/apache/fop/image/ImageFactory.java
src/java/org/apache/fop/image/analyser/ImageReaderFactory.java
src/java/org/apache/fop/image/analyser/SVGReader.java
src/java/org/apache/fop/image/analyser/XMLReader.java
src/java/org/apache/fop/layoutmgr/AbstractLayoutManager.java
src/java/org/apache/fop/layoutmgr/BlockContainerLayoutManager.java
src/java/org/apache/fop/layoutmgr/FlowLayoutManager.java
src/java/org/apache/fop/layoutmgr/InlineStackingLayoutManager.java
src/java/org/apache/fop/layoutmgr/LineLayoutManager.java
src/java/org/apache/fop/layoutmgr/PageLayoutManager.java
src/java/org/apache/fop/layoutmgr/RetrieveMarkerLayoutManager.java
src/java/org/apache/fop/layoutmgr/StaticContentLayoutManager.java
src/java/org/apache/fop/layoutmgr/TextLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/Body.java
src/java/org/apache/fop/layoutmgr/table/Cell.java
src/java/org/apache/fop/render/awt/AWTRenderer.java
src/java/org/apache/fop/render/pdf/PDFRenderer.java
src/java/org/apache/fop/render/pdf/PDFXMLHandler.java
src/java/org/apache/fop/render/ps/PSGraphics2D.java
src/java/org/apache/fop/render/ps/PSRenderer.java
src/java/org/apache/fop/render/ps/PSXMLHandler.java
src/java/org/apache/fop/svg/PDFImageElementBridge.java
src/java/org/apache/fop/svg/SVGUserAgent.java
src/java/org/apache/fop/tools/AreaTreeBuilder.java
src/java/org/apache/fop/tools/TestConverter.java
src/java/org/apache/fop/tools/anttasks/Fop.java

index 4cee9c3e1f8fff03137f5136fca7b6cb9dbef674..51f7e7249e1ec1b35b250fc035f3d765a613fd73 100644 (file)
@@ -245,7 +245,6 @@ public class Driver {
     public FOUserAgent getUserAgent() {
         if (userAgent == null) {
             userAgent = new FOUserAgent();
-            userAgent.setLogger(getLogger());
         }
         return userAgent;
     }
index 445c6bdbd20edc4775dae3294d148d235b6a4641..cd862c7cd9f24070b41a388ca52fe0ae81dfadd8 100644 (file)
@@ -23,8 +23,6 @@ import java.util.Map;
 import java.io.IOException;
 import java.io.InputStream;
 
-import org.apache.commons.logging.Log;
-
 // FOP
 import org.apache.fop.pdf.PDFEncryptionParams;
 
@@ -50,7 +48,6 @@ import org.apache.fop.pdf.PDFEncryptionParams;
  */
 public class FOUserAgent {
 
-    private Log log;
     /** Map containing various default values */
     public Map defaults = new java.util.HashMap();
     /** Map containing XML handlers for various document types */
@@ -59,22 +56,6 @@ public class FOUserAgent {
     private PDFEncryptionParams pdfEncryptionParams;
     private float px2mm = 0.35277777777777777778f; //72dpi (=25.4/dpi)
 
-    /**
-     * Sets the logger.
-     * @param log Logger to use
-     */
-    public void setLogger(Log log) {
-        this.log = log;
-    }
-
-    /**
-     * Returns the logger to use.
-     * (todo) This breaks IoC/SoC. Should be improved.
-     */
-    public Log getLogger() {
-        return this.log;
-    }
-
     /**
      * Sets the base URL.
      * @param baseURL base URL
index e1e59f354b6eddefcc73c5a1377961f6c6c59e9e..43c92452d56b1c8f48fcbaf741b8bd5beaff497e 100644 (file)
@@ -30,6 +30,7 @@ import java.util.Map;
 import java.util.Set;
 
 import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.fo.ElementMapping.Maker;
@@ -53,6 +54,11 @@ public class FOTreeBuilder extends DefaultHandler {
      */
     protected Map fobjTable = new java.util.HashMap();
 
+    /**
+     * logging instance
+     */
+    protected Log log = LogFactory.getLog(FOTreeBuilder.class);
+
     /**
      * Set of mapped namespaces.
      */
@@ -90,7 +96,7 @@ public class FOTreeBuilder extends DefaultHandler {
     }
 
     private Log getLogger() {
-        return userAgent.getLogger();
+        return log;
     }
 
     /**
index a4394f2d596e540369f3fed5b2c5c5e42d78b04b..9ff80787e138729e9921af42a499fb0c7e979785 100644 (file)
@@ -124,7 +124,7 @@ public class ExternalGraphic extends FObj {
                 return;
             }
             // load dimensions
-            if (!fopimage.load(FopImage.DIMENSIONS, getUserAgent().getLogger())) {
+            if (!fopimage.load(FopImage.DIMENSIONS)) {
                 // error
                 url = null;
                 return;
index 818768226ed9f9bc7c973e8a1ffb6b8c113b3dfa..57a4098cd789cc70f8ee64d5b7a05a17bbff4b88 100644 (file)
@@ -26,6 +26,7 @@ import java.awt.Color;
 
 // FOP
 import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 /**
  * Base class to implement the FopImage interface.
@@ -34,6 +35,12 @@ import org.apache.commons.logging.Log;
  * @see FopImage
  */
 public abstract class AbstractFopImage implements FopImage {
+
+    /**
+     * logging instance
+     */
+    protected static Log log = LogFactory.getLog(AbstractFopImage.class);
+
     /**
      * Keeps track of what has been loaded.
      */
@@ -125,13 +132,13 @@ public abstract class AbstractFopImage implements FopImage {
      * @param ua the user agent for handling logging etc.
      * @return true if the loading was successful
      */
-    public synchronized boolean load(int type, Log logger) {
+    public synchronized boolean load(int type) {
         if ((loaded & type) != 0) {
             return true;
         }
         boolean success = true;
         if (((type & DIMENSIONS) != 0) && ((loaded & DIMENSIONS) == 0)) {
-            success = success && loadDimensions(logger);
+            success = success && loadDimensions();
 
             if (!success) {
                 return false;
@@ -139,13 +146,13 @@ public abstract class AbstractFopImage implements FopImage {
             loaded = loaded | DIMENSIONS;
         }
         if (((type & BITMAP) != 0) && ((loaded & BITMAP) == 0)) {
-            success = success && loadBitmap(logger);
+            success = success && loadBitmap();
             if (success) {
                 loaded = loaded | BITMAP;
             }
         }
         if (((type & ORIGINAL_DATA) != 0) && ((loaded & ORIGINAL_DATA) == 0)) {
-            success = success && loadOriginalData(logger);
+            success = success && loadOriginalData();
             if (success) {
                 loaded = loaded | ORIGINAL_DATA;
             }
@@ -161,7 +168,7 @@ public abstract class AbstractFopImage implements FopImage {
      * @param ua the user agent
      * @return true if the loading was successful
      */
-    protected boolean loadDimensions(Log logger) {
+    protected boolean loadDimensions() {
         return false;
     }
 
@@ -173,7 +180,7 @@ public abstract class AbstractFopImage implements FopImage {
      * @param ua the user agent
      * @return true if the loading was successful
      */
-    protected boolean loadBitmap(Log logger) {
+    protected boolean loadBitmap() {
         return false;
     }
 
@@ -185,7 +192,7 @@ public abstract class AbstractFopImage implements FopImage {
      * @param ua the user agent
      * @return true if the loading was successful
      */
-    protected boolean loadOriginalData(Log logger) {
+    protected boolean loadOriginalData() {
         return false;
     }
 
index c1b4aaa86f3477d47914b79a182a1d3a4f9270ca..f72f97c90c9741ab0479f8d6f571299225636ea7 100644 (file)
@@ -61,10 +61,9 @@ public interface FopImage {
      * the information.
      *
      * @param type the type of loading required
-     * @param logger the Commons-logging component
      * @return boolean true if the information could be loaded
      */
-    boolean load(int type, Log logger);
+    boolean load(int type);
 
     /**
      * Returns the image width.
index 8a23a7d0c2da523c30867cdd9c3572cc7e762ff2..26ca233234fe1302cd47d42ccb7dcfeaea0564f3 100644 (file)
@@ -58,7 +58,7 @@ public class GifImage extends AbstractFopImage {
      * @param ua the user agent for loading
      * @return True if the load process succeeded
      */
-    protected boolean loadBitmap(Log logger) {
+    protected boolean loadBitmap() {
         int[] tmpMap = null;
         try {
             URLConnection con = new DummyConnection(inputStream);
@@ -81,7 +81,7 @@ public class GifImage extends AbstractFopImage {
             try {
                 tmpMap = consumer.getImage();
             } catch (Exception ex) {
-                logger.error("Image grabbing interrupted : "
+                log.error("Image grabbing interrupted : "
                               + ex.getMessage(), ex);
                 return false;
             }
@@ -146,7 +146,7 @@ public class GifImage extends AbstractFopImage {
                 this.isTransparent = false;
             }
         } catch (Exception ex) {
-            logger.error("Error while loading image "
+            log.error("Error while loading image "
                           + "" + " : "
                           + ex.getClass() + " - "
                           + ex.getMessage(), ex);
index 93253240a5844e80a0938669471bc42819853f80..75ffff95b3958edb24bae886dc1121fdd521d63e 100644 (file)
@@ -30,6 +30,7 @@ import java.util.Collections;
 import java.util.Iterator;
 
 import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 // FOP
 import org.apache.fop.image.analyser.ImageReaderFactory;
@@ -42,6 +43,11 @@ import org.apache.fop.apps.FOUserAgent;
  */
 public class ImageFactory {
 
+    /**
+     * logging instance
+     */
+    protected static Log log = LogFactory.getLog(FopImage.class);
+
     private static ImageFactory factory = new ImageFactory();
 
     private ImageCache cache = new ContextImageCache(true);
@@ -129,7 +135,6 @@ public class ImageFactory {
      * @return the fop image instance
      */
     public static FopImage loadImage(String href, FOUserAgent ua) {
-        Log log = ua.getLogger();
 
         InputStream in = openStream(href, ua);
 
@@ -214,7 +219,7 @@ public class ImageFactory {
      * @return a new FopImage object
      */
     protected static InputStream openStream(String href, FOUserAgent ua) {
-        Log log = ua.getLogger();
+
         // Get the absolute URL
         URL absoluteURL = null;
         InputStream in = null;
index 8d6ce8929b4884414b847f380018b730fa1266f4..826f4b60f08402f741fb5fec5bf5f80b44f95fa3 100644 (file)
@@ -27,6 +27,10 @@ import java.util.ArrayList;
 import org.apache.fop.image.FopImage;
 import org.apache.fop.apps.FOUserAgent;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+
 /**
  * Factory for ImageReader objects.
  *
@@ -37,6 +41,8 @@ public class ImageReaderFactory {
 
     private static ArrayList formats = new ArrayList();
 
+    protected static Log log = LogFactory.getLog(ImageReaderFactory.class);
+
     static {
         registerFormat(new JPEGReader());
         registerFormat(new BMPReader());
@@ -80,9 +86,8 @@ public class ImageReaderFactory {
                 }
             }
         } catch (IOException ex) {
-            ua.getLogger().error(
-                    "Error while recovering Image Informations ("
-                    + uri + ")", ex);
+            log.error("Error while recovering Image Informations ("
+                + uri + ")", ex);
         }
         return null;
     }
index 2a3bade9d62708ee075b8beaf7f0822b8c787827..3ac9a4c753e1b156405068afd776f13ae6139318 100644 (file)
@@ -89,7 +89,7 @@ public class SVGReader implements ImageReader {
         if (batik) {
             try {
                 Loader loader = new Loader();
-                return loader.getImage(uri, bis, ua.getLogger(), 
+                return loader.getImage(uri, bis, 
                     ua.getPixelUnitToMillimeter());
             } catch (NoClassDefFoundError e) {
                 batik = false;
@@ -107,7 +107,7 @@ public class SVGReader implements ImageReader {
      */
     class Loader {
         private FopImage.ImageInfo getImage(String uri, InputStream fis,
-                Log logger, float pixelUnitToMM) {
+                float pixelUnitToMM) {
             // parse document and get the size attributes of the svg element
 
             try {
@@ -177,7 +177,7 @@ public class SVGReader implements ImageReader {
 
                 Element e = doc.getRootElement();
                 String s;
-                SVGUserAgent userAg = new SVGUserAgent(logger, pixelUnitToMM,
+                SVGUserAgent userAg = new SVGUserAgent(pixelUnitToMM,
                             new AffineTransform());
                 BridgeContext ctx = new BridgeContext(userAg);
                 UnitProcessor.Context uctx =
index 9c13067ec5f73fea98dc4c83d23ac6d1a9de545d..bb85980c6141fc7cfb6642cc97ba221e546b8783 100644 (file)
@@ -32,9 +32,18 @@ import org.w3c.dom.Element;
 import org.apache.fop.image.FopImage;
 import org.apache.fop.apps.FOUserAgent;
 
+// Commons-Logging
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /** ImageReader object for XML document image type. */
 public class XMLReader implements ImageReader {
 
+    /**
+     * logging instance
+     */
+    private Log log = LogFactory.getLog(XMLReader.class);
+
     private static Map converters = new java.util.HashMap();
 
     /**
@@ -108,7 +117,7 @@ public class XMLReader implements ImageReader {
             info.data = doc;
 
             Element root = doc.getDocumentElement();
-            ua.getLogger().debug("XML image namespace: " + root.getAttribute("xmlns"));
+            log.debug("XML image namespace: " + root.getAttribute("xmlns"));
             String ns = root.getAttribute("xmlns");
             info.str = ns;
 
@@ -120,7 +129,7 @@ public class XMLReader implements ImageReader {
                 }
             }
         } catch (Exception e) {
-            ua.getLogger().warn("Error while constructing image from XML", e);
+            log.warn("Error while constructing image from XML", e);
             try {
                 is.reset();
             } catch (IOException ioe) {
index 1a11396ba3d2ac5a00f43cff015a35b2f1c468fd..0f5d51542b769ea86d29963176901b4f1f904839 100644 (file)
@@ -28,6 +28,7 @@ import org.apache.fop.fo.Constants;
 import org.apache.fop.fo.PropertyManager;
 
 import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 import java.util.ListIterator;
 import java.util.Map;
@@ -48,6 +49,11 @@ public abstract class AbstractLayoutManager implements LayoutManager, Constants
     protected ListIterator childLMiter;
     protected boolean bInited = false;
 
+    /**
+     * logging instance
+     */
+    protected static Log log = LogFactory.getLog(AbstractLayoutManager.class);
+
     /**
      * Abstract layout manager.
      */
@@ -93,10 +99,6 @@ public abstract class AbstractLayoutManager implements LayoutManager, Constants
         return userAgent;
     }
 
-    protected Log getLogger() {
-        return userAgent.getLogger();
-    }
-
     public void setParent(LayoutManager lm) {
         this.parentLM = lm;
     }
index ac0fa3414833b2a15a89cb8d8a476e66b7ee59a7..156d6efee64105b04c793f990f95e6f9ecdc25ef 100644 (file)
@@ -245,7 +245,7 @@ public class BlockContainerLayoutManager extends BlockStackingLayoutManager {
             if (overflow == Overflow.HIDDEN) {
                 clip = true;
             } else if (overflow == Overflow.ERROR_IF_OVERFLOW) {
-                getLogger().error("contents overflows block-container viewport: clipping");
+                log.error("contents overflows block-container viewport: clipping");
                 clip = true;
             }
         }
index 0e27e8c595782b59da1a349ad610dbe831fe946b..ae47270a75b00ef0e98a0b846186d4f4ec61a257 100644 (file)
@@ -60,7 +60,7 @@ public class FlowLayoutManager extends BlockStackingLayoutManager {
 
         while ((curLM = getChildLM()) != null) {
             if (curLM.generatesInlineAreas()) {
-                getLogger().error("inline area not allowed under flow - ignoring");
+                log.error("inline area not allowed under flow - ignoring");
                 curLM.setFinished(true);
                 continue;
             }
@@ -169,7 +169,7 @@ public class FlowLayoutManager extends BlockStackingLayoutManager {
      */
     public Marker retrieveMarker(String name, int pos, int boundary) {
         // error cannot retrieve markers in flow
-        getLogger().error("Cannot retrieve a marker from the flow");
+        log.error("Cannot retrieve a marker from the flow");
         return null;
     }
 }
index 6f1e90a1fafd8615449ef4447a620e6a848cf8af..a052618289a7bf33d3ecc8e49247ed973128237f 100644 (file)
@@ -267,7 +267,7 @@ public class InlineStackingLayoutManager extends AbstractLayoutManager {
 
             // ignore nested blocks for now
             if (!curLM.generatesInlineAreas()) {
-                getLogger().warn("ignoring block inside inline fo");
+                log.warn("ignoring block inside inline fo");
                 curLM.setFinished(true);
                 continue;
             }
index 3c456ee00676245df5c6d8ede3076b9c9f927ec9..a511c2cb39d7767871eafa7657278f82fe8029f8 100644 (file)
@@ -448,7 +448,7 @@ public class LineLayoutManager extends InlineStackingLayoutManager {
         while (bpIter.hasPrevious() && bpIter.previous() != prev) {
         }
         if (prev != null && bpIter.next() != prev) {
-            getLogger().error("findHyphenPoss: problem!");
+            log.error("findHyphenPoss: problem!");
             return null;
         }
         StringBuffer sbChars = new StringBuffer(30);
@@ -465,7 +465,7 @@ public class LineLayoutManager extends InlineStackingLayoutManager {
             prev = bp;
         }
         vecInlineBreaks.remove(vecInlineBreaks.size() - 1); // remove last
-        getLogger().debug("Word to hyphenate: " + sbChars.toString());
+        log.debug("Word to hyphenate: " + sbChars.toString());
 
         // Now find all hyphenation points in this word (get in an array of offsets)
         // hyphProps are from the block level?. Note that according to the spec,
index 958834654d5c61af837f50c10c1b368edf8e9944..b5711cb140a0f579ef1db6883d50359fc3d9f35e 100644 (file)
@@ -458,7 +458,7 @@ public class PageLayoutManager extends AbstractLayoutManager implements Runnable
         }
         RegionViewport reg = curPage.getPage().getRegionViewport(regionClass);
         if (reg == null) {
-            getLogger().error("no region viewport: shouldn't happen");
+            log.error("no region viewport: shouldn't happen");
         }
         StaticContentLayoutManager lm = getStaticContentLayoutManager(flow);
         lm.setUserAgent(getUserAgent());
@@ -476,7 +476,7 @@ public class PageLayoutManager extends AbstractLayoutManager implements Runnable
                 lm.addAreas(new BreakPossPosIter(vecBreakPoss, 0,
                                                   vecBreakPoss.size()), null);
             } else {
-              getLogger().error("bp==null  cls=" + regionClass);
+              log.error("bp==null  cls=" + regionClass);
             }
         }
         //lm.flush();
@@ -836,7 +836,7 @@ public class PageLayoutManager extends AbstractLayoutManager implements Runnable
         if ((columnCount > 1) && (r.overflow == Overflow.SCROLL)) {
             // recover by setting 'column-count' to 1. This is allowed but
             // not required by the spec.
-            getLogger().error("Setting 'column-count' to 1 because "
+            log.error("Setting 'column-count' to 1 because "
                     + "'overflow' is set to 'scroll'");
             columnCount = 1;
         }
index b5fd4d50233dcd025a259e5076824b1e7f7dcad9..1675d875f6eadb11234068c1e5bdf22813b354ec 100644 (file)
@@ -97,9 +97,9 @@ public class RetrieveMarkerLayoutManager extends AbstractLayoutManager {
                     replaceLM =  (LayoutManager)list.get(0);
                     replaceLM.setParent(this);
                     replaceLM.initialize();
-                    getLogger().debug("retrieved: " + replaceLM + ":" + list.size());
+                    log.debug("retrieved: " + replaceLM + ":" + list.size());
                 } else {
-                    getLogger().debug("found no marker with name: " + name);
+                    log.debug("found no marker with name: " + name);
                 }
             }
         }
index bbab0ca74cb64e612bec6693f6f136a57e5b9f3d..4242312763c18665d9134c8fe307f590b7818576 100644 (file)
@@ -62,7 +62,7 @@ public class StaticContentLayoutManager extends BlockStackingLayoutManager {
                 if ((bp = curLM.getNextBreakPoss(childLC)) != null) {
                     blockBreaks.add(bp);
                     if (bp.isForcedBreak()) {
-                        getLogger().error("Forced breaks are not allowed in "
+                        log.error("Forced breaks are not allowed in "
                                 + "static content - ignoring");
                         return null;
                     }
@@ -128,7 +128,7 @@ public class StaticContentLayoutManager extends BlockStackingLayoutManager {
      */
     public void addMarker(Map marks, boolean start, boolean isfirst) {
         // error markers not allowed in static
-        getLogger().error("Cannot add marker to static areas");
+        log.error("Cannot add marker to static areas");
     }
 }
 
index f7badde2f3bed67010e39e8ee0246196454191b1..a56233b8a68bd2504179f1ee21e787f34a0d47fc 100644 (file)
@@ -172,7 +172,7 @@ public class TextLayoutManager extends AbstractLayoutManager {
         if (prevPos != null) {
             // ASSERT (prevPos.getLM() == this)
             if (prevPos.getLM() != this) {
-                getLogger().error(
+                log.error(
                   "TextLayoutManager.resetPosition: " + "LM mismatch!!!");
             }
             LeafPosition tbp = (LeafPosition) prevPos;
index ac44124f5b899f2192f5fd6594142bc43b1948c0..4ba9a70622019c70f82b2b7c04c35923ab6d215e 100644 (file)
@@ -96,7 +96,7 @@ public class Body extends BlockStackingLayoutManager {
 
         if (columns == null) {
             setFinished(true);
-            getLogger().warn("ignoring fo:table-body with undefined fo:table-columns");
+            log.warn("ignoring fo:table-body with undefined fo:table-columns");
             return null;
         }
 
index 0819d0c94d3686ac5a8f36a922343b66b5e18be0..9d39d058aabea6f46d2cdfc5ceb5002bca061b8b 100644 (file)
@@ -89,7 +89,7 @@ public class Cell extends BlockStackingLayoutManager {
 
         while ((curLM = getChildLM()) != null) {
             if (curLM.generatesInlineAreas()) {
-                getLogger().error("table-cell must contain block areas - ignoring");
+                log.error("table-cell must contain block areas - ignoring");
                 curLM.setFinished(true);
                 continue;
             }
index 552e59bcf106359c066ae81f04ee4ba782b5e371..4f049a407cf6abcd34f45a569850552b07773b2c 100644 (file)
@@ -377,7 +377,7 @@ public class AWTRenderer extends AbstractRenderer implements Printable, Pageable
             if (back.getURL() != null) {  // TODO: implement
                 ImageFactory fact = ImageFactory.getInstance();
                 FopImage fopimage = fact.getImage(back.getURL(), userAgent);
-                if (fopimage != null && fopimage.load(FopImage.DIMENSIONS, userAgent.getLogger())) {
+                if (fopimage != null && fopimage.load(FopImage.DIMENSIONS)) {
                     if (back.getRepeat() == BackgroundRepeat.REPEAT) {
                         // create a pattern for the image
                     } else {
index ed35a16fefc0df8b607c2c41b7b3d0b150820507..5debef993b604aa66fe9300d14f304efdde388df 100644 (file)
@@ -594,7 +594,7 @@ public class PDFRenderer extends PrintRenderer {
             if (back.getURL() != null) {
                 ImageFactory fact = ImageFactory.getInstance();
                 FopImage fopimage = fact.getImage(back.getURL(), userAgent);
-                if (fopimage != null && fopimage.load(FopImage.DIMENSIONS, userAgent.getLogger())) {
+                if (fopimage != null && fopimage.load(FopImage.DIMENSIONS)) {
                     if (back.getRepeat() == BackgroundRepeat.REPEAT) {
                         // create a pattern for the image
                     } else {
@@ -1118,12 +1118,12 @@ public class PDFRenderer extends PrintRenderer {
         if (fopimage == null) {
             return;
         }
-        if (!fopimage.load(FopImage.DIMENSIONS, userAgent.getLogger())) {
+        if (!fopimage.load(FopImage.DIMENSIONS)) {
             return;
         }
         String mime = fopimage.getMimeType();
         if ("text/xml".equals(mime)) {
-            if (!fopimage.load(FopImage.ORIGINAL_DATA, userAgent.getLogger())) {
+            if (!fopimage.load(FopImage.ORIGINAL_DATA)) {
                 return;
             }
             Document doc = ((XMLImage) fopimage).getDocument();
@@ -1131,7 +1131,7 @@ public class PDFRenderer extends PrintRenderer {
 
             renderDocument(doc, ns, pos);
         } else if ("image/svg+xml".equals(mime)) {
-            if (!fopimage.load(FopImage.ORIGINAL_DATA, userAgent.getLogger())) {
+            if (!fopimage.load(FopImage.ORIGINAL_DATA)) {
                 return;
             }
             Document doc = ((XMLImage) fopimage).getDocument();
@@ -1139,14 +1139,14 @@ public class PDFRenderer extends PrintRenderer {
 
             renderDocument(doc, ns, pos);
         } else if ("image/eps".equals(mime)) {
-            if (!fopimage.load(FopImage.ORIGINAL_DATA, userAgent.getLogger())) {
+            if (!fopimage.load(FopImage.ORIGINAL_DATA)) {
                 return;
             }
             FopPDFImage pdfimage = new FopPDFImage(fopimage, url);
             int xobj = pdfDoc.addImage(currentContext, pdfimage).getXNumber();
             fact.releaseImage(url, userAgent);
         } else if ("image/jpeg".equals(mime)) {
-            if (!fopimage.load(FopImage.ORIGINAL_DATA, userAgent.getLogger())) {
+            if (!fopimage.load(FopImage.ORIGINAL_DATA)) {
                 return;
             }
             FopPDFImage pdfimage = new FopPDFImage(fopimage, url);
@@ -1158,7 +1158,7 @@ public class PDFRenderer extends PrintRenderer {
             placeImage((int) pos.getX() / 1000,
                        (int) pos.getY() / 1000, w, h, xobj);
         } else {
-            if (!fopimage.load(FopImage.BITMAP, userAgent.getLogger())) {
+            if (!fopimage.load(FopImage.BITMAP)) {
                 return;
             }
             FopPDFImage pdfimage = new FopPDFImage(fopimage, url);
index 38d28e90bd071f501ee07816ad6eb1e9417c1113..a1debb513701a973703ead389843839479a6a5f3 100644 (file)
@@ -32,8 +32,13 @@ import org.apache.fop.svg.PDFGraphics2D;
 import org.apache.fop.svg.SVGUserAgent;
 import org.apache.fop.fonts.FontInfo;
 
+// Commons-Logging
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+
 /* org.w3c.dom.Document is not imported to avoid conflict with
-   org.apache.fop.control.Document */
+   org.apache.fop.apps.Document */
 
 import java.io.OutputStream;
 
@@ -55,6 +60,12 @@ import java.awt.geom.AffineTransform;
  * The properties from the PDF renderer are subject to change.
  */
 public class PDFXMLHandler implements XMLHandler {
+
+    /**
+     * logging instance
+     */
+    private Log log = LogFactory.getLog(PDFXMLHandler.class);
+
     /**
      * The PDF document that is being drawn into.
      */
@@ -220,8 +231,7 @@ public class PDFXMLHandler implements XMLHandler {
             int yOffset = pdfInfo.currentYPosition;
 
             SVGUserAgent ua
-                 = new SVGUserAgent(context.getUserAgent().getLogger(), 
-                        context.getUserAgent().getPixelUnitToMillimeter(),
+                 = new SVGUserAgent(context.getUserAgent().getPixelUnitToMillimeter(),
                         new AffineTransform());
 
             GVTBuilder builder = new GVTBuilder();
@@ -240,7 +250,7 @@ public class PDFXMLHandler implements XMLHandler {
             try {
                 root = builder.build(ctx, doc);
             } catch (Exception e) {
-                context.getUserAgent().getLogger().error("svg graphic could not be built: "
+                log.error("svg graphic could not be built: "
                                        + e.getMessage(), e);
                 return;
             }
@@ -298,7 +308,7 @@ public class PDFXMLHandler implements XMLHandler {
                 root.paint(graphics);
                 pdfInfo.currentStream.add(graphics.getString());
             } catch (Exception e) {
-                context.getUserAgent().getLogger().error("svg graphic could not be rendered: "
+                log.error("svg graphic could not be rendered: "
                                        + e.getMessage(), e);
             }
 
index 039f3d4833dec7cb952ff8241c6010fd87b55404..4da7cc5c32de142e8bb256eb80d23b6d7bc1cb8e 100644 (file)
@@ -320,7 +320,7 @@ public class PSGraphics2D extends AbstractGraphics2D {
         /**
          * @see org.apache.fop.image.FopImage#load(int, org.apache.commons.logging.Log)
          */
-        public boolean load(int type, Log logger) {
+        public boolean load(int type) {
             switch (type) {
                 case FopImage.DIMENSIONS: break;
                 case FopImage.BITMAP: break;
index caf6654ea4cae255af61f98b40b70fdd3627d494..68708a2ebae47d4f56c3a7aaa1440f79e44cdbc8 100644 (file)
@@ -698,7 +698,7 @@ public class PSRenderer extends AbstractRenderer {
             if (back.getURL() != null) {
                 ImageFactory fact = ImageFactory.getInstance();
                 FopImage fopimage = fact.getImage(back.getURL(), userAgent);
-                if (fopimage != null && fopimage.load(FopImage.DIMENSIONS, userAgent.getLogger())) {
+                if (fopimage != null && fopimage.load(FopImage.DIMENSIONS)) {
                     if (back.getRepeat() == BackgroundRepeat.REPEAT) {
                         // create a pattern for the image
                     } else {
index 671a78dab21b3137efa01c0a3141b525d8c19e4a..3f5020bf91c60f62bc8939b6e4da21e888484cd2 100644 (file)
@@ -33,12 +33,17 @@ import org.apache.batik.bridge.GVTBuilder;
 import org.apache.batik.bridge.BridgeContext;
 import org.apache.batik.bridge.ViewBox;
 import org.apache.batik.gvt.GraphicsNode;
+
 // FOP
 import org.apache.fop.fonts.FontInfo;
 import org.apache.fop.render.XMLHandler;
 import org.apache.fop.render.RendererContext;
 import org.apache.fop.svg.SVGUserAgent;
 
+// Commons-Logging
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /**
  * PostScript XML handler.
  * This handler handles XML for foreign objects when rendering to PostScript.
@@ -50,6 +55,11 @@ import org.apache.fop.svg.SVGUserAgent;
  */
 public class PSXMLHandler implements XMLHandler {
 
+    /**
+     * logging instance
+     */
+    private Log log = LogFactory.getLog(PSXMLHandler.class);
+
     /**
      * The PostScript generator that is being used to drawn into.
      */
@@ -258,9 +268,9 @@ public class PSXMLHandler implements XMLHandler {
             PSGenerator gen = psInfo.psGenerator;
 
             SVGUserAgent ua
-                 = new SVGUserAgent(context.getUserAgent().getLogger(), 
-                        context.getUserAgent().getPixelUnitToMillimeter(),
-                        new AffineTransform());
+                 = new SVGUserAgent(
+                    context.getUserAgent().getPixelUnitToMillimeter(),
+                    new AffineTransform());
 
             GVTBuilder builder = new GVTBuilder();
             BridgeContext ctx = new BridgeContext(ua);
@@ -280,7 +290,7 @@ public class PSXMLHandler implements XMLHandler {
             try {
                 root = builder.build(ctx, doc);
             } catch (Exception e) {
-                context.getUserAgent().getLogger().error("SVG graphic could not be built: "
+                log.error("SVG graphic could not be built: "
                                        + e.getMessage(), e);
                 return;
             }
@@ -337,7 +347,7 @@ public class PSXMLHandler implements XMLHandler {
                     root.paint(graphics);
                     //psInfo.currentStream.add(graphics.getString());
                 } catch (Exception e) {
-                    context.getUserAgent().getLogger().error("SVG graphic could not be rendered: "
+                    log.error("SVG graphic could not be rendered: "
                                            + e.getMessage(), e);
                 }
 
@@ -345,7 +355,7 @@ public class PSXMLHandler implements XMLHandler {
                 //psInfo.pdfState.pop();
                 gen.writeln("%SVG graphic end ---");
             } catch (IOException ioe) {
-                context.getUserAgent().getLogger().error("SVG graphic could not be rendered: "
+                log.error("SVG graphic could not be rendered: "
                                        + ioe.getMessage(), ioe);
             }
         }
index b3eebb5efc9a03a82360dfa198d63092a0741f03..567e0c46d2d00a833c672217da06d44f434ec455 100644 (file)
@@ -68,7 +68,7 @@ public class PDFImageElementBridge extends SVGImageElementBridge {
                 JpegImage jpeg = new JpegImage(ii);
                 SimpleLog logger = new SimpleLog("FOP/SVG");
                 logger.setLevel(SimpleLog.LOG_LEVEL_INFO);
-                jpeg.load(FopImage.ORIGINAL_DATA, logger);
+                jpeg.load(FopImage.ORIGINAL_DATA);
                 PDFJpegNode node = new PDFJpegNode(jpeg, origGN);
 
                 Rectangle2D imgBounds = getImageBounds(ctx, e);
index aefdd1169e326e6d057550fc443af593ea69c755..bd66517d41b0432253cfb397dfccbd624ddca3de 100644 (file)
@@ -27,6 +27,10 @@ import java.awt.geom.AffineTransform;
 import java.awt.geom.Dimension2D;
 import java.awt.Dimension;
 
+// Commons-Logging
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /**
  * The SVG user agent.
  * This is an implementation of the batik svg user agent
@@ -34,9 +38,13 @@ import java.awt.Dimension;
  */
 public class SVGUserAgent extends UserAgentAdapter {
     private AffineTransform currentTransform = null;
-    private Log logger;
     private float pixelUnitToMillimeter = 0.0f;
 
+    /**
+     * logging instance
+     */
+    private Log logger = LogFactory.getLog(SVGUserAgent.class);
+
     /**
      * Creates a new SVGUserAgent.
      * @param log an Commons logging instance
@@ -44,8 +52,7 @@ public class SVGUserAgent extends UserAgentAdapter {
      *                  currently in effect
      * @param at the current transform
      */
-    public SVGUserAgent(Log log, float pixelUnitToMM, AffineTransform at) {
-        logger = log;
+    public SVGUserAgent(float pixelUnitToMM, AffineTransform at) {
         pixelUnitToMillimeter = pixelUnitToMM;
         currentTransform = at;
     }
index 74ab65ecd89a64d5a57ea95ea10c8d990bfec7e4..109a28e594551170bc74c90ec214ea9ff9a5804e 100644 (file)
@@ -163,7 +163,6 @@ public class AreaTreeBuilder {
         org.apache.fop.apps.Document doc = new org.apache.fop.apps.Document(null);
         rend.setupFontInfo(doc.getFontInfo());
         FOUserAgent ua = new FOUserAgent();
-        ua.setLogger(logger);
         rend.setUserAgent(ua);
 
         StorePagesModel sm = AreaTree.createStorePagesModel();
index a569a934d3bd5d5eb144c5397c638166f7f15a11..71b6e764abb5fd6917cd59cfe1be050ef569cde5 100644 (file)
@@ -307,7 +307,6 @@ public class TestConverter {
 
             Driver driver = new Driver();
             FOUserAgent userAgent = new FOUserAgent();
-            userAgent.setLogger(getLogger());
             userAgent.setBaseURL(baseURL);
             driver.setUserAgent(userAgent);
             if (outputPDF) {
index 0e37400f953e5b03da1ace5481317788f2e296b5..d82b922edb126763e172576f04ffe2aa9b80f357 100644 (file)
@@ -535,7 +535,6 @@ class FOPTaskStarter {
             driver.setLogger(getLogger());
             FOUserAgent userAgent = new FOUserAgent();
             userAgent.setBaseURL(this.baseURL);
-            userAgent.setLogger(getLogger());
             driver.setUserAgent(userAgent);
             driver.setRenderer(renderer);
             driver.setOutputStream(out);