Author: jeremias
Date: Mon Mar 17 04:01:41 2008
New Revision: 637833

URL: http://svn.apache.org/viewvc?rev=637833&view=rev
Log:
Exception while cloning for RetrieveMarker to be handled by user as suggested 
by Andreas.

Modified:
    
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/events/EventFormatter.xml
    
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/FOValidationEventProducer.java
    
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/flow/RetrieveMarker.java

Modified: 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/events/EventFormatter.xml
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/events/EventFormatter.xml?rev=637833&r1=637832&r2=637833&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/events/EventFormatter.xml
 (original)
+++ 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/events/EventFormatter.xml
 Mon Mar 17 04:01:41 2008
@@ -50,4 +50,5 @@
   <message key="org.apache.fop.events.ResourceEventProducer.imageError">Image 
not available. URI: {uri}. Reason: {e}{{locator}}</message>
   <message key="org.apache.fop.events.ResourceEventProducer.imageIOError">I/O 
error while loading image.  URI: {uri}. Reason: {ioe}{{locator}}</message>
   <message 
key="org.apache.fop.events.ResourceEventProducer.ifoNoIntrinsiceSize">The 
intrinsic dimensions of an instream-foreign-object could not be 
determined.{{locator}}</message>
+  <message 
key="org.apache.fop.fo.FOValidationEventProducer.markerCloningFailed">Unable to 
clone subtree of fo:marker (marker-class-name="{markerClassName}") for 
fo:retrieve-marker.{{locator}}</message>
 </catalogue>

Modified: 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/FOValidationEventProducer.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/FOValidationEventProducer.java?rev=637833&r1=637832&r2=637833&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/FOValidationEventProducer.java
 (original)
+++ 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/FOValidationEventProducer.java
 Mon Mar 17 04:01:41 2008
@@ -23,6 +23,7 @@
 
 import org.apache.xmlgraphics.util.QName;
 
+import org.apache.fop.apps.FOPException;
 import org.apache.fop.events.EventBroadcaster;
 import org.apache.fop.events.EventProducer;
 import org.apache.fop.fo.expr.PropertyException;
@@ -194,5 +195,15 @@
      */
     void missingLinkDestination(Object source, String elementName, Locator loc)
                 throws ValidationException;
+
+    /**
+     * Indicates a problem while cloning a marker (ex. due to invalid property 
values).
+     * @param source the event source
+     * @param markerClassName the "marker-class-name" of the marker
+     * @param fe the FOP exception that cause this problem
+     * @param loc the location of the error or null
+     * @event.severity ERROR
+     */
+    void markerCloningFailed(Object source, String markerClassName, 
FOPException fe, Locator loc);
 
 }

Modified: 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/flow/RetrieveMarker.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/flow/RetrieveMarker.java?rev=637833&r1=637832&r2=637833&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/flow/RetrieveMarker.java
 (original)
+++ 
xmlgraphics/fop/branches/Temp_ProcessingFeedback/src/java/org/apache/fop/fo/flow/RetrieveMarker.java
 Mon Mar 17 04:01:41 2008
@@ -49,16 +49,14 @@
 
     /**
      * Create a retrieve marker object.
-     *
+     * @param parent FONode that is the parent of this object
      * @see org.apache.fop.fo.FONode#FONode(FONode)
      */
     public RetrieveMarker(FONode parent) {
         super(parent);
     }
 
-    /**
-     * [EMAIL PROTECTED]
-     */
+    /** [EMAIL PROTECTED] */
     public void bind(PropertyList pList) throws FOPException {
         if (findAncestor(FO_STATIC_CONTENT) < 0) {
             invalidChildError(locator, getParent().getName(), FO_URI, 
getName(), 
@@ -208,13 +206,12 @@
             try {
                 cloneFromMarker(marker);
             } catch (FOPException exc) {
-                log.error("fo:retrieve-marker unable to clone "
-                        + "subtree of fo:marker (marker-class-name="
-                        + marker.getMarkerClassName() + ")", exc);
+                getFOValidationEventProducer().markerCloningFailed(this,
+                        marker.getMarkerClassName(), exc, getLocator());
                 return;
             }
-        } else if (log.isInfoEnabled()) {
-            log.info("Empty marker retrieved...");
+        } else if (log.isDebugEnabled()) {
+            log.debug("Empty marker retrieved...");
         }
         return;
     }
@@ -224,9 +221,7 @@
         return "retrieve-marker";
     }
 
-    /**
-     * [EMAIL PROTECTED]
-     */
+    /** [EMAIL PROTECTED] */
     public int getNameId() {
         return FO_RETRIEVE_MARKER;
     }    



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to