cziegeler    2002/12/04 03:06:44

  Modified:    src/java/org/apache/cocoon/components/source/impl
                        SitemapSource.java
               src/java/org/apache/cocoon/transformation
                        TraxTransformer.java
               src/webapp/WEB-INF cocoon.xconf
  Added:       src/java/org/apache/cocoon/components/source/impl
                        SitemapSourceFactory.java
  Removed:     src/java/org/apache/cocoon/components/source/impl
                        CocoonSourceFactory.java
  Log:
  Cleaning up code a little bit
  
  Revision  Changes    Path
  1.27      +38 -34    
xml-cocoon2/src/java/org/apache/cocoon/components/source/impl/SitemapSource.java
  
  Index: SitemapSource.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/source/impl/SitemapSource.java,v
  retrieving revision 1.26
  retrieving revision 1.27
  diff -u -r1.26 -r1.27
  --- SitemapSource.java        4 Dec 2002 09:48:40 -0000       1.26
  +++ SitemapSource.java        4 Dec 2002 11:06:44 -0000       1.27
  @@ -50,17 +50,10 @@
   */
   package org.apache.cocoon.components.source.impl;
   
  -import org.apache.excalibur.source.Recyclable;
  -import org.apache.excalibur.source.Source;
  -import org.apache.excalibur.source.SourceException;
  -import org.apache.excalibur.source.SourceNotFoundException;
  -import org.apache.excalibur.source.SourceValidity;
  -import org.apache.excalibur.source.impl.validity.AggregatedValidity;
  -
   import org.apache.avalon.excalibur.xml.XMLizable;
  -
   import org.apache.avalon.framework.component.ComponentException;
   import org.apache.avalon.framework.component.ComponentManager;
  +import org.apache.avalon.framework.logger.AbstractLogEnabled;
   import org.apache.avalon.framework.logger.Logger;
   import org.apache.cocoon.Processor;
   import org.apache.cocoon.ResourceNotFoundException;
  @@ -69,18 +62,21 @@
   import org.apache.cocoon.components.pipeline.ProcessingPipeline;
   import org.apache.cocoon.environment.Environment;
   import org.apache.cocoon.environment.ObjectModelHelper;
  -import org.apache.cocoon.environment.Request;
   import org.apache.cocoon.environment.wrapper.EnvironmentWrapper;
  -import org.apache.cocoon.xml.AbstractXMLConsumer;
   import org.apache.cocoon.xml.ContentHandlerWrapper;
   import org.apache.cocoon.xml.XMLConsumer;
  +import org.apache.excalibur.source.Recyclable;
  +import org.apache.excalibur.source.Source;
  +import org.apache.excalibur.source.SourceException;
  +import org.apache.excalibur.source.SourceNotFoundException;
  +import org.apache.excalibur.source.SourceValidity;
  +import org.apache.excalibur.source.impl.validity.AggregatedValidity;
   import org.xml.sax.Attributes;
   import org.xml.sax.ContentHandler;
   import org.xml.sax.InputSource;
   import org.xml.sax.Locator;
   import org.xml.sax.SAXException;
   import org.xml.sax.ext.LexicalHandler;
  -
   import java.io.ByteArrayInputStream;
   import java.io.ByteArrayOutputStream;
   import java.io.IOException;
  @@ -90,13 +86,14 @@
   import java.util.Map;
   
   /**
  - * Description of a source which is defined by a pipeline.
  + * Implementation of a {@link Source} that gets its content
  + * by invoking a pipeline.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
    * @version CVS $Id$
    */
   public final class SitemapSource
  -extends AbstractXMLConsumer
  +extends AbstractLogEnabled
   implements Source, XMLizable, Recyclable {
   
       /** validities for the internal pipeline */
  @@ -145,9 +142,9 @@
        * Construct a new object
        */
       public SitemapSource(ComponentManager manager,
  -                         String           uri,
  -                         Map              parameters,
  -                         Logger           logger)
  +                          String           uri,
  +                          Map              parameters,
  +                          Logger           logger)
       throws MalformedURLException {
   
           Environment env = CocoonComponentManager.getCurrentEnvironment();
  @@ -183,7 +180,7 @@
               this.prefix = null;
               this.processor = CocoonComponentManager.getCurrentProcessor();
           } else {
  -            throw new MalformedURLException("Malformed cocoon URI.");
  +            throw new MalformedURLException("Malformed cocoon URI: " + uri);
           }
   
           // create the queryString (if available)
  @@ -195,6 +192,7 @@
           } else if (position > 0) {
               uri = uri.substring(position);
           }
  +        this.uri = uri;
   
           // build the request uri which is relative to the context
           String requestURI = (this.prefix == null ? env.getURIPrefix() + uri : uri);
  @@ -204,14 +202,18 @@
               "cocoon://" + requestURI :
               "cocoon://" + requestURI + "?" + queryString;
   
  -        this.environment = new EnvironmentWrapper(env, requestURI, queryString, 
logger, manager, rawMode);
  +        // create environment...
  +        this.environment = new EnvironmentWrapper(env, requestURI, 
  +                                                   queryString, logger, manager, 
rawMode);
  +        // ...and put information passed from the parent request to the internal 
