]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Added tests and cleaning up of code
authorMehdi Houshmand <mehdi@apache.org>
Wed, 20 Jun 2012 13:51:14 +0000 (13:51 +0000)
committerMehdi Houshmand <mehdi@apache.org>
Wed, 20 Jun 2012 13:51:14 +0000 (13:51 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/branches/Temp_URI_Unification@1352101 13f79535-47bb-0310-9956-ffa450edef68

65 files changed:
src/java/org/apache/fop/afp/util/AFPResourceAccessor.java
src/java/org/apache/fop/apps/FOUserAgent.java
src/java/org/apache/fop/apps/MimeConstants.java
src/java/org/apache/fop/apps/io/URIResolverWrapper.java [deleted file]
src/java/org/apache/fop/cli/CommandLineOptions.java
src/java/org/apache/fop/fonts/DefaultFontConfigurator.java
src/java/org/apache/fop/fonts/FontManagerConfigurator.java
src/java/org/apache/fop/fonts/FontReader.java
src/java/org/apache/fop/fonts/FontSetup.java
src/java/org/apache/fop/fonts/LazyFont.java
src/java/org/apache/fop/fonts/autodetect/FontInfoFinder.java
src/java/org/apache/fop/fonts/type1/Type1FontLoader.java
src/java/org/apache/fop/hyphenation/Hyphenator.java
src/java/org/apache/fop/render/DefaultRendererConfigurator.java [new file with mode: 0644]
src/java/org/apache/fop/render/RendererConfig.java
src/java/org/apache/fop/render/RendererConfigOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/RendererConfigOptions.java [deleted file]
src/java/org/apache/fop/render/adobe/AdobeRendererConfigurator.java [deleted file]
src/java/org/apache/fop/render/afp/AFPRendererConfig.java
src/java/org/apache/fop/render/afp/AFPRendererConfigOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/bitmap/BitmapRendererConfig.java
src/java/org/apache/fop/render/bitmap/BitmapRendererConfigOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/bitmap/BitmapRendererConfigOptions.java [deleted file]
src/java/org/apache/fop/render/bitmap/BitmapRendererConfigurator.java
src/java/org/apache/fop/render/bitmap/BitmapRenderingSettings.java
src/java/org/apache/fop/render/bitmap/PNGRendererConfig.java
src/java/org/apache/fop/render/bitmap/PNGRendererMaker.java
src/java/org/apache/fop/render/bitmap/TIFFRendererConfig.java
src/java/org/apache/fop/render/intermediate/IFRendererConfig.java
src/java/org/apache/fop/render/intermediate/IFSerializer.java
src/java/org/apache/fop/render/java2d/Java2DRenderer.java
src/java/org/apache/fop/render/java2d/Java2DRendererConfig.java
src/java/org/apache/fop/render/java2d/Java2DRendererConfigurator.java
src/java/org/apache/fop/render/java2d/Java2DRendererOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/java2d/Java2DRendererOptions.java [deleted file]
src/java/org/apache/fop/render/pcl/Java2DRendererConfigOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/pcl/PCLRendererConfig.java
src/java/org/apache/fop/render/pdf/PDFRendererConfig.java
src/java/org/apache/fop/render/pdf/PDFRendererConfigOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/pdf/PDFRendererConfigOptions.java [deleted file]
src/java/org/apache/fop/render/pdf/PDFRendererConfigurator.java
src/java/org/apache/fop/render/pdf/PDFRenderingUtil.java
src/java/org/apache/fop/render/ps/PSRendererConfig.java
src/java/org/apache/fop/render/ps/PSRendererConfigurationOption.java [new file with mode: 0644]
src/java/org/apache/fop/render/ps/PSRendererConfigurationOptions.java [deleted file]
src/java/org/apache/fop/render/ps/PSRendererConfigurator.java
src/java/org/apache/fop/render/ps/PSRenderingUtil.java
src/java/org/apache/fop/render/txt/TXTRendererConfigurator.java
src/java/org/apache/fop/render/txt/TxtRendererConfig.java
src/java/org/apache/fop/svg/PDFDocumentGraphics2DConfigurator.java
test/java/org/apache/fop/apps/AFPRendererConfBuilder.java
test/java/org/apache/fop/apps/BitmapRendererConfBuilder.java
test/java/org/apache/fop/apps/EnvironmentalProfileFactoryTestCase.java [new file with mode: 0644]
test/java/org/apache/fop/apps/FopConfBuilder.java
test/java/org/apache/fop/apps/Java2DRendererConfBuilder.java
test/java/org/apache/fop/apps/PDFRendererConfBuilder.java
test/java/org/apache/fop/apps/PSRendererConfBuilder.java
test/java/org/apache/fop/apps/TIFFRendererConfBuilder.java
test/java/org/apache/fop/apps/TxtRendererConfBuilder.java
test/java/org/apache/fop/render/bitmap/AbstractBitmapRendererConfigParserTester.java
test/java/org/apache/fop/render/bitmap/AbstractBitmapRendererConfiguratorTest.java
test/java/org/apache/fop/render/bitmap/BitmapRendererConfigParserTestCase.java
test/java/org/apache/fop/render/pcl/PCLRendererConfBuilder.java
test/java/org/apache/fop/render/pdf/PDFRendererConfigParserTestCase.java
test/java/org/apache/fop/render/txt/TxtRendererConfigParserTestCase.java

index 9993fc81c8957430632cec8ad17bf64dc9958fd6..ddcfa001bcce3e65968461c44240fcdcb144bf4a 100644 (file)
@@ -24,6 +24,8 @@ import java.io.InputStream;
 import java.net.URI;
 import java.net.URISyntaxException;
 
+import org.apache.fop.apps.FOUserAgent;
+import org.apache.fop.apps.FopFactory;
 import org.apache.fop.apps.io.InternalResourceResolver;
 
 /**
index 987ba7d1e01cfa470b2aa2248bc7089e4a682c89..fc9287401177a5781709c30d483abd97623d6a65 100644 (file)
@@ -60,7 +60,7 @@ import org.apache.fop.render.ImageHandlerRegistry;
 import org.apache.fop.render.Renderer;
 import org.apache.fop.render.RendererConfig;
 import org.apache.fop.render.RendererConfig.RendererConfigParser;
-import org.apache.fop.render.RendererConfigOptions;
+import org.apache.fop.render.RendererConfigOption;
 import org.apache.fop.render.RendererFactory;
 import org.apache.fop.render.XMLHandlerRegistry;
 import org.apache.fop.render.intermediate.IFDocumentHandler;
@@ -390,7 +390,7 @@ public class FOUserAgent {
      * @param option the renderer option
      * @return the value
     */
-    public Object getRendererOption(RendererConfigOptions option) {
+    public Object getRendererOption(RendererConfigOption option) {
         return rendererOptions.get(option.getName());
     }
 
index 851690db1c1976c6396401d76e071c1c9b54d902..75b05c8b72c320c708a42beb5af8bede5f85a397 100644 (file)
@@ -32,4 +32,6 @@ public interface MimeConstants extends org.apache.xmlgraphics.util.MimeConstants
     String MIME_FOP_AREA_TREE   = "application/X-fop-areatree";
     /** Apache FOP's intermediate format XML */
     String MIME_FOP_IF          = "application/X-fop-intermediate-format";
+    /** Bitmap images */
+    String MIME_BITMAP          = "image/x-bitmap";
 }
diff --git a/src/java/org/apache/fop/apps/io/URIResolverWrapper.java b/src/java/org/apache/fop/apps/io/URIResolverWrapper.java
deleted file mode 100644 (file)
index 2d2420d..0000000
+++ /dev/null
@@ -1,94 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.apps.io;
-
-import java.io.IOException;
-import java.io.InputStream;
-import java.io.OutputStream;
-import java.net.URI;
-import java.net.URISyntaxException;
-
-import javax.xml.transform.Source;
-import javax.xml.transform.TransformerException;
-import javax.xml.transform.stream.StreamSource;
-
-import org.apache.xmlgraphics.util.uri.DataURIResolver;
-
-public class URIResolverWrapper {
-    private final URI baseUri;
-    private final ResourceResolver uriResolver;
-    private final DataURIResolver dataSchemeResolver = new DataURIResolver();
-
-    public URIResolverWrapper(URI baseUri, ResourceResolver uriResolver) {
-        this.baseUri = baseUri;
-        this.uriResolver = uriResolver;
-    }
-
-    public URI getBaseURI() {
-        return baseUri;
-    }
-
-    public InputStream resolveIn(String stringUri) throws IOException, URISyntaxException {
-        if (stringUri.startsWith("data:")) {
-            return resolveDataURI(stringUri);
-        }
-        return resolveIn(cleanURI(stringUri));
-    }
-
-    public InputStream resolveIn(URI uri) throws IOException {
-        if (uri.getScheme() != null && uri.getScheme().startsWith("data")) {
-            return resolveDataURI(uri.toASCIIString());
-        }
-        return uriResolver.getResource(resolveFromBase(uri));
-    }
-
-    public OutputStream resolveOut(URI uri) throws IOException {
-        return uriResolver.getOutputStream(resolveFromBase(uri));
-    }
-
-    private URI resolveFromBase(URI uri) {
-        return baseUri.resolve(uri);
-    }
-
-    public static URI cleanURI(String uriStr) throws URISyntaxException {
-        // replace back slash with forward slash to ensure windows file:/// URLS are supported
-        if (uriStr == null) {
-            return null;
-        }
-        String fixedUri = uriStr.replace('\\', '/');
-        fixedUri = fixedUri.replace(" ", "%20");
-        URI baseURI = new URI(fixedUri);
-        return baseURI;
-    }
-
-    public static URI getBaseURI(String base) throws URISyntaxException {
-        String path = base + (base.endsWith("/") ? "" : "/");
-        return cleanURI(path);
-    }
-
-    private InputStream resolveDataURI(String dataURI) {
-        try {
-            Source src = dataSchemeResolver.resolve(dataURI, "");
-            return src == null ? null : ((StreamSource) src).getInputStream();
-        } catch (TransformerException e) {
-            throw new RuntimeException(e);
-        }
-    }
-}
index e0cf0fbb6e67bca0619626b9ac02ba4012372935..780c8758bc2322d6058690d8736a63569eb89692 100644 (file)
@@ -57,7 +57,7 @@ import org.apache.fop.render.print.PrintRenderer;
 import org.apache.fop.render.xml.XMLRenderer;
 import org.apache.fop.util.CommandLineLogger;
 
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.ENCRYPTION_PARAMS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.ENCRYPTION_PARAMS;
 
 /**
  * Options parses the commandline arguments
index 54b2651d7878c931f60471ae5a08601c058e9ca1..717c75d4dc8f74515fcf720e7df4340574e0863a 100644 (file)
@@ -153,7 +153,6 @@ public class DefaultFontConfigurator implements FontConfigurator<EmbedFontInfo>
 
         // no font triplet info
         if (tripletList.size() == 0) {
-            //TODO: could be problematic!!
             URI fontUri = resourceResolver.resolveFromBase(embedUri);
             if (fontUri != null) {
                 FontInfoFinder finder = new FontInfoFinder();
index 31373c8b583762132b35767c32b0e1c179f0cb51..0def335a1cbf47dd5a382371730621fc6ece70ef 100644 (file)
@@ -31,8 +31,8 @@ import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
 import org.apache.fop.apps.FOPException;
-import org.apache.fop.apps.io.ResourceResolver;
 import org.apache.fop.apps.io.InternalResourceResolver;
+import org.apache.fop.apps.io.ResourceResolver;
 import org.apache.fop.fonts.substitute.FontSubstitutions;
 import org.apache.fop.fonts.substitute.FontSubstitutionsConfigurator;
 import org.apache.fop.util.LogUtil;
@@ -74,7 +74,6 @@ public class FontManagerConfigurator {
         // caching (fonts)
         if (cfg.getChild("use-cache", false) != null) {
             try {
-                // TODO: Find some way to deal with this!!
                 if (!cfg.getChild("use-cache").getValueAsBoolean()) {
                     fontManager.disableFontCache();
                 } else {
index 739532af5ee3928f8f1f168c5c1264c79e3d7cb8..7fed1d400305c4a2b77971234de2479d9d682d0c 100644 (file)
@@ -176,7 +176,7 @@ public class FontReader extends DefaultHandler {
             try {
                 returnFont.setEmbedURI(InternalResourceResolver.cleanURI(attributes.getValue("file")));
             } catch (URISyntaxException e) {
-                // TODO: dunno what to do here?!?!
+                throw new SAXException("URI syntax error in metrics file: " + e.getMessage(), e);
             }
             returnFont.setEmbedResourceName(attributes.getValue("class"));
         } else if ("cid-widths".equals(localName)) {
index 4c0e910ab193b597b7afc1e154bc09c0410b48f0..f9bae3332f166239dc8c9773e923eeae7ed427e2 100644 (file)
@@ -190,7 +190,8 @@ public final class FontSetup {
      * @param resourceResolver the font resolver
      */
     private static void addConfiguredFonts(FontInfo fontInfo,
-            List<EmbedFontInfo> embedFontInfoList, int num, InternalResourceResolver resourceResolver,
+            List<EmbedFontInfo> embedFontInfoList, int num,
+            InternalResourceResolver resourceResolver,
             boolean base14Kerning) {
         if (embedFontInfoList == null) {
             return; //No fonts to process
index eae094b8ca988d83c9eb2ef721c63a1b23cec8da..063e54d2425f4058dd3df24c67e5c63a03ee8b87 100644 (file)
@@ -108,8 +108,8 @@ public class LazyFont extends Typeface implements FontDescriptor, Substitutable,
                     if (fontEmbedURI == null) {
                         throw new RuntimeException("Cannot load font. No font URIs available.");
                     }
-                    realFont = FontLoader.loadFont(fontEmbedURI, this.subFontName,
-                            this.embedded, this.encodingMode, useKerning, useAdvanced, resourceResolver);
+                    realFont = FontLoader.loadFont(fontEmbedURI, this.subFontName, embedded,
+                            this.encodingMode, useKerning, useAdvanced, resourceResolver);
                 }
                 if (realFont instanceof FontDescriptor) {
                     realFontDescriptor = (FontDescriptor) realFont;
index 7f666b0a2d2dc70121d31e0eaa4710039afc1fe5..ebde373b8e430e4fb4703967d0ea20f56043f3bf 100644 (file)
@@ -131,6 +131,13 @@ public class FontInfoFinder {
         return style;
     }
 
+    /**
+     * Attempts to determine FontInfo from a given custom font
+     * @param fontUri the font URI
+     * @param customFont the custom font
+     * @param fontCache font cache (may be null)
+     * @return FontInfo from the given custom font
+     */
     private EmbedFontInfo getFontInfoFromCustomFont(URI fontUri, CustomFont customFont,
             FontCache fontCache, InternalResourceResolver resourceResolver) {
         List<FontTriplet> fontTripletList = new java.util.ArrayList<FontTriplet>();
index 6e28ac44564583bbbe55eb83133ae6e0f64e3917..5c6c32e7eb47b0c176850e90d54087ff01cb5787 100644 (file)
@@ -76,9 +76,9 @@ public class Type1FontLoader extends FontLoader {
         String fontFileStr = fontFileURI.toASCIIString();
         String partialAfmUri = fontFileStr.substring(0, fontFileStr.length() - 4);
         String afmUri = null;
-        for (int i = 0; i < AFM_EXTENSIONS.length; i++) {
+        for (String afmExtension : AFM_EXTENSIONS) {
             try {
-                afmUri = partialAfmUri + AFM_EXTENSIONS[i];
+                afmUri = partialAfmUri + afmExtension;
                 afmIn = resourceResolver.getResource(afmUri);
                 if (afmIn != null) {
                     break;
@@ -86,8 +86,7 @@ public class Type1FontLoader extends FontLoader {
             } catch (IOException ioe) {
                 // Ignore, AFM probably not available under the URI
             } catch (URISyntaxException e) {
-                // TODO: Not sure what the best thing to do here is?!?
-                throw new RuntimeException(e);
+                // Ignore, AFM probably not available under the URI
             }
         }
         if (afmIn != null) {
index f14331b377019b37f66269c5a78d78d4d21755dd..31cf8e1c8afb8cb0e248a71b29ce73f9b78fed3a 100644 (file)
@@ -69,12 +69,12 @@ public final class Hyphenator {
      * The hyphenation trees are cached.
      * @param lang the language
      * @param country the country (may be null or "none")
-     * @param resourceResolver resolver to find the hyphenation files
+     * @param resolver resolver to find the hyphenation files
      * @param hyphPatNames the map with user-configured hyphenation pattern file names
      * @return the hyphenation tree
      */
     public static HyphenationTree getHyphenationTree(String lang,
-            String country, InternalResourceResolver resourceResolver, Map hyphPatNames) {
+            String country, InternalResourceResolver resolver, Map hyphPatNames) {
         String llccKey = HyphenationTreeCache.constructLlccKey(lang, country);
         HyphenationTreeCache cache = getHyphenationTreeCache();
 
@@ -83,13 +83,13 @@ public final class Hyphenator {
             return null;
         }
 
-        HyphenationTree hTree = getHyphenationTree2(lang, country, resourceResolver, hyphPatNames);
+        HyphenationTree hTree = getHyphenationTree2(lang, country, resolver, hyphPatNames);
 
         // fallback to lang only
         if (hTree == null && country != null && !country.equals("none")) {
             String llKey = HyphenationTreeCache.constructLlccKey(lang, null);
             if (!cache.isMissing(llKey)) {
-                hTree = getHyphenationTree2(lang, null, resourceResolver, hyphPatNames);
+                hTree = getHyphenationTree2(lang, null, resolver, hyphPatNames);
                 if (hTree != null && log.isDebugEnabled()) {
                     log.debug("Couldn't find hyphenation pattern "
                               + "for lang=\"" + lang + "\",country=\"" + country + "\"."
@@ -125,7 +125,7 @@ public final class Hyphenator {
      * The hyphenation trees are cached.
      * @param lang the language
      * @param country the country (may be null or "none")
-     * @param resourceResolver resource resolver to find the hyphenation files
+     * @param resourceResolver resolver to find the hyphenation files
      * @param hyphPatNames the map with user-configured hyphenation pattern file names
      * @return the hyphenation tree
      */
@@ -298,7 +298,7 @@ public final class Hyphenator {
      * Hyphenates a word.
      * @param lang the language
      * @param country the optional country code (may be null or "none")
-     * @param resourceResolver resource resolver to find the hyphenation files
+     * @param resourceResolver resolver to find the hyphenation files
      * @param hyphPatNames the map with user-configured hyphenation pattern file names
      * @param word the word to hyphenate
      * @param leftMin the minimum number of characters before the hyphenation point
diff --git a/src/java/org/apache/fop/render/DefaultRendererConfigurator.java b/src/java/org/apache/fop/render/DefaultRendererConfigurator.java
new file mode 100644 (file)
index 0000000..061c257
--- /dev/null
@@ -0,0 +1,48 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.render;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.apache.fop.apps.FOUserAgent;
+import org.apache.fop.fonts.FontCollection;
+import org.apache.fop.fonts.FontManager;
+import org.apache.fop.fonts.base14.Base14FontCollection;
+import org.apache.fop.render.RendererConfig.RendererConfigParser;
+
+/**
+ * This object represents the default renderer configurator and contains the methods for most the
+ * RendererConfigurators.
+ */
+public class DefaultRendererConfigurator extends PrintRendererConfigurator {
+
+    public DefaultRendererConfigurator(FOUserAgent userAgent, RendererConfigParser rendererConfigParser) {
+        super(userAgent, rendererConfigParser);
+    }
+
+    protected List<FontCollection> getDefaultFontCollection() {
+        FontManager fontManager = userAgent.getFontManager();
+        List<FontCollection> fontCollection = new ArrayList<FontCollection>();
+        fontCollection.add(new Base14FontCollection(fontManager.isBase14KerningEnabled()));
+        return fontCollection;
+    }
+
+}
index 832f5e83ff5c94b072eff356a6fbbde86ec83bf4..4b3b63a51de88bcb8241e413d3bddabe2f2acc7a 100644 (file)
@@ -25,14 +25,40 @@ import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.fonts.FontConfig;
 
+/**
+ * Implementations of this interface have all the renderer-specific configuration data found in the
+ * FOP-conf. This object is just a data object that is created by the {@link RendererConfigParser}
+ * when the FOP conf is parsed.
+ */
 public interface RendererConfig {
 
+    /**
+     * Returns the render-specific font configuration information.
+     * @return the font config
+     */
     FontConfig getFontInfoConfig();
 
+    /**
+     * Implementations of this interface parse the relevant renderer-specific configuration data
+     * within the FOP-conf and create a {@link RendererConfig}.
+     */
     public interface RendererConfigParser {
 
+        /**
+         * Builds the object that contains the renderer configuration data.
+         *
+         * @param userAgent the user agent
+         * @param rendererConfiguration the Avalon config object for parsing the data
+         * @return the configuration data object
+         * @throws FOPException if an error occurs while parsing the fop conf
+         */
         RendererConfig build(FOUserAgent userAgent, Configuration rendererConfiguration) throws FOPException;
 
+        /**
+         * The MIME type of the renderer.
+         *
+         * @return the mime type
+         */
         String getMimeType();
     }
 }
diff --git a/src/java/org/apache/fop/render/RendererConfigOption.java b/src/java/org/apache/fop/render/RendererConfigOption.java
new file mode 100644 (file)
index 0000000..5b670d0
--- /dev/null
@@ -0,0 +1,29 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.render;
+
+/**
+ * Implementations of this interface enumerate the various renderer configuration options available
+ * to users in the FOP conf.
+ */
+public interface RendererConfigOption {
+    /** The name of the option. */
+    String getName();
+}
diff --git a/src/java/org/apache/fop/render/RendererConfigOptions.java b/src/java/org/apache/fop/render/RendererConfigOptions.java
deleted file mode 100644 (file)
index 9a4e734..0000000
+++ /dev/null
@@ -1,24 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.render;
-
-public interface RendererConfigOptions {
-    String getName();
-}
diff --git a/src/java/org/apache/fop/render/adobe/AdobeRendererConfigurator.java b/src/java/org/apache/fop/render/adobe/AdobeRendererConfigurator.java
deleted file mode 100644 (file)
index 16a29ba..0000000
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.render.adobe;
-
-import java.util.ArrayList;
-import java.util.List;
-
-import org.apache.fop.apps.FOUserAgent;
-import org.apache.fop.fonts.FontCollection;
-import org.apache.fop.fonts.FontManager;
-import org.apache.fop.fonts.base14.Base14FontCollection;
-import org.apache.fop.render.PrintRendererConfigurator;
-import org.apache.fop.render.RendererConfig.RendererConfigParser;
-
-public class AdobeRendererConfigurator extends PrintRendererConfigurator {
-
-    public AdobeRendererConfigurator(FOUserAgent userAgent, RendererConfigParser rendererConfigParser) {
-        super(userAgent, rendererConfigParser);
-    }
-
-    protected List<FontCollection> getDefaultFontCollection() {
-        FontManager fontManager = userAgent.getFontManager();
-        List<FontCollection> fontCollection = new ArrayList<FontCollection>();
-        fontCollection.add(new Base14FontCollection(fontManager.isBase14KerningEnabled()));
-        return fontCollection;
-    }
-
-}
index f8535723893a2de4cb80ac3e1f4769b2c852a546..6b031c9ebc0373fe391c1e9360e5ed374095ba75 100644 (file)
@@ -20,6 +20,7 @@
 package org.apache.fop.render.afp;
 
 import java.net.URI;
+import java.net.URISyntaxException;
 import java.util.EnumMap;
 
 import org.apache.avalon.framework.configuration.Configuration;
@@ -36,6 +37,7 @@ import org.apache.fop.afp.modca.triplets.MappingOptionTriplet;
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.apps.MimeConstants;
+import org.apache.fop.apps.io.InternalResourceResolver;
 import org.apache.fop.fonts.FontManager;
 import org.apache.fop.render.RendererConfig;
 import org.apache.fop.render.afp.AFPFontConfig.AFPFontInfoConfigParser;
@@ -43,26 +45,32 @@ import org.apache.fop.util.LogUtil;
 
 import static org.apache.fop.render.afp.AFPRendererConfig.ImagesModeOptions.MODE_COLOR;
 import static org.apache.fop.render.afp.AFPRendererConfig.ImagesModeOptions.MODE_GRAYSCALE;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.DEFAULT_RESOURCE_LEVELS;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.GOCA;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.GOCA_TEXT;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_DITHERING_QUALITY;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_FS45;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_JPEG;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_MAPPING_OPTION;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_MODE;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_NATIVE;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_WRAP_PSEG;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.JPEG_ALLOW_JPEG_EMBEDDING;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.JPEG_BITMAP_ENCODING_QUALITY;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.LINE_WIDTH_CORRECTION;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.RENDERER_RESOLUTION;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.RESOURCE_GROUP_URI;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.SHADING;
-
+import static org.apache.fop.render.afp.AFPRendererConfigOption.DEFAULT_RESOURCE_LEVELS;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.GOCA;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.GOCA_TEXT;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_DITHERING_QUALITY;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_FS45;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_JPEG;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_MAPPING_OPTION;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_MODE;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_NATIVE;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_WRAP_PSEG;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.JPEG_ALLOW_JPEG_EMBEDDING;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.JPEG_BITMAP_ENCODING_QUALITY;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.LINE_WIDTH_CORRECTION;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.RENDERER_RESOLUTION;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.RESOURCE_GROUP_URI;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.SHADING;
+
+/**
+ * The AFP renderer config object.
+ */
 public final class AFPRendererConfig implements RendererConfig {
 
+    /**
+     * An enumeration for the various images modes available to the AFP renderer.
+     */
     public enum ImagesModeOptions {
 
         MODE_GRAYSCALE("b+w", "bits-per-pixel"),
@@ -95,41 +103,7 @@ public final class AFPRendererConfig implements RendererConfig {
         }
     }
 
-    public enum Options {
-
-        DEFAULT_RESOURCE_LEVELS("default-resource-levels", AFPResourceLevelDefaults.class),
-        IMAGES("images", null),
-        IMAGES_JPEG("jpeg", null),
-        IMAGES_DITHERING_QUALITY("dithering-quality", Float.class),
-        IMAGES_FS45("fs45", Boolean.class),
-        IMAGES_MAPPING_OPTION("mapping_option", Byte.class),
-        IMAGES_MODE("mode", Boolean.class),
-        IMAGES_NATIVE("native", Boolean.class),
-        IMAGES_WRAP_PSEG("pseg", Boolean.class),
-        JPEG_ALLOW_JPEG_EMBEDDING("allow-embedding", Boolean.class),
-        JPEG_BITMAP_ENCODING_QUALITY("bitmap-encoding-quality", Float.class),
-        RENDERER_RESOLUTION("renderer-resolution", Integer.class),
-        RESOURCE_GROUP_URI("resource-group-file", URI.class),
-        SHADING("shading", AFPShadingMode.class),
-        LINE_WIDTH_CORRECTION("line-width-correction", Float.class),
-        GOCA("goca", Boolean.class),
-        GOCA_TEXT("text", Boolean.class);
-
-        private final String name;
-
-        private final Class<?> type;
-
-        private Options(String name, Class<?> type) {
-            this.name = name;
-            this.type = type;
-        }
-
-        public String getName() {
-            return name;
-        }
-    }
-
-    private final EnumMap<Options, Object> params = new EnumMap<Options, Object>(Options.class);
+    private final EnumMap<AFPRendererConfigOption, Object> params = new EnumMap<AFPRendererConfigOption, Object>(AFPRendererConfigOption.class);
 
     private final EnumMap<ImagesModeOptions, Object> imageModeParams
     = new EnumMap<ImagesModeOptions, Object>(ImagesModeOptions.class);
@@ -215,20 +189,24 @@ public final class AFPRendererConfig implements RendererConfig {
         return getParam(GOCA_TEXT, Boolean.class);
     }
 
-    private <T> T getParam(Options options, Class<T> type) {
-        assert options.type.equals(type);
+    private <T> T getParam(AFPRendererConfigOption options, Class<T> type) {
+        assert options.getType().equals(type);
         return type.cast(params.get(options));
     }
 
-    private <T> void setParam(Options option, T value) {
-        assert option.type.isInstance(value);
+    private <T> void setParam(AFPRendererConfigOption option, T value) {
+        assert option.getType().isInstance(value);
         params.put(option, value);
     }
 
+    /**
+     * The parser for AFP renderer specific data in the FOP conf.
+     */
     public static final class AFPRendererConfigParser implements RendererConfigParser {
 
         private static final Log LOG = LogFactory.getLog(AFPRendererConfigParser.class);
 
+        /** {@inheritDoc} */
         public AFPRendererConfig build(FOUserAgent userAgent, Configuration cfg) throws FOPException {
             boolean strict = userAgent != null ? userAgent.validateUserConfigStrictly() : false;
             AFPRendererConfig config = null;
@@ -241,12 +219,12 @@ public final class AFPRendererConfig implements RendererConfig {
             return config;
         }
 
+        /** {@inheritDoc} */
         public String getMimeType() {
             return MimeConstants.MIME_AFP;
         }
     }
 
-
     private static final class ParserHelper {
 
         private static final Log LOG = LogFactory.getLog(ParserHelper.class);
@@ -294,7 +272,7 @@ public final class AFPRendererConfig implements RendererConfig {
             createResourceLevel();
         }
 
-        private void setParam(Options option, Object value) {
+        private void setParam(AFPRendererConfigOption option, Object value) {
             config.setParam(option, value);
         }
 
@@ -311,7 +289,7 @@ public final class AFPRendererConfig implements RendererConfig {
                 config.imageModeParams.put(MODE_GRAYSCALE,
                         imagesCfg.getAttributeAsInteger(imagesMode.getModeAttribute(), 8));
             }
-            String dithering = imagesCfg.getAttribute(Options.IMAGES_DITHERING_QUALITY.getName(), "medium");
+            String dithering = imagesCfg.getAttribute(AFPRendererConfigOption.IMAGES_DITHERING_QUALITY.getName(), "medium");
             float dq;
             if (dithering.startsWith("min")) {
                 dq = 0.0f;
@@ -326,7 +304,7 @@ public final class AFPRendererConfig implements RendererConfig {
                 }
             }
             setParam(IMAGES_DITHERING_QUALITY, dq);
-            setParam(IMAGES_NATIVE, imagesCfg.getAttributeAsBoolean(Options.IMAGES_NATIVE.getName(), false));
+            setParam(IMAGES_NATIVE, imagesCfg.getAttributeAsBoolean(IMAGES_NATIVE.getName(), false));
             setParam(IMAGES_WRAP_PSEG,
                     imagesCfg.getAttributeAsBoolean(IMAGES_WRAP_PSEG.getName(), false));
             setParam(IMAGES_FS45, imagesCfg.getAttributeAsBoolean(IMAGES_FS45.getName(), false));
@@ -364,12 +342,13 @@ public final class AFPRendererConfig implements RendererConfig {
             try {
                 Configuration resourceGroupUriCfg = cfg.getChild(RESOURCE_GROUP_URI.getName(), false);
                 if (resourceGroupUriCfg != null) {
-                    URI resourceGroupUri = URI.create(resourceGroupUriCfg.getValue());
-                    // TODO validate?
+                    URI resourceGroupUri = InternalResourceResolver.cleanURI(resourceGroupUriCfg.getValue());
                     setParam(RESOURCE_GROUP_URI, resourceGroupUri);
                 }
             } catch (ConfigurationException e) {
                 LogUtil.handleException(LOG, e, strict);
+            } catch (URISyntaxException use) {
+                LogUtil.handleException(LOG, use, strict);
             }
         }
 
diff --git a/src/java/org/apache/fop/render/afp/AFPRendererConfigOption.java b/src/java/org/apache/fop/render/afp/AFPRendererConfigOption.java
new file mode 100644 (file)
index 0000000..5a99ae3
--- /dev/null
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.fop.render.afp;
+
+import java.net.URI;
+
+import org.apache.fop.afp.AFPResourceLevelDefaults;
+import org.apache.fop.render.RendererConfigOption;
+
+/**
+ * An enumeration of the configuration options available for the AFP renderer.
+ */
+public enum AFPRendererConfigOption implements RendererConfigOption {
+    DEFAULT_RESOURCE_LEVELS("default-resource-levels", AFPResourceLevelDefaults.class),
+    IMAGES("images", null),
+    IMAGES_JPEG("jpeg", null),
+    IMAGES_DITHERING_QUALITY("dithering-quality", Float.class),
+    IMAGES_FS45("fs45", Boolean.class),
+    IMAGES_MAPPING_OPTION("mapping_option", Byte.class),
+    IMAGES_MODE("mode", Boolean.class),
+    IMAGES_NATIVE("native", Boolean.class),
+    IMAGES_WRAP_PSEG("pseg", Boolean.class),
+    JPEG_ALLOW_JPEG_EMBEDDING("allow-embedding", Boolean.class),
+    JPEG_BITMAP_ENCODING_QUALITY("bitmap-encoding-quality", Float.class),
+    RENDERER_RESOLUTION("renderer-resolution", Integer.class),
+    RESOURCE_GROUP_URI("resource-group-file", URI.class),
+    SHADING("shading", AFPShadingMode.class),
+    LINE_WIDTH_CORRECTION("line-width-correction", Float.class),
+    GOCA("goca", Boolean.class),
+    GOCA_TEXT("text", Boolean.class);
+
+    private final String name;
+
+    private final Class<?> type;
+
+    private AFPRendererConfigOption(String name, Class<?> type) {
+        this.name = name;
+        this.type = type;
+    }
+
+    /** {@inheritDoc}}*/
+    public String getName() {
+        return name;
+    }
+
+    public Class<?> getType() {
+        return type;
+    }
+}
index 0efb2b93088236cee97c5008627dbcd6e3ddfa01..e76f941d8528b4f5826849a5d4f6994152be0bca 100644 (file)
@@ -34,18 +34,21 @@ import org.apache.fop.render.java2d.Java2DRendererConfig;
 import org.apache.fop.render.java2d.Java2DRendererConfig.Java2DRendererConfigParser;
 import org.apache.fop.util.ColorUtil;
 
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.ANTI_ALIASING;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.BACKGROUND_COLOR;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_QUALITY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_QUALITY_ELEMENT;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_SPEED;
-
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.ANTI_ALIASING;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.BACKGROUND_COLOR;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_QUALITY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_QUALITY_ELEMENT;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_SPEED;
+
+/**
+ * The Bitmap renderer config data object.
+ */
 public class BitmapRendererConfig implements RendererConfig {
 
-    private final EnumMap<BitmapRendererConfigOptions, Object> params
-            = new EnumMap<BitmapRendererConfigOptions, Object>(BitmapRendererConfigOptions.class);
+    private final EnumMap<BitmapRendererConfigOption, Object> params
+            = new EnumMap<BitmapRendererConfigOption, Object>(BitmapRendererConfigOption.class);
 
     private final DefaultFontConfig fontConfig;
 
@@ -79,10 +82,13 @@ public class BitmapRendererConfig implements RendererConfig {
                 : JAVA2D_TRANSPARENT_PAGE_BACKGROUND.getDefaultValue());
     }
 
-    private Object get(BitmapRendererConfigOptions option) {
+    private Object get(BitmapRendererConfigOption option) {
         return params.get(option);
     }
 
+    /**
+     * The parser for the Bitmap renderer configuration data.
+     */
     public static class BitmapRendererConfigParser implements RendererConfigParser {
 
         private final String mimeType;
@@ -91,12 +97,12 @@ public class BitmapRendererConfig implements RendererConfig {
             this.mimeType = mimeType;
         }
 
-        private void setParam(BitmapRendererConfig config, BitmapRendererConfigOptions option,
+        private void setParam(BitmapRendererConfig config, BitmapRendererConfigOption option,
                 Object value) {
             config.params.put(option, value != null ? value : option.getDefaultValue());
         }
 
-        protected void build(BitmapRendererConfig config, FOUserAgent userAgent,
+        void build(BitmapRendererConfig config, FOUserAgent userAgent,
                 Configuration cfg) throws FOPException {
             if (cfg != null) {
                 Java2DRendererConfig j2dConfig = new Java2DRendererConfigParser(null).build(
@@ -116,17 +122,17 @@ public class BitmapRendererConfig implements RendererConfig {
                             ColorUtil.parseColorString(userAgent, background));
                 }
 
-                setParam(config, BitmapRendererConfigOptions.ANTI_ALIASING,
+                setParam(config, BitmapRendererConfigOption.ANTI_ALIASING,
                         getChild(cfg, ANTI_ALIASING).getValueAsBoolean(
                                 (Boolean) ANTI_ALIASING.getDefaultValue()));
 
                 String optimization = getValue(cfg, RENDERING_QUALITY_ELEMENT);
                 setParam(config, RENDERING_QUALITY,
-                        !(BitmapRendererConfigOptions.getValue(optimization) == RENDERING_SPEED));
+                        !(BitmapRendererConfigOption.getValue(optimization) == RENDERING_SPEED));
 
                 String color = getValue(cfg, COLOR_MODE);
                 setParam(config, COLOR_MODE,
-                        getBufferedImageIntegerFromColor(BitmapRendererConfigOptions.getValue(color)));
+                        getBufferedImageIntegerFromColor(BitmapRendererConfigOption.getValue(color)));
             }
         }
 
@@ -138,7 +144,7 @@ public class BitmapRendererConfig implements RendererConfig {
             return config;
         }
 
-        private Integer getBufferedImageIntegerFromColor(BitmapRendererConfigOptions option) {
+        private Integer getBufferedImageIntegerFromColor(BitmapRendererConfigOption option) {
             if (option == null) {
                 return null;
             }
@@ -157,11 +163,11 @@ public class BitmapRendererConfig implements RendererConfig {
             }
         }
 
-        private Configuration getChild(Configuration cfg, BitmapRendererConfigOptions option) {
+        private Configuration getChild(Configuration cfg, BitmapRendererConfigOption option) {
             return cfg.getChild(option.getName());
         }
 
-        private String getValue(Configuration cfg, BitmapRendererConfigOptions option) {
+        private String getValue(Configuration cfg, BitmapRendererConfigOption option) {
             return cfg.getChild(option.getName()).getValue(null);
         }
 
diff --git a/src/java/org/apache/fop/render/bitmap/BitmapRendererConfigOption.java b/src/java/org/apache/fop/render/bitmap/BitmapRendererConfigOption.java
new file mode 100644 (file)
index 0000000..00b043a
--- /dev/null
@@ -0,0 +1,74 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.render.bitmap;
+
+import java.awt.Color;
+import java.awt.image.BufferedImage;
+
+import org.apache.fop.render.RendererConfigOption;
+
+/**
+ * An enumeration of the Bitmap renderer configuration opetions available to the user via the FOP
+ * conf.
+ */
+public enum BitmapRendererConfigOption implements RendererConfigOption {
+    JAVA2D_TRANSPARENT_PAGE_BACKGROUND("transparent-page-background", false),
+    BACKGROUND_COLOR("background-color", Color.WHITE),
+    ANTI_ALIASING("anti-aliasing", true),
+    RENDERING_QUALITY_ELEMENT("rendering"),
+    RENDERING_QUALITY("quality", true),
+    RENDERING_SPEED("speed"),
+    COLOR_MODE("color-mode", BufferedImage.TYPE_INT_ARGB),
+    COLOR_MODE_RGBA("rgba"),
+    COLOR_MODE_RGB("rgb"),
+    COLOR_MODE_GRAY("gray"),
+    COLOR_MODE_BINARY("binary"),
+    COLOR_MODE_BILEVEL("bi-level");
+
+    private final String name;
+    private final Object defaultValue;
+
+    private BitmapRendererConfigOption(String name, Object defaultValue) {
+        this.name = name;
+        this.defaultValue = defaultValue;
+    }
+
+    private BitmapRendererConfigOption(String name) {
+        this(name, null);
+    }
+
+    /** {@inheritDoc} */
+    public String getName() {
+        return name;
+    }
+
+    public Object getDefaultValue() {
+        return defaultValue;
+    }
+
+    public static BitmapRendererConfigOption getValue(String str) {
+        for (BitmapRendererConfigOption opt : BitmapRendererConfigOption.values()) {
+            if (opt.getName().equalsIgnoreCase(str)) {
+                return opt;
+            }
+        }
+        return null;
+    }
+}
diff --git a/src/java/org/apache/fop/render/bitmap/BitmapRendererConfigOptions.java b/src/java/org/apache/fop/render/bitmap/BitmapRendererConfigOptions.java
deleted file mode 100644 (file)
index 2fe0109..0000000
+++ /dev/null
@@ -1,69 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.render.bitmap;
-
-import java.awt.Color;
-import java.awt.image.BufferedImage;
-
-import org.apache.fop.render.RendererConfigOptions;
-
-public enum BitmapRendererConfigOptions implements RendererConfigOptions {
-    JAVA2D_TRANSPARENT_PAGE_BACKGROUND("transparent-page-background", false),
-    BACKGROUND_COLOR("background-color", Color.WHITE),
-    ANTI_ALIASING("anti-aliasing", true),
-    RENDERING_QUALITY_ELEMENT("rendering"),
-    RENDERING_QUALITY("quality", true),
-    RENDERING_SPEED("speed"),
-    COLOR_MODE("color-mode", BufferedImage.TYPE_INT_ARGB),
-    COLOR_MODE_RGBA("rgba"),
-    COLOR_MODE_RGB("rgb"),
-    COLOR_MODE_GRAY("gray"),
-    COLOR_MODE_BINARY("binary"),
-    COLOR_MODE_BILEVEL("bi-level");
-
-    private final String name;
-    private final Object defaultValue;
-
-    private BitmapRendererConfigOptions(String name, Object defaultValue) {
-        this.name = name;
-        this.defaultValue = defaultValue;
-    }
-
-    private BitmapRendererConfigOptions(String name) {
-        this(name, null);
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public Object getDefaultValue() {
-        return defaultValue;
-    }
-
-    public static BitmapRendererConfigOptions getValue(String str) {
-        for (BitmapRendererConfigOptions opt : BitmapRendererConfigOptions.values()) {
-            if (opt.getName().equalsIgnoreCase(str)) {
-                return opt;
-            }
-        }
-        return null;
-    }
-}
index 4f76cbb7d6b18e33871bcbc278e0cb9d0e7e5c6c..f0e32a9d74cbc64da6b1371b3809d64ba3bbbc09 100644 (file)
@@ -24,6 +24,7 @@ import java.util.List;
 
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
+import org.apache.fop.apps.MimeConstants;
 import org.apache.fop.apps.io.InternalResourceResolver;
 import org.apache.fop.fonts.EmbedFontInfo;
 import org.apache.fop.fonts.FontCollection;
@@ -55,8 +56,8 @@ public class BitmapRendererConfigurator extends Java2DRendererConfigurator {
     public void configure(IFDocumentHandler documentHandler) throws FOPException {
         AbstractBitmapDocumentHandler bitmapHandler = (AbstractBitmapDocumentHandler) documentHandler;
         BitmapRenderingSettings settings = bitmapHandler.getSettings();
-        // TODO: magic strings!
-        configure(documentHandler, settings, new BitmapRendererConfigParser("image/bitmap"));
+        configure(documentHandler, settings,
+                new BitmapRendererConfigParser(MimeConstants.MIME_BITMAP));
     }
 
     void configure(IFDocumentHandler documentHandler, BitmapRenderingSettings settings,
index 8c9a68b0242b3174a221b794a728654f613c8fa3..0648b4af0771f1eb778e923558c09dbe1556892c 100644 (file)
@@ -23,9 +23,9 @@ import org.apache.xmlgraphics.image.writer.ImageWriterParams;
 
 import org.apache.fop.render.java2d.Java2DRenderingSettings;
 
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.ANTI_ALIASING;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_QUALITY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.ANTI_ALIASING;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_QUALITY;
 
 /**
  * This class holds settings used when rendering to bitmaps.
index 14984963c277f4b9bc7c8a52db068bce39721dce..699faea5153872a030fad6ae108b57ef7f5a1e71 100644 (file)
@@ -26,12 +26,18 @@ import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.fonts.DefaultFontConfig;
 import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
 
+/**
+ * The PNG renderer configuration data object.
+ */
 public final class PNGRendererConfig extends BitmapRendererConfig {
 
     private PNGRendererConfig(DefaultFontConfig fontConfig) {
         super(fontConfig);
     }
 
+    /**
+     * The PNG renderer configuration parser.
+     */
     public static class PNGRendererConfigParser implements RendererConfigParser {
 
         public PNGRendererConfig build(FOUserAgent userAgent, Configuration cfg)
@@ -40,6 +46,7 @@ public final class PNGRendererConfig extends BitmapRendererConfig {
                     userAgent.validateStrictly()));
         }
 
+        /** {@inheritDoc} */
         public String getMimeType() {
             return MimeConstants.MIME_PNG;
         }
index 12bd8aa1e868721c45f03421f7f5a5838a47789d..fb38b61f61f9cd0fbf0d51f0ad275d3ce8d623ea 100644 (file)
@@ -24,6 +24,7 @@ import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.apps.MimeConstants;
 import org.apache.fop.render.AbstractRendererMaker;
 import org.apache.fop.render.Renderer;
+import org.apache.fop.render.bitmap.PNGRendererConfig.PNGRendererConfigParser;
 import org.apache.fop.render.java2d.Java2DRendererConfigurator;
 
 /**
@@ -40,8 +41,7 @@ public class PNGRendererMaker extends AbstractRendererMaker {
 
     @Override
     public void configureRenderer(FOUserAgent userAgent, Renderer renderer) throws FOPException {
-        // TODO what constructir params?
-        new Java2DRendererConfigurator(userAgent, null).configure(renderer);
+        new Java2DRendererConfigurator(userAgent, new PNGRendererConfigParser()).configure(renderer);
     }
 
     @Override
index 68401d9bc95c4b8a553a16ba4f2dd9df8b0aa844..152b669b6d9b2e9c9bdaec050c0915804012156e 100644 (file)
@@ -29,17 +29,20 @@ import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.fonts.DefaultFontConfig;
 import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
-import org.apache.fop.render.RendererConfigOptions;
+import org.apache.fop.render.RendererConfigOption;
 
+/**
+ * The renderer configuration object for the TIFF renderer.
+ */
 public final class TIFFRendererConfig extends BitmapRendererConfig {
 
-    public enum TIFFRendererConfigOptions implements RendererConfigOptions {
+    public enum TIFFRendererConfigOption implements RendererConfigOption {
         COMPRESSION("compression", TIFFCompressionValues.PACKBITS);
 
         private final String name;
         private final Object defaultValue;
 
-        private TIFFRendererConfigOptions(String name, Object defaultValue) {
+        private TIFFRendererConfigOption(String name, Object defaultValue) {
             this.name = name;
             this.defaultValue = defaultValue;
         }
@@ -53,17 +56,20 @@ public final class TIFFRendererConfig extends BitmapRendererConfig {
         }
     }
 
-    private final EnumMap<TIFFRendererConfigOptions, Object> params
-            = new EnumMap<TIFFRendererConfigOptions, Object>(TIFFRendererConfigOptions.class);
+    private final EnumMap<TIFFRendererConfigOption, Object> params
+            = new EnumMap<TIFFRendererConfigOption, Object>(TIFFRendererConfigOption.class);
 
     private TIFFRendererConfig(DefaultFontConfig fontConfig) {
         super(fontConfig);
     }
 
     public TIFFCompressionValues getCompressionType() {
-        return (TIFFCompressionValues) params.get(TIFFRendererConfigOptions.COMPRESSION);
+        return (TIFFCompressionValues) params.get(TIFFRendererConfigOption.COMPRESSION);
     }
 
+    /**
+     * The TIFF renderer configuration parser.
+     */
     public static final class TIFFRendererConfigParser extends BitmapRendererConfigParser {
 
         public TIFFRendererConfigParser() {
@@ -72,22 +78,23 @@ public final class TIFFRendererConfig extends BitmapRendererConfig {
 
         private TIFFRendererConfig config;
 
-        private void setParam(TIFFRendererConfigOptions option, Object value) {
+        private void setParam(TIFFRendererConfigOption option, Object value) {
             config.params.put(option, value != null ? value : option.getDefaultValue());
         }
 
-        private String getValue(Configuration cfg, TIFFRendererConfigOptions option) {
+        private String getValue(Configuration cfg, TIFFRendererConfigOption option) {
             return cfg.getChild(option.getName()).getValue(null);
         }
 
+        /** {@inheritDoc} */
         public TIFFRendererConfig build(FOUserAgent userAgent, Configuration cfg) throws FOPException {
             config = new TIFFRendererConfig(new DefaultFontConfigParser()
                     .parse(cfg, userAgent.validateStrictly()));
             super.build(config, userAgent, cfg);
             if (cfg != null) {
-            setParam(TIFFRendererConfigOptions.COMPRESSION,
+            setParam(TIFFRendererConfigOption.COMPRESSION,
                         TIFFCompressionValues.getValue(getValue(cfg,
-                                TIFFRendererConfigOptions.COMPRESSION)));
+                                TIFFRendererConfigOption.COMPRESSION)));
             }
             return config;
         }
index 27d9d6eaa35669d5bb8d046dd34aaed9b3baa798..753d9ff30f8324a5da15adb94f82c79e53588b42 100644 (file)
@@ -28,6 +28,9 @@ import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
 import org.apache.fop.fonts.FontConfig;
 import org.apache.fop.render.RendererConfig;
 
+/**
+ * The Intermediate Format renderer configuration data object.
+ */
 public final class IFRendererConfig implements RendererConfig {
 
     private final DefaultFontConfig fontConfig;
@@ -40,14 +43,19 @@ public final class IFRendererConfig implements RendererConfig {
         return fontConfig;
     }
 
+    /**
+     * The Intermediate Format configuration data parser.
+     */
     public static final class IFRendererConfigParser implements RendererConfigParser {
 
+        /** {@inheritDoc} */
         public RendererConfig build(FOUserAgent userAgent, Configuration cfg)
                 throws FOPException {
             return new IFRendererConfig(new DefaultFontConfigParser().parse(cfg,
                     userAgent.validateStrictly()));
         }
 
+        /** {@inheritDoc} */
         public String getMimeType() {
             return "application/X-fop-intermediate-format";
         }
index 3d20440f54c493e3eb1d32bb77c834b7bd6e35b9..41d1b01ef72073e42bee20609d4d3dcb90c8172b 100644 (file)
@@ -40,8 +40,8 @@ import org.apache.xmlgraphics.util.XMLizable;
 import org.apache.fop.accessibility.StructureTreeEventHandler;
 import org.apache.fop.fo.extensions.InternalElementMapping;
 import org.apache.fop.fonts.FontInfo;
+import org.apache.fop.render.DefaultRendererConfigurator;
 import org.apache.fop.render.RenderingContext;
-import org.apache.fop.render.adobe.AdobeRendererConfigurator;
 import org.apache.fop.render.intermediate.IFRendererConfig.IFRendererConfigParser;
 import org.apache.fop.render.intermediate.IFStructureTreeBuilder.IFStructureTreeElement;
 import org.apache.fop.render.intermediate.extensions.AbstractAction;
@@ -111,7 +111,7 @@ implements IFConstants, IFPainter, IFDocumentNavigationHandler {
         if (this.mimicHandler != null) {
             return getMimickedDocumentHandler().getConfigurator();
         } else {
-            return new AdobeRendererConfigurator(getUserAgent(), new IFRendererConfigParser());
+            return new DefaultRendererConfigurator(getUserAgent(), new IFRendererConfigParser());
         }
     }
 
index be5405cbb8f9ee50f7b594e29173ced585048fb2..91d38675d34ad4f2bd28d39f0f5e5a89a910cce8 100644 (file)
@@ -84,7 +84,7 @@ import org.apache.fop.render.pdf.CTMHelper;
 import org.apache.fop.util.CharUtilities;
 import org.apache.fop.util.ColorUtil;
 
-import static org.apache.fop.render.java2d.Java2DRendererOptions.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
+import static org.apache.fop.render.java2d.Java2DRendererOption.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
 
 /**
  * The <code>Java2DRenderer</code> class provides the abstract technical
index 139d05916018b53aa08db428f98c2d461f7e4b36..1ed0aec9659aaa3f01c099c396bfe232c992a540 100644 (file)
@@ -29,12 +29,15 @@ import org.apache.fop.fonts.DefaultFontConfig;
 import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
 import org.apache.fop.render.RendererConfig;
 
-import static org.apache.fop.render.java2d.Java2DRendererOptions.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
+import static org.apache.fop.render.java2d.Java2DRendererOption.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
 
+/**
+ * The Java2D renderer configuration data object.
+ */
 public final class Java2DRendererConfig implements RendererConfig {
 
-    private final EnumMap<Java2DRendererOptions, Object> params
-            = new EnumMap<Java2DRendererOptions, Object>(Java2DRendererOptions.class);
+    private final EnumMap<Java2DRendererOption, Object> params
+            = new EnumMap<Java2DRendererOption, Object>(Java2DRendererOption.class);
 
     private final DefaultFontConfig fontConfig;
 
@@ -50,6 +53,9 @@ public final class Java2DRendererConfig implements RendererConfig {
         return Boolean.class.cast(params.get(JAVA2D_TRANSPARENT_PAGE_BACKGROUND));
     }
 
+    /**
+     * The Java2D renderer configuration data parser.
+     */
     public static class Java2DRendererConfigParser implements RendererConfigParser {
 
         private final String mimeType;
@@ -58,6 +64,7 @@ public final class Java2DRendererConfig implements RendererConfig {
             this.mimeType = mimeType;
         }
 
+        /** {@inheritDoc} */
         public Java2DRendererConfig build(FOUserAgent userAgent, Configuration cfg)
                 throws FOPException {
             Java2DRendererConfig config = new Java2DRendererConfig(new DefaultFontConfigParser()
@@ -68,6 +75,7 @@ public final class Java2DRendererConfig implements RendererConfig {
             return config;
         }
 
+        /** {@inheritDoc} */
         public String getMimeType() {
             return mimeType;
         }
index 6b09ff740447b3d684bfc85b7e0587e8aeb813b7..9ff2ccdbdf9827eeb71f5948eb5d6140b94438d9 100644 (file)
@@ -21,14 +21,14 @@ package org.apache.fop.render.java2d;
 
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
+import org.apache.fop.render.DefaultRendererConfigurator;
 import org.apache.fop.render.Renderer;
 import org.apache.fop.render.RendererConfig.RendererConfigParser;
-import org.apache.fop.render.adobe.AdobeRendererConfigurator;
 
 /**
  * Configurerer for Java 2D renderer
  */
-public class Java2DRendererConfigurator extends AdobeRendererConfigurator {
+public class Java2DRendererConfigurator extends DefaultRendererConfigurator {
 
     /**
      * Default constructor
diff --git a/src/java/org/apache/fop/render/java2d/Java2DRendererOption.java b/src/java/org/apache/fop/render/java2d/Java2DRendererOption.java
new file mode 100644 (file)
index 0000000..b25f058
--- /dev/null
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.render.java2d;
+
+import org.apache.fop.render.RendererConfigOption;
+
+public enum Java2DRendererOption implements RendererConfigOption {
+    /** Rendering Options key for the controlling the transparent page background option. */
+    JAVA2D_TRANSPARENT_PAGE_BACKGROUND("transparent-page-background");
+
+    private final String name;
+
+    private Java2DRendererOption(String name) {
+        this.name = name;
+    }
+
+    public String getName() {
+        return name;
+    }
+}
diff --git a/src/java/org/apache/fop/render/java2d/Java2DRendererOptions.java b/src/java/org/apache/fop/render/java2d/Java2DRendererOptions.java
deleted file mode 100644 (file)
index c531aa7..0000000
+++ /dev/null
@@ -1,37 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.render.java2d;
-
-import org.apache.fop.render.RendererConfigOptions;
-
-public enum Java2DRendererOptions implements RendererConfigOptions {
-    /** Rendering Options key for the controlling the transparent page background option. */
-    JAVA2D_TRANSPARENT_PAGE_BACKGROUND("transparent-page-background");
-
-    private final String name;
-
-    private Java2DRendererOptions(String name) {
-        this.name = name;
-    }
-
-    public String getName() {
-        return name;
-    }
-}
diff --git a/src/java/org/apache/fop/render/pcl/Java2DRendererConfigOption.java b/src/java/org/apache/fop/render/pcl/Java2DRendererConfigOption.java
new file mode 100644 (file)
index 0000000..a3ce884
--- /dev/null
@@ -0,0 +1,49 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.fop.render.pcl;
+
+import org.apache.fop.render.RendererConfigOption;
+
+/**
+ * An enumeration of the renderer configuration options available to the Java2D renderer via the
+ * FOP conf.
+ */
+public enum Java2DRendererConfigOption implements RendererConfigOption {
+
+    RENDERING_MODE("rendering", PCLRenderingMode.class),
+    TEXT_RENDERING("text-rendering", Boolean.class),
+    DISABLE_PJL("disable-pjl", Boolean.class);
+
+    private final String name;
+
+    private final Class<?> type;
+
+    private Java2DRendererConfigOption(String name, Class<?> type) {
+        this.name = name;
+        this.type = type;
+    }
+
+    /** {@inheritDoc} */
+    public String getName() {
+        return name;
+    }
+
+    Class<?> getType() {
+        return type;
+    }
+}
index c46bd9e0bd51f4b40515e00932748fc7d7aaef01..d48c00cd3b4706209a8cbd7c6450165a1d3af7d9 100644 (file)
@@ -31,33 +31,17 @@ import org.apache.fop.fonts.DefaultFontConfig;
 import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
 import org.apache.fop.render.RendererConfig;
 
-import static org.apache.fop.render.pcl.PCLRendererConfig.Options.DISABLE_PJL;
-import static org.apache.fop.render.pcl.PCLRendererConfig.Options.RENDERING_MODE;
-import static org.apache.fop.render.pcl.PCLRendererConfig.Options.TEXT_RENDERING;
+import static org.apache.fop.render.pcl.Java2DRendererConfigOption.DISABLE_PJL;
+import static org.apache.fop.render.pcl.Java2DRendererConfigOption.RENDERING_MODE;
+import static org.apache.fop.render.pcl.Java2DRendererConfigOption.TEXT_RENDERING;
 
+/**
+ * The PCL renderer configuration data object.
+ */
 public final class PCLRendererConfig implements RendererConfig {
 
-    public enum Options {
-
-        RENDERING_MODE("rendering", PCLRenderingMode.class),
-        TEXT_RENDERING("text-rendering", Boolean.class),
-        DISABLE_PJL("disable-pjl", Boolean.class);
-
-        private final String name;
-
-        private final Class<?> type;
-
-        private Options(String name, Class<?> type) {
-            this.name = name;
-            this.type = type;
-        }
-
-        public String getName() {
-            return name;
-        }
-    }
-
-    private final Map<Options, Object> params = new EnumMap<Options, Object>(Options.class);
+    private final Map<Java2DRendererConfigOption, Object> params
+        = new EnumMap<Java2DRendererConfigOption, Object>(Java2DRendererConfigOption.class);
 
     private final DefaultFontConfig fontConfig;
 
@@ -81,18 +65,22 @@ public final class PCLRendererConfig implements RendererConfig {
         return getParam(DISABLE_PJL, Boolean.class);
     }
 
-    private <T> T getParam(Options option, Class<T> type) {
-        assert option.type.equals(type);
+    private <T> T getParam(Java2DRendererConfigOption option, Class<T> type) {
+        assert option.getType().equals(type);
         return type.cast(params.get(option));
     }
 
-    private <T> void setParam(Options option, T value) {
-        assert option.type.isInstance(value);
+    private <T> void setParam(Java2DRendererConfigOption option, T value) {
+        assert option.getType().isInstance(value);
         params.put(option, value);
     }
 
+    /**
+     * The PCL renderer configuration data parser.
+     */
     public static final class PCLRendererConfigParser implements RendererConfigParser {
 
+        /** {@inheritDoc} */
         public PCLRendererConfig build(FOUserAgent userAgent, Configuration cfg) throws FOPException {
             PCLRendererConfig config = new PCLRendererConfig(new DefaultFontConfigParser()
                     .parse(cfg, userAgent.validateStrictly()));
@@ -126,6 +114,7 @@ public final class PCLRendererConfig implements RendererConfig {
             }
         }
 
+        /** {@inheritDoc} */
         public String getMimeType() {
             return MimeConstants.MIME_PCL;
         }
index 1fb61a380bf59469191539bfb88627c059906960..0337ef5a7a5ad776f8e3042a0085168cc5dd238c 100644 (file)
@@ -43,32 +43,34 @@ import org.apache.fop.pdf.Version;
 import org.apache.fop.render.RendererConfig;
 import org.apache.fop.util.LogUtil;
 
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.DISABLE_SRGB_COLORSPACE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.ENCRYPTION_LENGTH;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.ENCRYPTION_PARAMS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.FILTER_LIST;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_ACCESSCONTENT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_ANNOTATIONS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_ASSEMBLEDOC;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_COPY_CONTENT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_EDIT_CONTENT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_FILLINFORMS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_PRINT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_PRINTHQ;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.OUTPUT_PROFILE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.OWNER_PASSWORD;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.PDF_A_MODE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.PDF_X_MODE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.USER_PASSWORD;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.VERSION;
-
-
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.DISABLE_SRGB_COLORSPACE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.ENCRYPTION_LENGTH;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.ENCRYPTION_PARAMS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.FILTER_LIST;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_ACCESSCONTENT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_ANNOTATIONS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_ASSEMBLEDOC;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_COPY_CONTENT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_EDIT_CONTENT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_FILLINFORMS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_PRINT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_PRINTHQ;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.OUTPUT_PROFILE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.OWNER_PASSWORD;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.PDF_A_MODE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.PDF_X_MODE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.USER_PASSWORD;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.VERSION;
+
+/**
+ * The PDF renderer configuration data object.
+ */
 public final class PDFRendererConfig implements RendererConfig {
 
     private static final Log LOG = LogFactory.getLog(PDFRendererConfig.class);
 
-    private final Map<PDFRendererConfigOptions, Object> configOptions
-            = new EnumMap<PDFRendererConfigOptions, Object>(PDFRendererConfigOptions.class);
+    private final Map<PDFRendererConfigOption, Object> configOptions
+            = new EnumMap<PDFRendererConfigOption, Object>(PDFRendererConfigOption.class);
 
     private final DefaultFontConfig fontConfig;
 
@@ -109,6 +111,9 @@ public final class PDFRendererConfig implements RendererConfig {
         return pdfVersion == null ? null : Version.getValueOf(pdfVersion);
     }
 
+    /**
+     * The PDF renderer configuration data parser.
+     */
     public static final class PDFRendererConfigParser implements RendererConfigParser {
 
         public PDFRendererConfig build(FOUserAgent userAgent, Configuration cfg) throws FOPException {
@@ -132,7 +137,7 @@ public final class PDFRendererConfig implements RendererConfig {
             }
         }
 
-        private void put(PDFRendererConfigOptions option, Object value) {
+        private void put(PDFRendererConfigOption option, Object value) {
             if (value != null && !value.equals(option.getDefaultValue())) {
                 pdfConfig.configOptions.put(option, value);
             }
@@ -209,12 +214,12 @@ public final class PDFRendererConfig implements RendererConfig {
             put(FILTER_LIST, filterMap);
         }
 
-        private String parseConfig(Configuration cfg, PDFRendererConfigOptions option) {
+        private String parseConfig(Configuration cfg, PDFRendererConfigOption option) {
             Configuration child = cfg.getChild(option.getName());
             return child.getValue(null);
         }
 
-        private boolean doesValueExist(Configuration cfg, PDFRendererConfigOptions option) {
+        private boolean doesValueExist(Configuration cfg, PDFRendererConfigOption option) {
             return cfg.getChild(option.getName(), false) != null;
         }
 
diff --git a/src/java/org/apache/fop/render/pdf/PDFRendererConfigOption.java b/src/java/org/apache/fop/render/pdf/PDFRendererConfigOption.java
new file mode 100644 (file)
index 0000000..faa3302
--- /dev/null
@@ -0,0 +1,112 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.render.pdf;
+
+import org.apache.fop.pdf.PDFAMode;
+import org.apache.fop.pdf.PDFXMode;
+import org.apache.fop.render.RendererConfigOption;
+
+public enum PDFRendererConfigOption implements RendererConfigOption {
+    FILTER_LIST("filterList"),
+    /** Rendering Options key for the PDF/A mode, default: {@link PDFAMode#DISABLED} */
+    PDF_A_MODE("pdf-a-mode", PDFAMode.DISABLED),
+    /** Rendering Options key for the PDF/X mode, default: {@link PDFXMode#DISABLED} */
+    PDF_X_MODE("pdf-x-mode", PDFXMode.DISABLED),
+    /** PDF version entry: specify the version of the PDF document created, datatype: String */
+    VERSION("version"),
+    /**
+     * Rendering Options key for disabling the sRGB color space (only possible if no PDF/A or
+     * PDF/X profile is active), default: false
+     */
+    DISABLE_SRGB_COLORSPACE("disable-srgb-colorspace", false),
+    /** Rendering Options key for the ICC profile for the output intent. */
+    OUTPUT_PROFILE("output-profile"),
+    /** PDF encryption parameter: all parameters as object, datatype: PDFEncryptionParams */
+    ENCRYPTION_PARAMS("encryption-params"),
+    /**
+     * PDF encryption length parameter: must be a multiple of 8 between 40 and 128,
+     * default value 40, datatype: int, default: 40
+     */
+    ENCRYPTION_LENGTH("encryption-length", 40),
+    /**
+     * PDF encryption parameter: Forbids printing to high quality, datatype: Boolean or
+     * "true"/"false", default: false
+     */
+    NO_PRINTHQ("noprinthq", 40),
+    /**
+     * PDF encryption parameter: Forbids assembling document, datatype: Boolean or
+     * "true"/"false", default: false
+     */
+    NO_ASSEMBLEDOC("noassembledoc", false),
+    /**
+     * PDF encryption parameter: Forbids extracting text and graphics, datatype: Boolean
+     * or "true"/"false", default: false
+     */
+    NO_ACCESSCONTENT("noaccesscontent", false),
+    /**
+     * PDF encryption parameter: Forbids filling in existing interactive forms, datatype:
+     * Boolean or "true"/"false", default: false
+     */
+    NO_FILLINFORMS("nofillinforms", false),
+    /**
+     * PDF encryption parameter: Forbids annotations, datatype: Boolean or "true"/"false",
+     * default: false
+     */
+    NO_ANNOTATIONS("noannotations", false),
+    /**
+     * PDF encryption parameter: Forbids printing, datatype: Boolean or "true"/"false",
+     * default: false
+     */
+    NO_PRINT("noprint", false),
+    /**
+     * PDF encryption parameter: Forbids copying content, datatype: Boolean or "true"/"false",
+     * default: false
+     */
+    NO_COPY_CONTENT("nocopy", false),
+    /**
+     * PDF encryption parameter: Forbids editing content, datatype: Boolean or "true"/"false",
+     * default: false
+     */
+    NO_EDIT_CONTENT("noedit", false),
+    /** PDF encryption parameter: user password, datatype: String, default: "" */
+    USER_PASSWORD("user-password", ""),
+    /** PDF encryption parameter: owner password, datatype: String, default: "" */
+    OWNER_PASSWORD("owner-password", "");
+
+    private final String name;
+    private final Object defaultValue;
+
+    private PDFRendererConfigOption(String name, Object defaultValue) {
+        this.name = name;
+        this.defaultValue = defaultValue;
+    }
+
+    private PDFRendererConfigOption(String name) {
+        this(name, null);
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public Object getDefaultValue() {
+        return defaultValue;
+    }
+}
diff --git a/src/java/org/apache/fop/render/pdf/PDFRendererConfigOptions.java b/src/java/org/apache/fop/render/pdf/PDFRendererConfigOptions.java
deleted file mode 100644 (file)
index 81a410c..0000000
+++ /dev/null
@@ -1,112 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.render.pdf;
-
-import org.apache.fop.pdf.PDFAMode;
-import org.apache.fop.pdf.PDFXMode;
-import org.apache.fop.render.RendererConfigOptions;
-
-public enum PDFRendererConfigOptions implements RendererConfigOptions {
-    FILTER_LIST("filterList"),
-    /** Rendering Options key for the PDF/A mode, default: {@link PDFAMode#DISABLED} */
-    PDF_A_MODE("pdf-a-mode", PDFAMode.DISABLED),
-    /** Rendering Options key for the PDF/X mode, default: {@link PDFXMode#DISABLED} */
-    PDF_X_MODE("pdf-x-mode", PDFXMode.DISABLED),
-    /** PDF version entry: specify the version of the PDF document created, datatype: String */
-    VERSION("version"),
-    /**
-     * Rendering Options key for disabling the sRGB color space (only possible if no PDF/A or
-     * PDF/X profile is active), default: false
-     */
-    DISABLE_SRGB_COLORSPACE("disable-srgb-colorspace", false),
-    /** Rendering Options key for the ICC profile for the output intent. */
-    OUTPUT_PROFILE("output-profile"),
-    /** PDF encryption parameter: all parameters as object, datatype: PDFEncryptionParams */
-    ENCRYPTION_PARAMS("encryption-params"),
-    /**
-     * PDF encryption length parameter: must be a multiple of 8 between 40 and 128,
-     * default value 40, datatype: int, default: 40
-     */
-    ENCRYPTION_LENGTH("encryption-length", 40),
-    /**
-     * PDF encryption parameter: Forbids printing to high quality, datatype: Boolean or
-     * "true"/"false", default: false
-     */
-    NO_PRINTHQ("noprinthq", 40),
-    /**
-     * PDF encryption parameter: Forbids assembling document, datatype: Boolean or
-     * "true"/"false", default: false
-     */
-    NO_ASSEMBLEDOC("noassembledoc", false),
-    /**
-     * PDF encryption parameter: Forbids extracting text and graphics, datatype: Boolean
-     * or "true"/"false", default: false
-     */
-    NO_ACCESSCONTENT("noaccesscontent", false),
-    /**
-     * PDF encryption parameter: Forbids filling in existing interactive forms, datatype:
-     * Boolean or "true"/"false", default: false
-     */
-    NO_FILLINFORMS("nofillinforms", false),
-    /**
-     * PDF encryption parameter: Forbids annotations, datatype: Boolean or "true"/"false",
-     * default: false
-     */
-    NO_ANNOTATIONS("noannotations", false),
-    /**
-     * PDF encryption parameter: Forbids printing, datatype: Boolean or "true"/"false",
-     * default: false
-     */
-    NO_PRINT("noprint", false),
-    /**
-     * PDF encryption parameter: Forbids copying content, datatype: Boolean or "true"/"false",
-     * default: false
-     */
-    NO_COPY_CONTENT("nocopy", false),
-    /**
-     * PDF encryption parameter: Forbids editing content, datatype: Boolean or "true"/"false",
-     * default: false
-     */
-    NO_EDIT_CONTENT("noedit", false),
-    /** PDF encryption parameter: user password, datatype: String, default: "" */
-    USER_PASSWORD("user-password", ""),
-    /** PDF encryption parameter: owner password, datatype: String, default: "" */
-    OWNER_PASSWORD("owner-password", "");
-
-    private final String name;
-    private final Object defaultValue;
-
-    private PDFRendererConfigOptions(String name, Object defaultValue) {
-        this.name = name;
-        this.defaultValue = defaultValue;
-    }
-
-    private PDFRendererConfigOptions(String name) {
-        this(name, null);
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public Object getDefaultValue() {
-        return defaultValue;
-    }
-}
index a4892dd7050da1900f47dd7e5eeae8e633c144b4..9cd89fa01f8285f343652311d5977f1edcfaa7f2 100644 (file)
@@ -22,14 +22,14 @@ package org.apache.fop.render.pdf;
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.pdf.PDFEncryptionParams;
+import org.apache.fop.render.DefaultRendererConfigurator;
 import org.apache.fop.render.RendererConfig.RendererConfigParser;
-import org.apache.fop.render.adobe.AdobeRendererConfigurator;
 import org.apache.fop.render.intermediate.IFDocumentHandler;
 
 /**
  * PDF renderer configurator.
  */
-public class PDFRendererConfigurator extends AdobeRendererConfigurator {
+public class PDFRendererConfigurator extends DefaultRendererConfigurator {
 
     /**
      * Default constructor
index c7252baf1738d9633e62dbf14389ab1d7705161f..daffe2d3f608fe54635c7e52ad759f0c5cb103f9 100644 (file)
@@ -68,21 +68,21 @@ import org.apache.fop.pdf.Version;
 import org.apache.fop.pdf.VersionController;
 import org.apache.fop.render.pdf.extensions.PDFEmbeddedFileExtensionAttachment;
 
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.DISABLE_SRGB_COLORSPACE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.ENCRYPTION_PARAMS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_ACCESSCONTENT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_ANNOTATIONS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_ASSEMBLEDOC;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_COPY_CONTENT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_EDIT_CONTENT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_FILLINFORMS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_PRINT;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.NO_PRINTHQ;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.OUTPUT_PROFILE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.OWNER_PASSWORD;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.PDF_A_MODE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.PDF_X_MODE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.USER_PASSWORD;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.DISABLE_SRGB_COLORSPACE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.ENCRYPTION_PARAMS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_ACCESSCONTENT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_ANNOTATIONS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_ASSEMBLEDOC;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_COPY_CONTENT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_EDIT_CONTENT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_FILLINFORMS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_PRINT;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.NO_PRINTHQ;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.OUTPUT_PROFILE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.OWNER_PASSWORD;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.PDF_A_MODE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.PDF_X_MODE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.USER_PASSWORD;
 
 /**
  * Utility class which enables all sorts of features that are not directly connected to the
@@ -99,10 +99,10 @@ class PDFRenderingUtil {
     private PDFDocument pdfDoc;
 
     /** the PDF/A mode (Default: disabled) */
-    private PDFAMode pdfAMode = (PDFAMode) PDFRendererConfigOptions.PDF_A_MODE.getDefaultValue();
+    private PDFAMode pdfAMode = (PDFAMode) PDFRendererConfigOption.PDF_A_MODE.getDefaultValue();
 
     /** the PDF/X mode (Default: disabled) */
-    private PDFXMode pdfXMode = (PDFXMode) PDFRendererConfigOptions.PDF_X_MODE.getDefaultValue();
+    private PDFXMode pdfXMode = (PDFXMode) PDFRendererConfigOption.PDF_X_MODE.getDefaultValue();
 
     /** the (optional) encryption parameters */
     private PDFEncryptionParams encryptionParams;
index f023b46477f89bf371f07a26b38644cba9ee8f2e..0d043c10d418985b2f8a9990468fc8dda875ba93 100644 (file)
@@ -35,17 +35,20 @@ import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
 import org.apache.fop.render.RendererConfig;
 import org.apache.fop.util.LogUtil;
 
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.AUTO_ROTATE_LANDSCAPE;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.DSC_COMPLIANT;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.LANGUAGE_LEVEL;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.OPTIMIZE_RESOURCES;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.RENDERING_MODE;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.SAFE_SET_PAGE_DEVICE;
-
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.AUTO_ROTATE_LANDSCAPE;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.DSC_COMPLIANT;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.LANGUAGE_LEVEL;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.OPTIMIZE_RESOURCES;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.RENDERING_MODE;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.SAFE_SET_PAGE_DEVICE;
+
+/**
+ * The PostScript renderer configuration data object.
+ */
 public final class PSRendererConfig implements RendererConfig {
 
-    private final EnumMap<PSRendererConfigurationOptions, Object> params
-            = new EnumMap<PSRendererConfigurationOptions, Object>(PSRendererConfigurationOptions.class);
+    private final EnumMap<PSRendererConfigurationOption, Object> params
+            = new EnumMap<PSRendererConfigurationOption, Object>(PSRendererConfigurationOption.class);
 
     private final DefaultFontConfig fontConfig;
 
@@ -81,6 +84,9 @@ public final class PSRendererConfig implements RendererConfig {
         return (PSRenderingMode) params.get(RENDERING_MODE);
     }
 
+    /**
+     * The PostScript renderer configuration data parser.
+     */
     public static final class PSRendererConfigParser implements RendererConfigParser {
 
         private static final Log LOG = LogFactory.getLog(PSRendererConfigParser.class);
@@ -126,12 +132,12 @@ public final class PSRendererConfig implements RendererConfig {
             }
         }
 
-        private void setConfigParameter(PSRendererConfigurationOptions option,
+        private void setConfigParameter(PSRendererConfigurationOption option,
                 Object value) {
             config.params.put(option, value != null ? value : option.getDefaultValue());
         }
 
-        private void setBoolConfigParam(Configuration cfg, PSRendererConfigurationOptions option) {
+        private void setBoolConfigParam(Configuration cfg, PSRendererConfigurationOption option) {
             setConfigParameter(option, cfg.getChild(
                     option.getName()).getValueAsBoolean((Boolean) option.getDefaultValue()));
         }
diff --git a/src/java/org/apache/fop/render/ps/PSRendererConfigurationOption.java b/src/java/org/apache/fop/render/ps/PSRendererConfigurationOption.java
new file mode 100644 (file)
index 0000000..7358517
--- /dev/null
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.render.ps;
+
+import org.apache.xmlgraphics.ps.PSGenerator;
+
+import org.apache.fop.render.RendererConfigOption;
+
+/**
+ * An enumeration of the PostScript renderer configuration options along with their default values.
+ */
+public enum PSRendererConfigurationOption implements RendererConfigOption {
+    /** Indicates whether landscape pages should be rotated, default: false */
+    AUTO_ROTATE_LANDSCAPE("auto-rotate-landscape", false),
+    /** Sets the PostScript language leven, default: {@see PSGenerator#DEFAULT_LANGUAGE_LEVEL}*/
+    LANGUAGE_LEVEL("language-level", PSGenerator.DEFAULT_LANGUAGE_LEVEL),
+    /** Whether resources should be optimized in a post-processing run, default: false */
+    OPTIMIZE_RESOURCES("optimize-resources", false),
+    /** Indicates whether the "safe setpagedevice" mode is active, default: false */
+    SAFE_SET_PAGE_DEVICE("safe-set-page-device", false),
+    /** Indicates whether the PostScript output should be DSC compliant, default: true*/
+    DSC_COMPLIANT("dsc-compliant", true),
+    RENDERING_MODE("rendering", PSRenderingMode.QUALITY);
+
+    private final String name;
+    private final Object defaultValue;
+
+    private PSRendererConfigurationOption(String name, Object defaultValue) {
+        this.name = name;
+        this.defaultValue = defaultValue;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public Object getDefaultValue() {
+        return defaultValue;
+    }
+}
diff --git a/src/java/org/apache/fop/render/ps/PSRendererConfigurationOptions.java b/src/java/org/apache/fop/render/ps/PSRendererConfigurationOptions.java
deleted file mode 100644 (file)
index ffecbf6..0000000
+++ /dev/null
@@ -1,57 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* $Id$ */
-
-package org.apache.fop.render.ps;
-
-import org.apache.xmlgraphics.ps.PSGenerator;
-
-import org.apache.fop.render.RendererConfigOptions;
-
-/**
- * An enumeration of the PostScript renderer configuration options along with their default values.
- */
-public enum PSRendererConfigurationOptions implements RendererConfigOptions {
-    /** Indicates whether landscape pages should be rotated, default: false */
-    AUTO_ROTATE_LANDSCAPE("auto-rotate-landscape", false),
-    /** Sets the PostScript language leven, default: {@see PSGenerator#DEFAULT_LANGUAGE_LEVEL}*/
-    LANGUAGE_LEVEL("language-level", PSGenerator.DEFAULT_LANGUAGE_LEVEL),
-    /** Whether resources should be optimized in a post-processing run, default: false */
-    OPTIMIZE_RESOURCES("optimize-resources", false),
-    /** Indicates whether the "safe setpagedevice" mode is active, default: false */
-    SAFE_SET_PAGE_DEVICE("safe-set-page-device", false),
-    /** Indicates whether the PostScript output should be DSC compliant, default: true*/
-    DSC_COMPLIANT("dsc-compliant", true),
-    RENDERING_MODE("rendering", PSRenderingMode.QUALITY);
-
-    private final String name;
-    private final Object defaultValue;
-
-    private PSRendererConfigurationOptions(String name, Object defaultValue) {
-        this.name = name;
-        this.defaultValue = defaultValue;
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public Object getDefaultValue() {
-        return defaultValue;
-    }
-}
index cec3a654766a1ba24d00870ada1d7cd51e478e49..b453de413af0dca05d309eb61398e1d956061c55 100644 (file)
@@ -21,8 +21,8 @@ package org.apache.fop.render.ps;
 
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
+import org.apache.fop.render.DefaultRendererConfigurator;
 import org.apache.fop.render.RendererConfig.RendererConfigParser;
-import org.apache.fop.render.adobe.AdobeRendererConfigurator;
 import org.apache.fop.render.intermediate.IFDocumentHandler;
 import org.apache.fop.render.intermediate.IFDocumentHandlerConfigurator;
 
@@ -30,7 +30,7 @@ import org.apache.fop.render.intermediate.IFDocumentHandlerConfigurator;
 /**
  * Postscript renderer config
  */
-public class PSRendererConfigurator extends AdobeRendererConfigurator
+public class PSRendererConfigurator extends DefaultRendererConfigurator
         implements IFDocumentHandlerConfigurator {
 
     /**
index 9d40c378c5939a97fb518860f49e7b994f8ee459..1322d5e4b9c18838e22501bfb3dae30ca4304388 100644 (file)
@@ -33,9 +33,9 @@ import org.apache.fop.render.ps.extensions.PSCommentBefore;
 import org.apache.fop.render.ps.extensions.PSExtensionAttachment;
 import org.apache.fop.render.ps.extensions.PSSetupCode;
 
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.AUTO_ROTATE_LANDSCAPE;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.LANGUAGE_LEVEL;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.OPTIMIZE_RESOURCES;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.AUTO_ROTATE_LANDSCAPE;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.LANGUAGE_LEVEL;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.OPTIMIZE_RESOURCES;
 
 /**
  * Utility class which enables all sorts of features that are not directly connected to the
index 8532873ddb8496f6457be645b877a45f3c66533d..a600047ae1a88dd4179fc9db0efd7ee69944e25d 100644 (file)
@@ -21,14 +21,14 @@ package org.apache.fop.render.txt;
 
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
+import org.apache.fop.render.DefaultRendererConfigurator;
 import org.apache.fop.render.Renderer;
 import org.apache.fop.render.RendererConfig.RendererConfigParser;
-import org.apache.fop.render.adobe.AdobeRendererConfigurator;
 
 /**
  * TXT Renderer configurator
  */
-public class TXTRendererConfigurator extends AdobeRendererConfigurator {
+public class TXTRendererConfigurator extends DefaultRendererConfigurator {
 
     /**
      * Default constructor
index c0c819206a39f91b9f7d1aeeef7674ec478c68bc..7b00456642a823d25934f14aa63fdadea5c3053a 100644 (file)
@@ -29,17 +29,20 @@ import org.apache.fop.apps.MimeConstants;
 import org.apache.fop.fonts.DefaultFontConfig;
 import org.apache.fop.fonts.DefaultFontConfig.DefaultFontConfigParser;
 import org.apache.fop.render.RendererConfig;
-import org.apache.fop.render.RendererConfigOptions;
+import org.apache.fop.render.RendererConfigOption;
 
-public class TxtRendererConfig implements RendererConfig {
+/**
+ * The Text renderer configuration data object.
+ */
+public final class TxtRendererConfig implements RendererConfig {
 
-    public enum TxtRendererConfigOptions implements RendererConfigOptions {
+    public enum TxtRendererConfigOption implements RendererConfigOption {
         ENCODING("encoding", "UTF-8");
 
         private final String name;
         private final Object defaultValue;
 
-        private TxtRendererConfigOptions(String name, Object defaultValue) {
+        private TxtRendererConfigOption(String name, Object defaultValue) {
             this.name = name;
             this.defaultValue = defaultValue;
         }
@@ -53,8 +56,8 @@ public class TxtRendererConfig implements RendererConfig {
         }
     }
 
-    private final EnumMap<TxtRendererConfigOptions, Object> params
-            = new EnumMap<TxtRendererConfigOptions, Object>(TxtRendererConfigOptions.class);
+    private final EnumMap<TxtRendererConfigOption, Object> params
+            = new EnumMap<TxtRendererConfigOption, Object>(TxtRendererConfigOption.class);
 
     private final DefaultFontConfig fontConfig;
 
@@ -67,22 +70,27 @@ public class TxtRendererConfig implements RendererConfig {
     }
 
     public String getEncoding() {
-        return (String) params.get(TxtRendererConfigOptions.ENCODING);
+        return (String) params.get(TxtRendererConfigOption.ENCODING);
     }
 
+    /**
+     * The Text renderer configuration data parser.
+     */
     public static final class TxtRendererConfigParser implements RendererConfigParser {
 
+        /** {@inheritDoc} */
         public TxtRendererConfig build(FOUserAgent userAgent, Configuration cfg) throws FOPException {
             TxtRendererConfig config = new TxtRendererConfig(new DefaultFontConfigParser().parse(cfg,
                     userAgent.validateStrictly()));
             if (cfg != null) {
-                TxtRendererConfigOptions option = TxtRendererConfigOptions.ENCODING;
+                TxtRendererConfigOption option = TxtRendererConfigOption.ENCODING;
                 String value = cfg.getChild(option.getName(), true).getValue(null);
                 config.params.put(option, value != null ? value : option.getDefaultValue());
             }
             return config;
         }
 
+        /** {@inheritDoc} */
         public String getMimeType() {
             return MimeConstants.MIME_PLAIN_TEXT;
         }
index d02694cda859d0789c173e2a4eedda53d44f559f..2199df4f9f3704b3421f899961a8b1174bc22b7e 100644 (file)
@@ -25,8 +25,8 @@ import org.apache.avalon.framework.configuration.Configuration;
 import org.apache.avalon.framework.configuration.ConfigurationException;
 
 import org.apache.fop.apps.FOPException;
-import org.apache.fop.apps.io.ResourceResolverFactory;
 import org.apache.fop.apps.io.InternalResourceResolver;
+import org.apache.fop.apps.io.ResourceResolverFactory;
 import org.apache.fop.fonts.DefaultFontConfig;
 import org.apache.fop.fonts.DefaultFontConfigurator;
 import org.apache.fop.fonts.EmbedFontInfo;
index 4ce9cb2a0d1b5e8e798463564ed32dedaaea0511..5e7204f9e555080da3cadb3b250f3918999af0da 100644 (file)
 
 package org.apache.fop.apps;
 
-
-import java.net.URI;
 import java.util.Map;
 
 import org.w3c.dom.Element;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
 import org.apache.fop.render.afp.AFPRendererConfig;
-import org.apache.fop.render.afp.AFPRendererConfig.Options;
+import org.apache.fop.render.afp.AFPRendererConfigOption;
 import org.apache.fop.render.afp.AFPShadingMode;
 
 import static org.apache.fop.render.afp.AFPRendererConfig.ImagesModeOptions.MODE_GRAYSCALE;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.DEFAULT_RESOURCE_LEVELS;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_DITHERING_QUALITY;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_FS45;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_JPEG;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_MAPPING_OPTION;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_MODE;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_NATIVE;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.IMAGES_WRAP_PSEG;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.JPEG_ALLOW_JPEG_EMBEDDING;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.JPEG_BITMAP_ENCODING_QUALITY;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.RENDERER_RESOLUTION;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.LINE_WIDTH_CORRECTION;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.RESOURCE_GROUP_URI;
-import static org.apache.fop.render.afp.AFPRendererConfig.Options.SHADING;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.DEFAULT_RESOURCE_LEVELS;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_DITHERING_QUALITY;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_FS45;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_JPEG;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_MAPPING_OPTION;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_MODE;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_NATIVE;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.IMAGES_WRAP_PSEG;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.JPEG_ALLOW_JPEG_EMBEDDING;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.JPEG_BITMAP_ENCODING_QUALITY;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.LINE_WIDTH_CORRECTION;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.RENDERER_RESOLUTION;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.RESOURCE_GROUP_URI;
+import static org.apache.fop.render.afp.AFPRendererConfigOption.SHADING;
 
 /**
  * A config builder specific to a particular renderer for specific MIME type.
@@ -58,7 +56,7 @@ public final class AFPRendererConfBuilder extends RendererConfBuilder {
         super(MimeConstants.MIME_AFP);
     }
 
-    private AFPRendererConfBuilder createTextElement(Options option, String value) {
+    private AFPRendererConfBuilder createTextElement(AFPRendererConfigOption option, String value) {
         createTextElement(option.getName(), value);
         return this;
     }
@@ -155,7 +153,7 @@ public final class AFPRendererConfBuilder extends RendererConfBuilder {
             return AFPRendererConfBuilder.this.endImages();
         }
 
-        private ImagesBuilder setAttribute(Options options, Object value) {
+        private ImagesBuilder setAttribute(AFPRendererConfigOption options, Object value) {
             return setAttribute(options.getName(), value);
         }
 
index 2a75aad6ca230f6d3c7210d2701b0dfe15c24838..b0e468be01e1de85f7a16e7513e3dd151228df83 100644 (file)
@@ -21,11 +21,11 @@ package org.apache.fop.apps;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
 
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.ANTI_ALIASING;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.BACKGROUND_COLOR;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_QUALITY_ELEMENT;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.ANTI_ALIASING;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.BACKGROUND_COLOR;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_QUALITY_ELEMENT;
 
 public class BitmapRendererConfBuilder extends RendererConfBuilder {
 
diff --git a/test/java/org/apache/fop/apps/EnvironmentalProfileFactoryTestCase.java b/test/java/org/apache/fop/apps/EnvironmentalProfileFactoryTestCase.java
new file mode 100644 (file)
index 0000000..7b03396
--- /dev/null
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.fop.apps;
+
+import java.net.URI;
+
+import org.junit.Test;
+
+import org.apache.fop.apps.io.ResourceResolver;
+
+import static org.junit.Assert.assertEquals;
+import static org.mockito.Mockito.mock;
+
+public class EnvironmentalProfileFactoryTestCase {
+
+    private final URI testURI = URI.create("this.is.purely.for.test.purposes");
+
+    @Test
+    public void testCreateDefault() {
+        ResourceResolver mockedResolver = mock(ResourceResolver.class);
+        EnvironmentProfile sut = EnvironmentalProfileFactory.createDefault(testURI, mockedResolver);
+        assertEquals(mockedResolver, sut.getResourceResolver());
+    }
+}
index 9c2c32a2f601f0a6291cbf924a4a6dc88845779c..1efb2d69834e2f880c94b6f8ad48c0d6b1e6e468 100644 (file)
@@ -42,7 +42,7 @@ import javax.xml.transform.stream.StreamResult;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 
-import org.apache.fop.render.RendererConfigOptions;
+import org.apache.fop.render.RendererConfigOption;
 
 /**
  * A builder class for creating fop.xconf XML DOMs for test purposes. You can set all the necessary
@@ -336,7 +336,7 @@ public class FopConfBuilder implements FontConfigurator<FopConfBuilder> {
             return createTextElement(name, value, rendererEl);
         }
 
-        protected final Element createTextElement(RendererConfigOptions option, String value) {
+        protected final Element createTextElement(RendererConfigOption option, String value) {
             return createTextElement(option.getName(), value, rendererEl);
         }
 
index 59a6fae219a77fb490d863da0e826e4b78583ae5..00568cfc5a4afb5e7397acbf96d5ea72c3145bc7 100644 (file)
@@ -21,7 +21,7 @@ package org.apache.fop.apps;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
 
-import static org.apache.fop.render.java2d.Java2DRendererOptions.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
+import static org.apache.fop.render.java2d.Java2DRendererOption.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
 
 public class Java2DRendererConfBuilder extends RendererConfBuilder {
 
index ca57cbfc7cb754f8da07a64cff79b31d2b444064..bf16c59cae3a9a4100f449f70d2195be1b191ad8 100644 (file)
@@ -22,19 +22,19 @@ package org.apache.fop.apps;
 import org.w3c.dom.Element;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
-import org.apache.fop.render.RendererConfigOptions;
-import org.apache.fop.render.pdf.PDFRendererConfigOptions;
-
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.DISABLE_SRGB_COLORSPACE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.ENCRYPTION_LENGTH;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.ENCRYPTION_PARAMS;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.FILTER_LIST;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.OUTPUT_PROFILE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.OWNER_PASSWORD;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.PDF_A_MODE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.PDF_X_MODE;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.USER_PASSWORD;
-import static org.apache.fop.render.pdf.PDFRendererConfigOptions.VERSION;
+import org.apache.fop.render.RendererConfigOption;
+import org.apache.fop.render.pdf.PDFRendererConfigOption;
+
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.DISABLE_SRGB_COLORSPACE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.ENCRYPTION_LENGTH;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.ENCRYPTION_PARAMS;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.FILTER_LIST;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.OUTPUT_PROFILE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.OWNER_PASSWORD;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.PDF_A_MODE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.PDF_X_MODE;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.USER_PASSWORD;
+import static org.apache.fop.render.pdf.PDFRendererConfigOption.VERSION;
 
 /**
  * A config builder specific to a particular renderer for specific MIME type.
@@ -115,7 +115,7 @@ public final class PDFRendererConfBuilder extends RendererConfBuilder {
             return this;
         }
 
-        public EncryptionParamsBuilder setAllowParam(PDFRendererConfigOptions option) {
+        public EncryptionParamsBuilder setAllowParam(PDFRendererConfigOption option) {
             el.appendChild(createElement(option.getName()));
             return this;
         }
@@ -124,7 +124,7 @@ public final class PDFRendererConfBuilder extends RendererConfBuilder {
             return PDFRendererConfBuilder.this.endEncryptionParams();
         }
 
-        private void createTextElement(RendererConfigOptions name, String value) {
+        private void createTextElement(RendererConfigOption name, String value) {
             PDFRendererConfBuilder.this.createTextElement(name.getName(), value, el);
         }
 
index 6b708efc0e13083df4325390f85cbdcb42b4e004..2fec155ffa4dfe909734c938069f5fdc57501d64 100644 (file)
@@ -21,11 +21,11 @@ package org.apache.fop.apps;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
 
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.AUTO_ROTATE_LANDSCAPE;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.DSC_COMPLIANT;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.LANGUAGE_LEVEL;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.OPTIMIZE_RESOURCES;
-import static org.apache.fop.render.ps.PSRendererConfigurationOptions.SAFE_SET_PAGE_DEVICE;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.AUTO_ROTATE_LANDSCAPE;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.DSC_COMPLIANT;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.LANGUAGE_LEVEL;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.OPTIMIZE_RESOURCES;
+import static org.apache.fop.render.ps.PSRendererConfigurationOption.SAFE_SET_PAGE_DEVICE;
 
 /**
  * A fop conf builder specific to a particular renderer for Postscript.
index a7a1a76900f73ca097a900d76cf2f5f571644cd9..0ae06e62f624d3bc74a90f166d58ee5dfac46bf9 100644 (file)
@@ -19,7 +19,7 @@
 
 package org.apache.fop.apps;
 
-import static org.apache.fop.render.bitmap.TIFFRendererConfig.TIFFRendererConfigOptions.COMPRESSION;
+import static org.apache.fop.render.bitmap.TIFFRendererConfig.TIFFRendererConfigOption.COMPRESSION;
 
 public class TIFFRendererConfBuilder extends BitmapRendererConfBuilder {
 
index 5e9f6af42e44ff979bea72ad6af4c252cde23468..3e022762a124ad8d265e7e23cf7e70eab4031436 100644 (file)
@@ -21,7 +21,7 @@ package org.apache.fop.apps;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
 
-import static org.apache.fop.render.txt.TxtRendererConfig.TxtRendererConfigOptions.ENCODING;
+import static org.apache.fop.render.txt.TxtRendererConfig.TxtRendererConfigOption.ENCODING;
 
 public class TxtRendererConfBuilder extends RendererConfBuilder {
 
index 5a885a143261879718a45f29cf345d93f80e0397..312ee43c5a60bc13443794639d8194796d3df550 100644 (file)
@@ -28,14 +28,14 @@ import org.apache.fop.apps.AbstractRendererConfigParserTester;
 import org.apache.fop.apps.BitmapRendererConfBuilder;
 import org.apache.fop.render.bitmap.BitmapRendererConfig.BitmapRendererConfigParser;
 
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_BILEVEL;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_BINARY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_GRAY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_RGB;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_RGBA;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_QUALITY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_SPEED;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_BILEVEL;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_BINARY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_GRAY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_RGB;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_RGBA;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.JAVA2D_TRANSPARENT_PAGE_BACKGROUND;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_QUALITY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_SPEED;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNull;
index 77e8f12470c77e4951e69bb1d6029bc1743b4d73..2e2bccc36f3b85c0f8a123024974429f070ec7c0 100644 (file)
@@ -28,13 +28,13 @@ import org.apache.fop.apps.BitmapRendererConfBuilder;
 import org.apache.fop.render.intermediate.IFDocumentHandler;
 import org.apache.fop.util.ColorUtil;
 
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_BILEVEL;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_BINARY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_GRAY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_RGB;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.COLOR_MODE_RGBA;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_QUALITY;
-import static org.apache.fop.render.bitmap.BitmapRendererConfigOptions.RENDERING_SPEED;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_BILEVEL;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_BINARY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_GRAY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_RGB;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.COLOR_MODE_RGBA;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_QUALITY;
+import static org.apache.fop.render.bitmap.BitmapRendererConfigOption.RENDERING_SPEED;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
index 82dbfa5907bbc889a38bb5bf5ea241ca634a6171..3c44bfdf74fcc02b06a763b6afd800a3463a10b9 100644 (file)
 
 package org.apache.fop.render.bitmap;
 
+import org.apache.fop.apps.MimeConstants;
 import org.apache.fop.render.bitmap.BitmapRendererConfig.BitmapRendererConfigParser;
 
 public class BitmapRendererConfigParserTestCase extends AbstractBitmapRendererConfigParserTester {
 
     public BitmapRendererConfigParserTestCase() {
-        super(new BitmapRendererConfigParser("bitmap"));
+        super(new BitmapRendererConfigParser(MimeConstants.MIME_BITMAP));
     }
 }
index bba656e78d90d35c2c4df434c3de765c3687528a..b0f1f46b7d6dd719d6cbef9a64a8e02fae6b6f7c 100644 (file)
@@ -22,11 +22,10 @@ package org.apache.fop.render.pcl;
 
 import org.apache.fop.apps.FopConfBuilder.RendererConfBuilder;
 import org.apache.fop.apps.MimeConstants;
-import org.apache.fop.render.pcl.PCLRendererConfig.Options;
 
-import static org.apache.fop.render.pcl.PCLRendererConfig.Options.DISABLE_PJL;
-import static org.apache.fop.render.pcl.PCLRendererConfig.Options.RENDERING_MODE;
-import static org.apache.fop.render.pcl.PCLRendererConfig.Options.TEXT_RENDERING;
+import static org.apache.fop.render.pcl.Java2DRendererConfigOption.DISABLE_PJL;
+import static org.apache.fop.render.pcl.Java2DRendererConfigOption.RENDERING_MODE;
+import static org.apache.fop.render.pcl.Java2DRendererConfigOption.TEXT_RENDERING;
 /**
  * A config builder specific to a particular renderer for specific MIME type.
  */
@@ -52,7 +51,7 @@ public final class PCLRendererConfBuilder extends RendererConfBuilder {
         return createTextElement(DISABLE_PJL, String.valueOf(value));
     }
 
-    private PCLRendererConfBuilder createTextElement(Options option, String value) {
+    private PCLRendererConfBuilder createTextElement(Java2DRendererConfigOption option, String value) {
         createTextElement(option.getName(), value);
         return this;
     }
index f3071af57846055ded4d26d745ef8985cca3f642..366525ea9d42045c1de3a61818f641b29d8e2b67 100644 (file)
@@ -51,7 +51,7 @@ public class PDFRendererConfigParserTestCase
         assertEquals(testPassword, conf.getEncryptionParameters().getUserPassword());
     }
 
-    private void testRestrictEncryptionParameter(PDFRendererConfigOptions option)
+    private void testRestrictEncryptionParameter(PDFRendererConfigOption option)
             throws Exception {
         parseConfig(createRenderer().startEncryptionParams()
                                 .setAllowParam(option)
@@ -62,7 +62,7 @@ public class PDFRendererConfigParserTestCase
         assertTrue(testEncryptionParameter(option));
     }
 
-    public boolean testEncryptionParameter(PDFRendererConfigOptions option) throws Exception {
+    public boolean testEncryptionParameter(PDFRendererConfigOption option) throws Exception {
         switch (option) {
         case NO_PRINT:
             return conf.getEncryptionParameters().isAllowPrint();
@@ -88,14 +88,14 @@ public class PDFRendererConfigParserTestCase
 
     @Test
     public void testAllEncryptionRestrictions() throws Exception {
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_PRINT);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_ACCESSCONTENT);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_ANNOTATIONS);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_ASSEMBLEDOC);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_COPY_CONTENT);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_EDIT_CONTENT);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_FILLINFORMS);
-        testRestrictEncryptionParameter(PDFRendererConfigOptions.NO_PRINTHQ);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_PRINT);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_ACCESSCONTENT);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_ANNOTATIONS);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_ASSEMBLEDOC);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_COPY_CONTENT);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_EDIT_CONTENT);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_FILLINFORMS);
+        testRestrictEncryptionParameter(PDFRendererConfigOption.NO_PRINTHQ);
     }
 
     @Test
index 67bf5b0c874b2bdb4c56b1a14ec92928d12b3aa7..1b83707874a1ca556a5e225b6346d1f6f411e84d 100644 (file)
@@ -24,7 +24,7 @@ import org.junit.Test;
 import org.apache.fop.apps.AbstractRendererConfigParserTester;
 import org.apache.fop.apps.TxtRendererConfBuilder;
 import org.apache.fop.render.RendererConfig.RendererConfigParser;
-import org.apache.fop.render.txt.TxtRendererConfig.TxtRendererConfigOptions;
+import org.apache.fop.render.txt.TxtRendererConfig.TxtRendererConfigOption;
 import org.apache.fop.render.txt.TxtRendererConfig.TxtRendererConfigParser;
 
 import static org.junit.Assert.assertEquals;
@@ -47,7 +47,7 @@ public class TxtRendererConfigParserTestCase
 
         // Check the default behaviour is expected
         parseConfig(createRenderer());
-        assertEquals(TxtRendererConfigOptions.ENCODING.getDefaultValue(), conf.getEncoding());
+        assertEquals(TxtRendererConfigOption.ENCODING.getDefaultValue(), conf.getEncoding());
     }
 
 }