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]