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]

Reply via email to