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]

Reply via email to