Author: vgritsenko
Date: Sat Oct 23 17:10:48 2004
New Revision: 55400

Modified:
   
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/AbstractEnvironment.java
   
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/wrapper/MutableEnvironmentFacade.java
Log:
tabs/spaces


Modified: 
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/AbstractEnvironment.java
==============================================================================
--- 
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/AbstractEnvironment.java
        (original)
+++ 
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/AbstractEnvironment.java
        Sat Oct 23 17:10:48 2004
@@ -1,12 +1,12 @@
 /*
  * Copyright 1999-2004 The Apache Software Foundation.
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -44,7 +44,7 @@
  * @author <a href="mailto:[EMAIL PROTECTED]">Bj&ouml;rn L&uuml;tkemeier</a>
  * @author <a href="mailto:[EMAIL PROTECTED]">Giacomo Pati</a>
  * @author <a href="mailto:[EMAIL PROTECTED]">Carsten Ziegeler</a>
- * @version CVS $Id: AbstractEnvironment.java,v 1.19 2004/03/18 15:08:12 
cziegeler Exp $
+ * @version CVS $Id$
  */
 public abstract class AbstractEnvironment extends AbstractLogEnabled 
implements Environment {
 
@@ -92,7 +92,7 @@
 
     /** Do we have our components ? */
     protected boolean initializedComponents = false;
-    
+
     /**
      * Constructs the abstract environment
      */
@@ -196,7 +196,7 @@
      * Get the current Context
      */
     public String getContext() {
-        if ( !this.initializedComponents) {
+        if (!this.initializedComponents) {
             this.initComponents();
         }
         return this.context;
@@ -244,7 +244,7 @@
      */
     public void changeContext(String prefix, String newContext)
     throws IOException {
-        if ( !this.initializedComponents) {
+        if (!this.initializedComponents) {
             this.initComponents();
         }
 
@@ -254,6 +254,7 @@
             getLogger().debug("  to context(" + newContext + ") and prefix(" + 
prefix + ")");
             getLogger().debug("  at URI " + this.uris);
         }
+
         int l = prefix.length();
         if (l >= 1) {
             if (!this.uris.startsWith(prefix)) {
@@ -278,7 +279,7 @@
             if (getLogger().isDebugEnabled()) {
                 getLogger().debug("Base context is zip: " + this.context);
             }
-            
+
             org.apache.excalibur.source.Source source = null;
             try {
                 source = this.sourceResolver.resolveURI(this.context + 
newContext);
@@ -305,7 +306,7 @@
             if (i != -1 && i + 1 < sContext.length()) {
                 sContext = sContext.substring(0, i + 1);
             }
-            
+
             org.apache.excalibur.source.Source source = null;
             try {
                 source = this.sourceResolver.resolveURI(sContext);
@@ -368,13 +369,17 @@
      */
     public Source resolve(String systemId)
     throws ProcessingException, SAXException, IOException {
-        if ( !this.initializedComponents) {
-            this.initComponents();
+        if (!this.initializedComponents) {
+            initComponents();
         }
+
         if (getLogger().isDebugEnabled()) {
-            this.getLogger().debug("Resolving '"+systemId+"' in context '" + 
this.context + "'");
+            getLogger().debug("Resolving '" + systemId + "' in context '" + 
this.context + "'");
+        }
+
+        if (systemId == null) {
+            throw new SAXException("Invalid System ID");
         }
-        if (systemId == null) throw new SAXException("Invalid System ID");
 
         // get the wrapper class - we don't want to import the wrapper directly
         // to avoid a direct dependency from the core to the deprecation 
package
@@ -386,8 +391,9 @@
                                           +"Please either update your code to 
use the new resolveURI() method or"
                                           +" install the deprecation 
support.", e);
         }
-        if ( null == avalonToCocoonSourceWrapper ) {
-            synchronized (this.getClass()) {
+
+        if (null == avalonToCocoonSourceWrapper) {
+            synchronized (getClass()) {
                 try {
                     avalonToCocoonSourceWrapper = 
clazz.getDeclaredMethod("createProxy",
                            new Class[] 
{ClassUtils.loadClass("org.apache.excalibur.source.Source"),
@@ -400,13 +406,13 @@
                                                   +" install the deprecation 
support.", e);
                 }
             }
-           
         }
+
         try {
-            org.apache.excalibur.source.Source source = this.resolveURI( 
systemId );
-            Source wrappedSource;
-            wrappedSource = (Source)avalonToCocoonSourceWrapper.invoke(clazz,
-                        new Object[] {source, this.sourceResolver, this, 
this.manager});
+            org.apache.excalibur.source.Source source = resolveURI(systemId);
+            Source wrappedSource = (Source)avalonToCocoonSourceWrapper.invoke(
+                    clazz,
+                    new Object[] {source, this.sourceResolver, this, 
this.manager});
             return wrappedSource;
         } catch (SourceException se) {
             throw SourceUtil.handle(se);
@@ -504,7 +510,7 @@
     throws IOException {
         if (this.secureOutputStream != null) {
             this.secureOutputStream.realFlush();
-        } else if ( this.outputStream != null ){
+        } else if (this.outputStream != null) {
             this.outputStream.flush();
         }
     }
@@ -513,8 +519,7 @@
      * Get a <code>Source</code> object.
      */
     public org.apache.excalibur.source.Source resolveURI(final String location)
-    throws MalformedURLException, IOException, SourceException
-    {
+    throws MalformedURLException, IOException, SourceException {
         return this.resolveURI(location, null, null);
     }
 
@@ -523,9 +528,9 @@
      */
     public org.apache.excalibur.source.Source resolveURI(final String location,
                                                          String baseURI,
-                                                         final Map    
parameters)
+                                                         final Map parameters)
     throws MalformedURLException, IOException, SourceException {
-        if ( !this.initializedComponents) {
+        if (!this.initializedComponents) {
             this.initComponents();
         }
         return this.sourceResolver.resolveURI(location, baseURI, parameters);
@@ -534,9 +539,9 @@
     /**
      * Releases a resolved resource
      */
-    public void release( final org.apache.excalibur.source.Source source ) {
-        if ( null != source ) {
-            this.sourceResolver.release( source );
+    public void release(final org.apache.excalibur.source.Source source) {
+        if (null != source) {
+            this.sourceResolver.release(source);
         }
     }
 
@@ -554,7 +559,7 @@
                 try {
                     source = 
this.sourceResolver.resolveURI(this.tempInitContext);
                     this.context = source.getURI();
-                    
+
                     if (this.rootContext == null) // hack for 
EnvironmentWrapper
                         this.rootContext = this.context;
                 } finally {
@@ -569,7 +574,7 @@
             throw new CascadingRuntimeException("Unable to resolve URI: 
"+this.tempInitContext, ie);
         }
     }
-    
+
     /**
      * Notify that the processing starts.
      */
@@ -582,14 +587,14 @@
      * This can be used to cleanup the environment object
      */
     public void finishingProcessing() {
-        if ( null != this.manager ) {
-            this.manager.release( this.sourceResolver );
+        if (null != this.manager) {
+            this.manager.release(this.sourceResolver);
             this.manager = null;
             this.sourceResolver = null;
         }
         this.initializedComponents = false;
     }
-    
+
     /* (non-Javadoc)
      * @see org.apache.cocoon.environment.Environment#isInternRedirect()
      */

Modified: 
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/wrapper/MutableEnvironmentFacade.java
==============================================================================
--- 
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/wrapper/MutableEnvironmentFacade.java
   (original)
+++ 
cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/environment/wrapper/MutableEnvironmentFacade.java
   Sat Oct 23 17:10:48 2004
@@ -135,7 +135,7 @@
      * @see org.apache.cocoon.environment.Environment#getAction()
      */
     public String getAction() {
-            return env.getAction();
+        return env.getAction();
     }
 
     /* (non-Javadoc)

Reply via email to