You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by vg...@apache.org on 2001/12/15 01:15:48 UTC
cvs commit: xml-cocoon2/src/org/apache/cocoon/transformation LDAPTransformer.java SQLTransformer.java XIncludeTransformer.java
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:ricardo@apache.org">Ricardo Rocha</a>
* @author <a href="mailto:sylvain@apache.org">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:giacomo.pati@pwr.ch">Giacomo Pati</a>
* (PWR Organisation & Entwicklung)
* @author <a href="mailto:sven.beauprez@the-ecorp.com">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:balld@webslingerZ.com">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: webmaster@xml.apache.org
To unsubscribe, e-mail: cocoon-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: cocoon-cvs-help@xml.apache.org