vgritsenko 01/12/14 16:21:19 Modified: src/org/apache/cocoon/generation ServerPagesGenerator.java src/org/apache/cocoon/transformation LDAPTransformer.java SQLTransformer.java XIncludeTransformer.java Log: Fix bug #4062: ServerPagesGenerator now sets DocumentLocator Revision Changes Path 1.20 +6 -1 xml-cocoon2/src/org/apache/cocoon/generation/ServerPagesGenerator.java Index: ServerPagesGenerator.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/generation/ServerPagesGenerator.java,v retrieving revision 1.19 retrieving revision 1.20 diff -u -r1.19 -r1.20 --- ServerPagesGenerator.java 2001/12/04 08:30:09 1.19 +++ ServerPagesGenerator.java 2001/12/15 00:21:19 1.20 @@ -58,7 +58,7 @@ * * @author <a href="mailto:[EMAIL PROTECTED]">Ricardo Rocha</a> * @author <a href="mailto:[EMAIL PROTECTED]">Sylvain Wallez</a> - * @version CVS $Revision: 1.19 $ $Date: 2001/12/04 08:30:09 $ + * @version CVS $Revision: 1.20 $ $Date: 2001/12/15 00:21:19 $ */ public class ServerPagesGenerator extends ServletGenerator implements Recyclable, Disposable, Cacheable, Configurable @@ -214,6 +214,11 @@ } } + // Fixes BUG#4062: Set document locator which is used by XIncludeTransformer + org.xml.sax.helpers.LocatorImpl locator = new org.xml.sax.helpers.LocatorImpl(); + locator.setSystemId(this.inputSource.getSystemId()); + this.contentHandler.setDocumentLocator(locator); + // log exception and ensure that generator is released. try { generator.generate(); 1.9 +4 -2 xml-cocoon2/src/org/apache/cocoon/transformation/LDAPTransformer.java Index: LDAPTransformer.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/transformation/LDAPTransformer.java,v retrieving revision 1.8 retrieving revision 1.9 diff -u -r1.8 -r1.9 --- LDAPTransformer.java 2001/10/25 20:22:45 1.8 +++ LDAPTransformer.java 2001/12/15 00:21:19 1.9 @@ -906,8 +906,10 @@ /** BEGIN SAX ContentHandler handlers **/ public void setDocumentLocator(Locator locator) { - getLogger().info("PUBLIC ID"+locator.getPublicId()); - getLogger().info("SYSTEM ID"+locator.getSystemId()); + if (getLogger().isDebugEnabled()) { + getLogger().debug( "PUBLIC ID: " + locator.getPublicId() ); + getLogger().debug( "SYSTEM ID: " + locator.getSystemId() ); + } if (super.contentHandler != null) super.contentHandler.setDocumentLocator(locator); } 1.24 +5 -3 xml-cocoon2/src/org/apache/cocoon/transformation/SQLTransformer.java Index: SQLTransformer.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/transformation/SQLTransformer.java,v retrieving revision 1.23 retrieving revision 1.24 diff -u -r1.23 -r1.24 --- SQLTransformer.java 2001/11/28 11:45:39 1.23 +++ SQLTransformer.java 2001/12/15 00:21:19 1.24 @@ -35,7 +35,7 @@ * @author <a href="mailto:[EMAIL PROTECTED]">Giacomo Pati</a> * (PWR Organisation & Entwicklung) * @author <a href="mailto:[EMAIL PROTECTED]">Sven Beauprez</a> - * @version CVS $Revision: 1.23 $ $Date: 2001/11/28 11:45:39 $ $Author: cziegeler $ + * @version CVS $Revision: 1.24 $ $Date: 2001/12/15 00:21:19 $ $Author: vgritsenko $ */ public class SQLTransformer extends AbstractTransformer implements Composable, Recyclable, Disposable, Configurable { @@ -644,8 +644,10 @@ public void setDocumentLocator( Locator locator ) { - getLogger().info( "PUBLIC ID: " + locator.getPublicId() ); - getLogger().info( "SYSTEM ID: " + locator.getSystemId() ); + if (getLogger().isDebugEnabled()) { + getLogger().debug( "PUBLIC ID: " + locator.getPublicId() ); + getLogger().debug( "SYSTEM ID: " + locator.getSystemId() ); + } if ( super.contentHandler != null ) super.contentHandler.setDocumentLocator( locator ); } 1.20 +8 -9 xml-cocoon2/src/org/apache/cocoon/transformation/XIncludeTransformer.java Index: XIncludeTransformer.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/org/apache/cocoon/transformation/XIncludeTransformer.java,v retrieving revision 1.19 retrieving revision 1.20 diff -u -r1.19 -r1.20 --- XIncludeTransformer.java 2001/10/19 11:44:55 1.19 +++ XIncludeTransformer.java 2001/12/15 00:21:19 1.20 @@ -41,7 +41,7 @@ * by the SAX event FSM yet. * * @author <a href="mailto:[EMAIL PROTECTED]">Donald Ball</a> - * @version CVS $Revision: 1.19 $ $Date: 2001/10/19 11:44:55 $ $Author: morrijr $ + * @version CVS $Revision: 1.20 $ $Date: 2001/12/15 00:21:19 $ $Author: vgritsenko $ */ public class XIncludeTransformer extends AbstractTransformer implements Composable, Recyclable, Disposable { @@ -99,7 +99,7 @@ try { startXMLBaseAttribute(uri,name,value); } catch (ProcessingException e) { - getLogger().debug("XincludeTransformer", e); + getLogger().debug("Rethrowing exception", e); throw new SAXException(e); } } @@ -112,10 +112,10 @@ try { processXIncludeElement(href, parse); } catch (ProcessingException e) { - getLogger().debug("XincludeTransformer", e); + getLogger().debug("Rethrowing exception", e); throw new SAXException(e); } catch (IOException e) { - getLogger().debug("XincludeTransformer", e); + getLogger().debug("Rethrowing exception", e); throw new SAXException(e); } return; @@ -135,9 +135,8 @@ public void setDocumentLocator(Locator locator) { try { - if (getLogger().isDebugEnabled()) { - getLogger().debug("XIncludeTransformer: setDocumentLocator called " + locator.getSystemId()); + getLogger().debug("setDocumentLocator called " + locator.getSystemId()); } base_xmlbase_uri = this.resolver.resolve(locator.getSystemId()); @@ -151,7 +150,7 @@ current_xmlbase_uri = base_xmlbase_uri; } - } catch (Exception e) {getLogger().debug("XincludeTransformer", e);} + } catch (Exception e) { getLogger().debug("Exception ignored", e); } super.setDocumentLocator(locator); } @@ -163,7 +162,7 @@ } if (getLogger().isDebugEnabled()) { - getLogger().debug("XIncludeTransformer: XMLBase = " + urlLoc); + getLogger().debug("XMLBase = " + urlLoc); } if (current_xmlbase_uri != null) { @@ -185,7 +184,7 @@ protected void endXMLBaseAttribute() { if (getLogger().isDebugEnabled()) { - getLogger().debug("XIncludeTransformer: XMLBase ended"); + getLogger().debug("XMLBase ended"); } if (xmlbase_stack.size() > 0) {
---------------------------------------------------------------------- In case of troubles, e-mail: [EMAIL PROTECTED] To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]