vgritsenko 01/12/14 16:15:48 Modified: src/org/apache/cocoon/generation Tag: cocoon_20_branch ServerPagesGenerator.java src/org/apache/cocoon/transformation Tag: cocoon_20_branch LDAPTransformer.java SQLTransformer.java XIncludeTransformer.java Log: Fix bug #4062: ServerPagesGenerator now sets DocumentLocator Revision Changes Path No revision No revision 1.7.2.13 +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.7.2.12 retrieving revision 1.7.2.13 diff -u -r1.7.2.12 -r1.7.2.13 --- ServerPagesGenerator.java 2001/12/04 08:28:33 1.7.2.12 +++ ServerPagesGenerator.java 2001/12/15 00:15:47 1.7.2.13 @@ -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.7.2.12 $ $Date: 2001/12/04 08:28:33 $ + * @version CVS $Revision: 1.7.2.13 $ $Date: 2001/12/15 00:15:47 $ */ 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(); No revision No revision 1.1.2.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.1.2.8 retrieving revision 1.1.2.9 diff -u -r1.1.2.8 -r1.1.2.9 --- LDAPTransformer.java 2001/10/25 20:22:22 1.1.2.8 +++ LDAPTransformer.java 2001/12/15 00:15:48 1.1.2.9 @@ -907,8 +907,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.5.2.16 +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.5.2.15 retrieving revision 1.5.2.16 diff -u -r1.5.2.15 -r1.5.2.16 --- SQLTransformer.java 2001/11/15 11:20:40 1.5.2.15 +++ SQLTransformer.java 2001/12/15 00:15:48 1.5.2.16 @@ -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.5.2.15 $ $Date: 2001/11/15 11:20:40 $ $Author: cziegeler $ + * @version CVS $Revision: 1.5.2.16 $ $Date: 2001/12/15 00:15:48 $ $Author: vgritsenko $ */ public class SQLTransformer extends AbstractTransformer implements Composable, Recyclable, Disposable, Configurable { @@ -589,8 +589,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.6.2.12 +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.6.2.11 retrieving revision 1.6.2.12 diff -u -r1.6.2.11 -r1.6.2.12 --- XIncludeTransformer.java 2001/10/18 17:23:46 1.6.2.11 +++ XIncludeTransformer.java 2001/12/15 00:15:48 1.6.2.12 @@ -42,7 +42,7 @@ * by the SAX event FSM yet. * * @author <a href="mailto:[EMAIL PROTECTED]">Donald Ball</a> - * @version CVS $Revision: 1.6.2.11 $ $Date: 2001/10/18 17:23:46 $ $Author: bloritsch $ + * @version CVS $Revision: 1.6.2.12 $ $Date: 2001/12/15 00:15:48 $ $Author: vgritsenko $ */ public class XIncludeTransformer extends AbstractTransformer implements Composable, Recyclable, Disposable { @@ -92,7 +92,7 @@ try { startXMLBaseAttribute(uri,name,value); } catch (ProcessingException e) { - getLogger().debug("XincludeTransformer", e); + getLogger().debug("Rethrowing exception", e); throw new SAXException(e); } } @@ -105,10 +105,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; @@ -128,9 +128,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()); @@ -144,7 +143,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); } @@ -156,7 +155,7 @@ } if (getLogger().isDebugEnabled()) { - getLogger().debug("XIncludeTransformer: XMLBase = " + urlLoc); + getLogger().debug("XMLBase = " + urlLoc); } if (current_xmlbase_uri != null) { @@ -178,7 +177,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]