/*
 * Copyright 2005 The Apache Software Foundation.
 * 
 * Licensed 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.xerces.util;

import org.apache.xerces.xni.parser.XMLErrorHandler;
import org.xml.sax.ErrorHandler;
import org.xml.sax.SAXException;
import org.xml.sax.SAXParseException;

/**
 * Wraps {@link XMLErrorHandler} and make it look like a SAX {@link ErrorHandler}.  
 * 
 * <p>
 * The derived class should override the {@link #getErrorHandler()} method
 * so that it will return the correct {@link XMLErrorHandler} instance.
 * This method will be called whenever an error/warning is found.
 * 
 * <p>
 * Experience shows that it is better to store the actual
 * {@link XMLErrorHandler} in one place and looks up that variable,
 * rather than copying it into every component that needs an error handler
 * and update all of them whenever it is changed, IMO.
 * 
 * @author Kohsuke Kawaguchi (kohsuke.kawaguchi@sun.com)
 */
public abstract class ErrorHandlerProxy implements ErrorHandler {
    
    public void error(SAXParseException e) throws SAXException {
        XMLErrorHandler eh = getErrorHandler();
        if(eh instanceof ErrorHandlerWrapper) {
            ((ErrorHandlerWrapper)eh).fErrorHandler.error(e);
        }
        eh.error("","",ErrorHandlerWrapper.createXMLParseException(e));
        // if an XNIException is thrown, just let it go.
        // REVISIT: is this OK? or should we try to wrap it into SAXException?
    }

    public void fatalError(SAXParseException e) throws SAXException {
        XMLErrorHandler eh = getErrorHandler();
        if(eh instanceof ErrorHandlerWrapper) {
            ((ErrorHandlerWrapper)eh).fErrorHandler.fatalError(e);
        }
        eh.fatalError("","",ErrorHandlerWrapper.createXMLParseException(e));
    }

    public void warning(SAXParseException e) throws SAXException {
        XMLErrorHandler eh = getErrorHandler();
        if(eh instanceof ErrorHandlerWrapper) {
            ((ErrorHandlerWrapper)eh).fErrorHandler.warning(e);
        }
        eh.warning("","",ErrorHandlerWrapper.createXMLParseException(e));
    }

    protected abstract XMLErrorHandler getErrorHandler();
}

@@ -1635,32 +1635,33 @@
 er();%0A        if
+ 
 (eh instanceof E
@@ -1674,32 +1674,32 @@
 ndlerWrapper) %7B%0A
-
             ((Er
@@ -1746,32 +1746,51 @@
 r(e);%0A        %7D%0A
+        else %7B%0A    
         eh.error
@@ -1841,24 +1841,34 @@
 eption(e));%0A
+        %7D%0A
         // i
@@ -2118,32 +2118,33 @@
 er();%0A        if
+ 
 (eh instanceof E
@@ -2234,32 +2234,51 @@
 r(e);%0A        %7D%0A
+        else %7B%0A    
         eh.fatal
@@ -2330,32 +2330,42 @@
 eException(e));%0A
+        %7D%0A
     %7D%0A%0A    publi
@@ -2480,16 +2480,17 @@
       if
+ 
 (eh inst
@@ -2585,32 +2585,51 @@
 g(e);%0A        %7D%0A
+        else %7B%0A    
         eh.warni
@@ -2682,24 +2682,34 @@
 eption(e));%0A
+        %7D%0A
     %7D%0A%0A    p

Fixing a bug. It was possible that the error handler would report the same error twice.