request
           if ( null != parameters ) {
               this.environment.getObjectModel().put(ObjectModelHelper.PARENT_CONTEXT, 
parameters);
           } else {
               
this.environment.getObjectModel().remove(ObjectModelHelper.PARENT_CONTEXT);
           }
  -        this.uri = uri;
  -        this.discardValidity();
  +        
  +        // initialize
  +        this.init();
       }
   
       /**
  @@ -310,9 +312,15 @@
        * and content length.
        */
       public void discardValidity() {
  -        reset();
  -        try {
  +        this.reset();
  +        this.init();
  +    }
   
  +    /**
  +     * Initialize
  +     */
  +    protected void init() {
  +        try {
               this.processKey = 
CocoonComponentManager.startProcessing(this.environment);
               this.environment.setURI(this.prefix, this.uri);
               this.processingPipeline = 
this.processor.processInternal(this.environment);
  @@ -345,16 +353,6 @@
       }
   
       /**
  -     * Return a new <code>InputSource</code> object
  -     */
  -    public InputSource getInputSource()
  -    throws IOException, SourceException {
  -        InputSource newObject = new InputSource(this.getInputStream());
  -        newObject.setSystemId(this.systemId);
  -        return newObject;
  -    }
  -
  -    /**
        * Stream content to the content handler
        */
       public void toSAX(ContentHandler contentHandler)
  @@ -403,6 +401,9 @@
           }
       }
   
  +    /**
  +     * Reset everything
  +     */
       private void reset() {
           if (this.processingPipeline != null) this.processingPipeline.release();
           if (this.processKey != null) {
  @@ -420,8 +421,11 @@
           this.pipelineProcessor = null;
       }
   
  +    /**
  +     * Recyclable
  +     */
       public void recycle() {
  -        reset();
  +        this.reset();
       }
   
       /**
  
  
  
  1.1                  
xml-cocoon2/src/java/org/apache/cocoon/components/source/impl/SitemapSourceFactory.java
  
  Index: SitemapSourceFactory.java
  ===================================================================
  /*
  
   ============================================================================
                     The Apache Software License, Version 1.1
   ============================================================================
  
   Copyright (C) 1999-2002 The Apache Software Foundation. All rights reserved.
  
   Redistribution and use in source and binary forms, with or without modifica-
   tion, are permitted provided that the following conditions are met:
  
   1. Redistributions of  source code must  retain the above copyright  notice,
      this list of conditions and the following disclaimer.
  
   2. Redistributions in binary form must reproduce the above copyright notice,
      this list of conditions and the following disclaimer in the documentation
      and/or other materials provided with the distribution.
  
   3. The end-user documentation included with the redistribution, if any, must
      include  the following  acknowledgment:  "This product includes  software
      developed  by the  Apache Software Foundation  (http://www.apache.org/)."
      Alternately, this  acknowledgment may  appear in the software itself,  if
      and wherever such third-party acknowledgments normally appear.
  
   4. The names "Apache Cocoon" and  "Apache Software Foundation" must  not  be
      used to  endorse or promote  products derived from  this software without
      prior written permission. For written permission, please contact
      [EMAIL PROTECTED]
  
   5. Products  derived from this software may not  be called "Apache", nor may
      "Apache" appear  in their name,  without prior written permission  of the
      Apache Software Foundation.
  
   THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESSED OR IMPLIED WARRANTIES,
   INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
   FITNESS  FOR A PARTICULAR  PURPOSE ARE  DISCLAIMED.  IN NO  EVENT SHALL  THE
   APACHE SOFTWARE  FOUNDATION  OR ITS CONTRIBUTORS  BE LIABLE FOR  ANY DIRECT,
   INDIRECT, INCIDENTAL, SPECIAL,  EXEMPLARY, OR CONSEQUENTIAL  DAMAGES (INCLU-
   DING, BUT NOT LIMITED TO, PROCUREMENT  OF SUBSTITUTE GOODS OR SERVICES; LOSS
   OF USE, DATA, OR  PROFITS; OR BUSINESS  INTERRUPTION)  HOWEVER CAUSED AND ON
   ANY  THEORY OF LIABILITY,  WHETHER  IN CONTRACT,  STRICT LIABILITY,  OR TORT
   (INCLUDING  NEGLIGENCE OR  OTHERWISE) ARISING IN  ANY WAY OUT OF THE  USE OF
   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  
   This software  consists of voluntary contributions made  by many individuals
   on  behalf of the Apache Software  Foundation and was  originally created by
   Stefano Mazzocchi  <[EMAIL PROTECTED]>. For more  information on the Apache
   Software Foundation, please see <http://www.apache.org/>.
  
  */
  package org.apache.cocoon.components.source.impl;
  
  import java.io.IOException;
  import java.net.MalformedURLException;
  import java.util.Map;
  
  import org.apache.avalon.framework.component.ComponentManager;
  import org.apache.avalon.framework.component.Composable;
  import org.apache.avalon.framework.logger.AbstractLogEnabled;
  import org.apache.avalon.framework.thread.ThreadSafe;
  
  import org.apache.excalibur.source.Source;
  import org.apache.excalibur.source.SourceFactory;
  
  /**
   * This class implements the cocoon: protocol.
   * It cannot be used like other source factories
   * as it needs the current <code>Sitemap</code> as input.
   *
   * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
   * @version CVS $Id: CocoonSourceFactory.java,v 1.3 2002/08/08 02:10:40 vgritsenko 
Exp $
   */
  public final class SitemapSourceFactory
      extends AbstractLogEnabled
      implements SourceFactory, ThreadSafe, Composable
  {
      /** The <code>ComponentManager</code> */
      private ComponentManager manager;
  
      /**
       * Composable
       */
      public void compose(ComponentManager manager) {
          this.manager = manager;
      }
  
      /**
       * Get a <code>Source</code> object.
       * @param parameters This is optional.
       */
      public Source getSource( String location, Map parameters )
          throws MalformedURLException, IOException
      {
          if( getLogger().isDebugEnabled() )
          {
              getLogger().debug( "Creating source object for " + location );
          }
  
          return new SitemapSource( this.manager,
                                    location,
                                    parameters,
                                    getLogger());
      }
  }
  
  
  
  1.37      +12 -9     
xml-cocoon2/src/java/org/apache/cocoon/transformation/TraxTransformer.java
  
  Index: TraxTransformer.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/transformation/TraxTransformer.java,v
  retrieving revision 1.36
  retrieving revision 1.37
  diff -u -r1.36 -r1.37
  --- TraxTransformer.java      11 Nov 2002 14:20:49 -0000      1.36
  +++ TraxTransformer.java      4 Dec 2002 11:06:44 -0000       1.37
  @@ -52,12 +52,10 @@
   
   import org.apache.avalon.excalibur.xml.xslt.XSLTProcessor;
   import org.apache.avalon.excalibur.xml.xslt.XSLTProcessorException;
  -
   import org.apache.avalon.framework.activity.Disposable;
   import org.apache.avalon.framework.component.ComponentException;
   import org.apache.avalon.framework.component.ComponentManager;
   import org.apache.avalon.framework.component.Composable;
  -import org.apache.avalon.framework.component.ComponentException;
   import org.apache.avalon.framework.configuration.Configurable;
   import org.apache.avalon.framework.configuration.Configuration;
   import org.apache.avalon.framework.configuration.ConfigurationException;
  @@ -68,27 +66,26 @@
   import org.apache.cocoon.components.browser.Browser;
   import org.apache.cocoon.components.deli.Deli;
   import org.apache.cocoon.components.source.SourceUtil;
  -import org.apache.cocoon.environment.Session;
   import org.apache.cocoon.environment.Cookie;
   import org.apache.cocoon.environment.ObjectModelHelper;
   import org.apache.cocoon.environment.Request;
  +import org.apache.cocoon.environment.Session;
   import org.apache.cocoon.environment.SourceResolver;
   import org.apache.cocoon.xml.XMLConsumer;
  -
   import org.apache.excalibur.source.Source;
   import org.apache.excalibur.source.SourceException;
   import org.apache.excalibur.source.SourceValidity;
  -
   import org.xml.sax.SAXException;
   
   import javax.xml.transform.sax.SAXResult;
   import javax.xml.transform.sax.TransformerHandler;
  +
   import java.io.IOException;
  -import java.util.Map;
  +import java.util.Enumeration;
  +import java.util.HashMap;
   import java.util.Iterator;
  +import java.util.Map;
   import java.util.Set;
  -import java.util.HashMap;
  -import java.util.Enumeration;
   
   /**
    * This Transformer is used to transform this incomming SAX stream using
  @@ -580,6 +577,9 @@
           return true;
       }
   
  +    /**
  +     * Disposable
  +     */
       public void dispose() {
           this.manager.release(this.browser);
           this.manager.release(this.xsltProcessor);
  @@ -590,6 +590,9 @@
           this.manager = null;
       }
   
  +    /**
  +     * Recyclable
  +     */
       public void recycle() {
           this.transformerHandler = null;
           this.transformerValidity = null;
  
  
  
  1.47      +1 -1      xml-cocoon2/src/webapp/WEB-INF/cocoon.xconf
  
  Index: cocoon.xconf
  ===================================================================
  RCS file: /home/cvs/xml-cocoon2/src/webapp/WEB-INF/cocoon.xconf,v
  retrieving revision 1.46
  retrieving revision 1.47
  diff -u -r1.46 -r1.47
  --- cocoon.xconf      4 Dec 2002 10:30:58 -0000       1.46
  +++ cocoon.xconf      4 Dec 2002 11:06:44 -0000       1.47
  @@ -157,7 +157,7 @@
     <source-factories>
         <component-instance 
class="org.apache.excalibur.source.impl.ResourceSourceFactory" name="resource"/>
         <component-instance 
class="org.apache.cocoon.components.source.impl.ContextSourceFactory" name="context"/>
  -      <component-instance 
class="org.apache.cocoon.components.source.impl.CocoonSourceFactory" name="cocoon"/>
  +      <component-instance 
class="org.apache.cocoon.components.source.impl.SitemapSourceFactory" name="cocoon"/>
         <!-- file protocol : this is a WriteableSource -->
         <component-instance 
class="org.apache.cocoon.components.source.impl.FileSourceFactory" name="file"/>
     
  
  
  

----------------------------------------------------------------------
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