]> source.dussan.org Git - archiva.git/commitdiff
general clean up, addition of copyrights
authorBrett Porter <brett@apache.org>
Wed, 7 Jun 2006 05:47:45 +0000 (05:47 +0000)
committerBrett Porter <brett@apache.org>
Wed, 7 Jun 2006 05:47:45 +0000 (05:47 +0000)
git-svn-id: https://svn.apache.org/repos/asf/maven/repository-manager/trunk@412295 13f79535-47bb-0310-9956-ffa450edef68

68 files changed:
maven-repository-application/src/main/java/org/apache/maven/repository/manager/cli/IndexSearcherCli.java
maven-repository-artifact-applet/pom.xml
maven-repository-artifact-applet/src/main/java/org/apache/maven/repository/applet/ChecksumApplet.java
maven-repository-converter/src/main/java/org/apache/maven/repository/converter/DefaultRepositoryConverter.java
maven-repository-converter/src/main/java/org/apache/maven/repository/converter/transaction/TransactionEvent.java
maven-repository-converter/src/main/resources/org/apache/maven/repository/converter/DefaultRepositoryConverter.properties
maven-repository-converter/src/test/java/org/apache/maven/repository/converter/RepositoryConverterTest.java
maven-repository-converter/src/test/source-repository/test/poms/newversion-artifact-1.0.1.pom
maven-repository-converter/src/test/source-repository/test/poms/v3-warnings-artifact-1.0.0.pom
maven-repository-converter/src/test/target-repository/test/newversion-artifact/1.0.0/newversion-artifact-1.0.0.pom
maven-repository-discovery/src/main/java/org/apache/maven/repository/discovery/AbstractArtifactDiscoverer.java
maven-repository-discovery/src/main/java/org/apache/maven/repository/discovery/AbstractDiscoverer.java
maven-repository-discovery/src/main/java/org/apache/maven/repository/discovery/DefaultMetadataDiscoverer.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/AbstractRepositoryIndex.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndex.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexSearcher.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/EclipseRepositoryIndex.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/MetadataRepositoryIndex.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/PomRepositoryIndex.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndex.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearchHit.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearchLayer.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexingFactory.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/query/Query.java
maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/query/SinglePhraseQuery.java
maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexingTest.java
maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/EclipseRepositoryIndexTest.java
maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/MetadataRepositoryIndexingTest.java
maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/PomRepositoryIndexingTest.java
maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/RepositoryIndexSearchLayerTest.java
maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/query/QueryTest.java
maven-repository-manager-site/pom.xml
maven-repository-manager-site/src/site/site.xml
maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/DefaultProxyManager.java
maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/ProxyManager.java
maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/configuration/MavenProxyPropertyLoader.java
maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/configuration/ProxyConfiguration.java
maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/configuration/ValidationException.java
maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/repository/ProxyRepository.java
maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/DefaultProxyManagerTest.java
maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/LegacyProxyManagerTest.java
maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/configuration/MavenProxyPropertyLoaderTest.java
maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/configuration/ProxyConfigurationTest.java
maven-repository-reports-standard/src/main/java/org/apache/maven/repository/reporting/Cache.java
maven-repository-utils/src/main/java/org/apache/maven/repository/ArtifactUtils.java
maven-repository-utils/src/main/java/org/apache/maven/repository/digest/DefaultDigester.java
maven-repository-utils/src/test/java/org/apache/maven/repository/ArtifactUtilsLegacyTest.java
maven-repository-webapp/pom.xml
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/BaseAction.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/GeneralSearchAction.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/PackageSearchAction.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/RepositoryBrowseAction.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/execution/DiscovererExecution.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/job/Configuration.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/job/DiscovererJob.java
maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/job/DiscovererScheduler.java
maven-repository-webapp/src/main/webapp/WEB-INF/decorators.xml
maven-repository-webapp/src/main/webapp/WEB-INF/jsp/browse.jsp
maven-repository-webapp/src/main/webapp/WEB-INF/jsp/decorators/default.jsp
maven-repository-webapp/src/main/webapp/WEB-INF/jsp/form.jspf
maven-repository-webapp/src/main/webapp/WEB-INF/jsp/index.jsp
maven-repository-webapp/src/main/webapp/WEB-INF/web.xml
maven-repository-webapp/src/main/webapp/css/maven-base.css
maven-repository-webapp/src/main/webapp/css/maven-theme.css
maven-repository-webapp/src/main/webapp/css/print.css
maven-repository-webapp/src/main/webapp/css/site.css
maven-repository-webapp/src/main/webapp/index.jsp

index f711a269d9b0cf6836f350ed33d95870cf7d54cb..97470feff905baeec268e574de50541878519aa9 100644 (file)
@@ -20,10 +20,10 @@ import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;
 import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;
 import org.apache.maven.repository.indexing.ArtifactRepositoryIndex;
+import org.apache.maven.repository.indexing.DefaultRepositoryIndexSearcher;
 import org.apache.maven.repository.indexing.RepositoryIndexException;
 import org.apache.maven.repository.indexing.RepositoryIndexSearchException;
 import org.apache.maven.repository.indexing.RepositoryIndexingFactory;
-import org.apache.maven.repository.indexing.DefaultRepositoryIndexSearcher;
 import org.apache.maven.repository.indexing.query.SinglePhraseQuery;
 import org.codehaus.classworlds.ClassWorld;
 import org.codehaus.plexus.PlexusContainerException;
@@ -40,6 +40,10 @@ import java.net.MalformedURLException;
  */
 public class IndexSearcherCli
 {
+    private IndexSearcherCli()
+    {
+    }
+
     public static void main( String[] args )
         throws PlexusContainerException, ComponentLookupException, RepositoryIndexException, MalformedURLException,
         RepositoryIndexSearchException
@@ -62,7 +66,7 @@ public class IndexSearcherCli
 
         ArtifactRepositoryIndex index =
             indexFactory.createArtifactRepositoryIndex( new File( args[0], ".index" ).getAbsolutePath(), repository );
-              
+
         DefaultRepositoryIndexSearcher searcher = indexFactory.createDefaultRepositoryIndexSearcher( index );
 
         try
index 144879018837505e000a3cc582a203d1b5d3279e..c18b795a9cda38f7ebca2878a0a65e2c160c5c82 100644 (file)
@@ -1,3 +1,19 @@
+<!--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
index d89470d6e6a22eb69bc32911a1dfef97d94072ab..635d8edfaed18ca961d1fba1cdbf0f66b9d747bd 100644 (file)
@@ -14,7 +14,6 @@ package org.apache.maven.repository.applet;
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  * See the License for the specific language governing permissions and
  * limitations under the License.
- *
  */
 
 import javax.swing.*;
@@ -56,53 +55,73 @@ public class ChecksumApplet
     public String generateMd5( final String file )
         throws IOException, NoSuchAlgorithmException
     {
-        return (String) AccessController.doPrivileged( new PrivilegedAction()
+        Object o = AccessController.doPrivileged( new PrivilegedAction()
         {
             public Object run()
             {
                 try
                 {
-                    MessageDigest digest = MessageDigest.getInstance( "MD5" );
-
-                    long total = new File( file ).length();
-                    InputStream fis = new FileInputStream( file );
-                    try
-                    {
-                        long totalRead = 0;
-                        byte[] buffer = new byte[CHECKSUM_BUFFER_SIZE];
-                        int numRead;
-                        do
-                        {
-                            numRead = fis.read( buffer );
-                            if ( numRead > 0 )
-                            {
-                                digest.update( buffer, 0, numRead );
-                                totalRead += numRead;
-                                progressBar.setValue( (int) ( totalRead * progressBar.getMaximum() / total ) );
-                            }
-                        }
-                        while ( numRead != -1 );
-                    }
-                    finally
-                    {
-                        fis.close();
-                    }
-
-                    return byteArrayToHexStr( digest.digest() );
+                    return checksumFile( file );
                 }
                 catch ( NoSuchAlgorithmException e )
                 {
-                    throw new RuntimeException( e );
+                    return e;
                 }
                 catch ( IOException e )
                 {
-                    throw new RuntimeException( e );
+                    return e;
                 }
             }
         } );
+
+        //noinspection ChainOfInstanceofChecks
+        if ( o instanceof IOException )
+        {
+            throw (IOException) o;
+        }
+        else if ( o instanceof NoSuchAlgorithmException )
+        {
+            throw (NoSuchAlgorithmException) o;
+        }
+        else
+        {
+            return (String) o;
+        }
+    }
+
+    protected String checksumFile( String file )
+        throws NoSuchAlgorithmException, IOException
+    {
+        MessageDigest digest = MessageDigest.getInstance( "MD5" );
+
+        long total = new File( file ).length();
+        InputStream fis = new FileInputStream( file );
+        try
+        {
+            long totalRead = 0;
+            byte[] buffer = new byte[CHECKSUM_BUFFER_SIZE];
+            int numRead;
+            do
+            {
+                numRead = fis.read( buffer );
+                if ( numRead > 0 )
+                {
+                    digest.update( buffer, 0, numRead );
+                    totalRead += numRead;
+                    progressBar.setValue( (int) ( totalRead * progressBar.getMaximum() / total ) );
+                }
+            }
+            while ( numRead != -1 );
+        }
+        finally
+        {
+            fis.close();
+        }
+
+        return byteArrayToHexStr( digest.digest() );
     }
 
-    private static String byteArrayToHexStr( byte[] data )
+    protected static String byteArrayToHexStr( byte[] data )
     {
         String output = "";
 
index ee96ccd0068135e9fedfda6b9fc229b5a047a210..a14c8016caa22561a45824b2032a4f411dfca619 100644 (file)
@@ -32,6 +32,7 @@ import org.apache.maven.model.Model;
 import org.apache.maven.model.Relocation;
 import org.apache.maven.model.converter.ArtifactPomRewriter;
 import org.apache.maven.model.converter.ModelConverter;
+import org.apache.maven.model.converter.PomTranslationException;
 import org.apache.maven.model.io.xpp3.MavenXpp3Writer;
 import org.apache.maven.model.v3_0_0.io.xpp3.MavenXpp3Reader;
 import org.apache.maven.repository.converter.transaction.FileTransaction;
@@ -43,6 +44,7 @@ import org.codehaus.plexus.util.IOUtil;
 import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
 
 import java.io.File;
+import java.io.FileNotFoundException;
 import java.io.FileReader;
 import java.io.IOException;
 import java.io.StringReader;
@@ -213,6 +215,10 @@ public class DefaultRepositoryConverter
             fileReader = new FileReader( file );
             metadata = reader.read( fileReader );
         }
+        catch ( FileNotFoundException e )
+        {
+            throw new RepositoryConversionException( "Error reading target metadata", e );
+        }
         catch ( IOException e )
         {
             throw new RepositoryConversionException( "Error reading target metadata", e );
@@ -443,10 +449,16 @@ public class DefaultRepositoryConverter
                     reporter.addFailure( artifact, getI18NString( "failure.invalid.source.pom", e.getMessage() ) );
                     result = false;
                 }
-                catch ( Exception e )
+                catch ( IOException e )
                 {
                     throw new RepositoryConversionException( "Unable to write converted POM", e );
                 }
+                catch ( PomTranslationException e )
+                {
+                    // TODO! check handling, fix error message
+                    reporter.addFailure( artifact, getI18NString( "failure.invalid.source.pom", e.getMessage() ) );
+                    result = false;
+                }
                 finally
                 {
                     IOUtil.close( writer );
index 45b792992c6a424fb878c0f2094b61d93fdfeec0..2f8a528e487ee20456bf5d2f31dd6de2203260f7 100644 (file)
@@ -1,7 +1,5 @@
 package org.apache.maven.repository.converter.transaction;
 
-import java.io.IOException;
-
 /*
  * Copyright 2005-2006 The Apache Software Foundation.
  *
@@ -18,6 +16,8 @@ import java.io.IOException;
  * limitations under the License.
  */
 
+import java.io.IOException;
+
 /**
  * Interface for individual events in a transaction.
  *
index f6089c1f5ee16f6e22851bbf7a317d16181d31e7..88e4d1f304d8face362f3ec29a80b1008fba74df 100644 (file)
@@ -23,6 +23,7 @@ warning.missing.pom=The artifact had no POM in the source repository.
 
 exception.repositories.match=Source and target repositories are identical.
 
+# TODO! update definitions
 failure.incorrect.groupMetadata.groupId=The group ID in the source group metadata is incorrect.
 
 failure.incorrect.artifactMetadata.artifactId=The artifact ID in the source artifact metadata is incorrect.
index 33817594d29b3e143c58a2f60dcc4762e1000e33..3dadd5420bea1c16a5a592a06bff713021142357 100644 (file)
@@ -581,10 +581,10 @@ public class RepositoryConverterTest
         File targetFile = new File( targetRepository.getBasedir(), targetRepository.pathOf( artifact ) );
         File targetPomFile = new File( targetRepository.getBasedir(), targetRepository.pathOf( pomArtifact ) );
 
-        SimpleDateFormat dateFormat = new SimpleDateFormat( "yyyy-MM-dd" );
+        SimpleDateFormat dateFormat = new SimpleDateFormat( "yyyy-MM-dd", Locale.getDefault() );
         long origTime = dateFormat.parse( "2006-03-03" ).getTime();
-       targetFile.setLastModified( origTime );
-       targetPomFile.setLastModified( origTime );
+        targetFile.setLastModified( origTime );
+        targetPomFile.setLastModified( origTime );
 
         sourceFile.setLastModified( dateFormat.parse( "2006-01-01" ).getTime() );
         sourcePomFile.setLastModified( dateFormat.parse( "2006-02-02" ).getTime() );
@@ -954,7 +954,7 @@ public class RepositoryConverterTest
             }
             else if ( file.isDirectory() )
             {
-                if ( !file.getName().equals( ".svn" ) )
+                if ( !".svn".equals( file.getName() ) )
                 {
                     if ( !destination.exists() && !destination.mkdirs() )
                     {
index 5f24f98c175d50da4b4427e304f8bbe510a4fb2a..f441c9a46230725d6f079bbc71914fafbd180252 100644 (file)
@@ -12,7 +12,6 @@
   ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   ~ See the License for the specific language governing permissions and
   ~ limitations under the License.
-  ~
   -->
 
 <project>
index 6e56d6956022168a19eab2400557a6b41150a93c..5f347f371a121a167eb88e0803ecd6698903f012 100644 (file)
@@ -12,7 +12,6 @@
   ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   ~ See the License for the specific language governing permissions and
   ~ limitations under the License.
-  ~
   -->
 
 <project>
index 5182dfaf4b8809498f99ac314bb0b8fe2abb1cab..836cc236bafeb093a34b5842ef014b541b3218e0 100644 (file)
@@ -12,7 +12,6 @@
   ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   ~ See the License for the specific language governing permissions and
   ~ limitations under the License.
-  ~
   -->
 
 <project>
index ea8a01354b599b3eb809caa3bd830521f9b15cce..167737f63b1c47ee6e24bb77e77432fcd9f5572c 100644 (file)
@@ -20,9 +20,12 @@ import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.model.Model;
 import org.apache.maven.model.io.xpp3.MavenXpp3Reader;
+import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
 
 import java.io.File;
+import java.io.FileNotFoundException;
 import java.io.FileReader;
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -42,12 +45,12 @@ public abstract class AbstractArtifactDiscoverer
         "**/*.MD5", "**/*.sha1", "**/*.SHA1", "**/*snapshot-version", "*/website/**", "*/licenses/**", "*/licences/**",
         "**/.htaccess", "**/*.html", "**/*.asc", "**/*.txt", "**/*.xml", "**/README*", "**/CHANGELOG*", "**/KEYS*"};
 
-    protected static final String POM = ".pom";
+    private static final String POM = ".pom";
 
     /**
      * Scan the repository for artifact paths.
      */
-    protected String[] scanForArtifactPaths( File repositoryBase, String blacklistedPatterns )
+    private String[] scanForArtifactPaths( File repositoryBase, String blacklistedPatterns )
     {
         return scanForArtifactPaths( repositoryBase, blacklistedPatterns, null, STANDARD_DISCOVERY_EXCLUDES );
     }
@@ -114,7 +117,7 @@ public abstract class AbstractArtifactDiscoverer
                 try
                 {
                     Model model = mavenReader.read( new FileReader( filename ) );
-                    if ( ( pomArtifact != null ) && ( "pom".equals( model.getPackaging() ) ) )
+                    if ( pomArtifact != null && "pom".equals( model.getPackaging() ) )
                     {
                         if ( includeSnapshots || !pomArtifact.isSnapshot() )
                         {
@@ -122,10 +125,19 @@ public abstract class AbstractArtifactDiscoverer
                         }
                     }
                 }
-                catch ( Exception e )
+                catch ( FileNotFoundException e )
                 {
-                    getLogger().info( "error reading file: " + filename );
-                    e.printStackTrace();
+                    // this should never happen
+                    getLogger().error( "Error finding file during POM discovery: " + filename, e );
+                }
+                catch ( IOException e )
+                {
+                    getLogger().error( "Error reading file during POM discovery: " + filename + ": " + e );
+                }
+                catch ( XmlPullParserException e )
+                {
+                    getLogger().error(
+                        "Parse error reading file during POM discovery: " + filename + ": " + e.getMessage() );
                 }
             }
         }
index 9b62af1a25f8360a36e35dc91b5ad7f330ce6c50..d2efc3a58a70e317b4944c27d7db5e3c53906a9c 100644 (file)
@@ -1,5 +1,21 @@
 package org.apache.maven.repository.discovery;
 
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 import org.apache.maven.artifact.factory.ArtifactFactory;
 import org.codehaus.plexus.logging.AbstractLogEnabled;
 import org.codehaus.plexus.util.DirectoryScanner;
@@ -16,7 +32,7 @@ import java.util.List;
  *
  * @author <a href="mailto:brett@apache.org">Brett Porter</a>
  */
-public class AbstractDiscoverer
+public abstract class AbstractDiscoverer
     extends AbstractLogEnabled
 {
     private List kickedOutPaths = new ArrayList();
index 21d56acf14d55ada2c17563439b0028333125da0..20f9d3d87184ffc175f8f2107712a8c97e20ee17 100644 (file)
@@ -129,7 +129,7 @@ public class DefaultMetadataDiscoverer
             //}
 
             Artifact artifact = null;
-            if ( metaVersion != null && !metaVersion.equals( "" ) )
+            if ( metaVersion != null && !"".equals( metaVersion ) )
             {
                 artifact = artifactFactory.createBuildArtifact( metaGroupId, metaArtifactId, metaVersion, "jar" );
             }
index 08b1a7ba9dd502717be80a054fb8be93a45092d8..32f514a05385560fc3e1d0abfc53f9bcb41a35e9 100644 (file)
@@ -54,10 +54,8 @@ public abstract class AbstractRepositoryIndex
      *
      * @param indexPath
      * @param repository
-     * @throws RepositoryIndexException
      */
     protected AbstractRepositoryIndex( String indexPath, ArtifactRepository repository )
-        throws RepositoryIndexException
     {
         this.repository = repository;
         this.indexPath = indexPath;
@@ -237,11 +235,10 @@ public abstract class AbstractRepositoryIndex
      * Check if the index already exists.
      *
      * @return true if the index already exists
-     * @throws IOException
      * @throws RepositoryIndexException
      */
     protected boolean indexExists()
-        throws IOException, RepositoryIndexException
+        throws RepositoryIndexException
     {
         File indexDir = new File( indexPath );
 
@@ -332,7 +329,7 @@ public abstract class AbstractRepositoryIndex
         return isAdded;
     }
 
-    private class ArtifactRepositoryIndexAnalyzer
+    private static class ArtifactRepositoryIndexAnalyzer
         extends Analyzer
     {
         private Analyzer defaultAnalyzer;
@@ -342,7 +339,7 @@ public abstract class AbstractRepositoryIndex
          *
          * @param defaultAnalyzer the analyzer to use as default for the general fields of the artifact indeces
          */
-        public ArtifactRepositoryIndexAnalyzer( Analyzer defaultAnalyzer )
+        ArtifactRepositoryIndexAnalyzer( Analyzer defaultAnalyzer )
         {
             this.defaultAnalyzer = defaultAnalyzer;
         }
@@ -369,33 +366,33 @@ public abstract class AbstractRepositoryIndex
 
             return tokenStream;
         }
+    }
 
+    /**
+     * Class used to tokenize an artifact's version.
+     */
+    private static class VersionTokenizer
+        extends CharTokenizer
+    {
         /**
-         * Class used to tokenize an artifact's version.
+         * Constructor with the required reader to the index stream
+         *
+         * @param reader the Reader object of the index stream
          */
-        private class VersionTokenizer
-            extends CharTokenizer
+        VersionTokenizer( Reader reader )
         {
-            /**
-             * Constructor with the required reader to the index stream
-             *
-             * @param reader the Reader object of the index stream
-             */
-            VersionTokenizer( Reader reader )
-            {
-                super( reader );
-            }
+            super( reader );
+        }
 
-            /**
-             * method that lucene calls to check tokenization of a stream character
-             *
-             * @param character char currently being processed
-             * @return true if the char is a token, false if the char is a stop char
-             */
-            protected boolean isTokenChar( char character )
-            {
-                return character != '.' && character != '-';
-            }
+        /**
+         * method that lucene calls to check tokenization of a stream character
+         *
+         * @param character char currently being processed
+         * @return true if the char is a token, false if the char is a stop char
+         */
+        protected boolean isTokenChar( char character )
+        {
+            return character != '.' && character != '-';
         }
     }
 }
index 23c9a66bbcbcf75cc43b3fd1d0abf9dadca2e5a3..bb410b81561e254a10586bb05866fbeb1066f610 100644 (file)
@@ -47,10 +47,8 @@ public class ArtifactRepositoryIndex
      * @param indexPath  the path where the lucene index will be created/updated.
      * @param repository the repository where the indexed artifacts are located
      * @param digester   the digester object to generate the checksum strings
-     * @throws RepositoryIndexException
      */
     public ArtifactRepositoryIndex( String indexPath, ArtifactRepository repository, Digester digester )
-        throws RepositoryIndexException
     {
         super( indexPath, repository );
         this.digester = digester;
@@ -179,12 +177,9 @@ public class ArtifactRepositoryIndex
      *
      * @param name     the complete path name of the class
      * @param packages the packages buffer
-     * @return true if the package is successfully added
      */
-    private boolean addClassPackage( String name, StringBuffer packages )
+    private void addClassPackage( String name, StringBuffer packages )
     {
-        boolean isAdded = false;
-
         int idx = name.lastIndexOf( '/' );
         if ( idx > 0 )
         {
@@ -193,10 +188,7 @@ public class ArtifactRepositoryIndex
             {
                 packages.append( packageName ).append( "\n" );
             }
-            isAdded = true;
         }
-
-        return isAdded;
     }
 
     /**
@@ -204,9 +196,8 @@ public class ArtifactRepositoryIndex
      *
      * @param entry the zip entry to be added
      * @param files the buffer of files to update
-     * @return true if the file was successfully added
      */
-    private boolean addFile( ZipEntry entry, StringBuffer files )
+    private void addFile( ZipEntry entry, StringBuffer files )
     {
         String name = entry.getName();
         int idx = name.lastIndexOf( '/' );
@@ -215,14 +206,9 @@ public class ArtifactRepositoryIndex
             name = name.substring( idx + 1 );
         }
 
-        boolean isAdded = false;
-
         if ( files.indexOf( name + "\n" ) < 0 )
         {
             files.append( name ).append( "\n" );
-            isAdded = true;
         }
-
-        return isAdded;
     }
 }
index 56ae555e96e12418808f563a16033f58ab9d2391..2b7db9e976259ceb73397e08c1d597247ce7ea0e 100644 (file)
@@ -103,6 +103,10 @@ public class DefaultRepositoryIndexSearcher
             Hits hits = searcher.search( luceneQuery );\r
             docs = buildList( hits );\r
         }\r
+        catch ( MalformedURLException e )\r
+        {\r
+            throw new RepositoryIndexSearchException( "Unable to search index: " + e.getMessage(), e );\r
+        }\r
         catch ( IOException e )\r
         {\r
             throw new RepositoryIndexSearchException( "Unable to search index: " + e.getMessage(), e );\r
@@ -168,17 +172,15 @@ public class DefaultRepositoryIndexSearcher
     protected RepositoryIndexSearchHit createSearchedObjectFromIndexDocument( Document doc )\r
         throws MalformedURLException, IOException, XmlPullParserException\r
     {\r
-        String groupId, artifactId, version, name, packaging;\r
         RepositoryIndexSearchHit searchHit = null;\r
 \r
         // the document is of type artifact\r
         if ( doc.get( RepositoryIndex.FLD_DOCTYPE ).equals( RepositoryIndex.ARTIFACT ) )\r
         {\r
-            groupId = doc.get( RepositoryIndex.FLD_GROUPID );\r
-            artifactId = doc.get( RepositoryIndex.FLD_ARTIFACTID );\r
-            version = doc.get( RepositoryIndex.FLD_VERSION );\r
-            name = doc.get( RepositoryIndex.FLD_NAME );\r
-            packaging = doc.get( RepositoryIndex.FLD_PACKAGING );\r
+            String groupId = doc.get( RepositoryIndex.FLD_GROUPID );\r
+            String artifactId = doc.get( RepositoryIndex.FLD_ARTIFACTID );\r
+            String version = doc.get( RepositoryIndex.FLD_VERSION );\r
+            String packaging = doc.get( RepositoryIndex.FLD_PACKAGING );\r
             Artifact artifact = factory.createBuildArtifact( groupId, artifactId, version, packaging );\r
 \r
             artifact.setFile(\r
@@ -224,7 +226,7 @@ public class DefaultRepositoryIndexSearcher
             String metadataFile = (String) it.next();\r
             String tmpDir = (String) it.next();\r
 \r
-            String metadataType = "";\r
+            String metadataType;\r
             if ( tmpDir.equals( doc.get( RepositoryIndex.FLD_VERSION ) ) )\r
             {\r
                 metadataType = MetadataRepositoryIndex.SNAPSHOT_METADATA;\r
@@ -258,16 +260,17 @@ public class DefaultRepositoryIndexSearcher
      * @param filename     the name of the metadata file\r
      * @param metadataType the type of RepositoryMetadata object to be created (GROUP, ARTIFACT or SNAPSHOT)\r
      * @return RepositoryMetadata\r
-     * @throws MalformedURLException\r
      * @throws IOException\r
      * @throws XmlPullParserException\r
      */\r
     private RepositoryMetadata getMetadata( String groupId, String artifactId, String version, String filename,\r
                                             String metadataType )\r
-        throws MalformedURLException, IOException, XmlPullParserException\r
+        throws IOException, XmlPullParserException\r
     {\r
         RepositoryMetadata repoMetadata = null;\r
-        InputStream is = null;\r
+\r
+        // TODO! file handles left open\r
+        InputStream is;\r
         MetadataXpp3Reader metadataReader = new MetadataXpp3Reader();\r
 \r
         //group metadata\r
index da25294aa6c44573a6e2b14a3ba038ea067bc495..8d38b715a02cca6724a13f6a3f16828745ff3227 100644 (file)
@@ -1,14 +1,13 @@
 package org.apache.maven.repository.indexing;
 
 /*
- * Copyright 2001-2005 The Apache Software Foundation.
+ * Copyright 2005-2006 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,
index f96c3032eae4a84dade2d1ac2252765ee8214d15..c54ea9f3cfff398c6530275365f475ef8d744b64 100644 (file)
@@ -67,10 +67,8 @@ public class EclipseRepositoryIndex
      * @param indexPath  the path where the lucene index will be created/updated.
      * @param repository the repository where the indexed artifacts are located
      * @param digester   the digester object to generate the checksum strings
-     * @throws RepositoryIndexException
      */
     public EclipseRepositoryIndex( String indexPath, ArtifactRepository repository, Digester digester )
-        throws RepositoryIndexException
     {
         super( indexPath, repository );
 
index e412f6049fae349b73a5103522665a6ee759d474..ab39819350cdafd524d98901fc9f235f9caf4817 100644 (file)
@@ -45,10 +45,8 @@ public class MetadataRepositoryIndex
      *\r
      * @param indexPath  the path to the index\r
      * @param repository the repository where the metadata to be indexed is located\r
-     * @throws RepositoryIndexException\r
      */\r
     public MetadataRepositoryIndex( String indexPath, ArtifactRepository repository )\r
-        throws RepositoryIndexException\r
     {\r
         super( indexPath, repository );\r
     }\r
@@ -105,7 +103,7 @@ public class MetadataRepositoryIndex
                 repoMetadata.getBaseVersion() + "/";\r
         }\r
 \r
-        if ( !repoMetadata.getRemoteFilename().equals( "" ) && repoMetadata.getRemoteFilename() != null )\r
+        if ( !"".equals( repoMetadata.getRemoteFilename() ) && repoMetadata.getRemoteFilename() != null )\r
         {\r
             path = path + repoMetadata.getRemoteFilename();\r
         }\r
@@ -130,7 +128,7 @@ public class MetadataRepositoryIndex
         for ( Iterator iter = plugins.iterator(); iter.hasNext(); )\r
         {\r
             Plugin plugin = (Plugin) iter.next();\r
-            if ( plugin.getPrefix() != null && !plugin.getPrefix().equals( "" ) )\r
+            if ( plugin.getPrefix() != null && !"".equals( plugin.getPrefix() ) )\r
             {\r
                 pluginAppended = plugin.getPrefix() + "\n";\r
             }\r
@@ -138,7 +136,7 @@ public class MetadataRepositoryIndex
         doc.add( Field.Text( FLD_PLUGINPREFIX, pluginAppended ) );\r
         doc.add( Field.Text( FLD_GROUPID, metadata.getGroupId() ) );\r
 \r
-        if ( metadata.getArtifactId() != null && !metadata.getArtifactId().equals( "" ) )\r
+        if ( metadata.getArtifactId() != null && !"".equals( metadata.getArtifactId() ) )\r
         {\r
             doc.add( Field.Text( FLD_ARTIFACTID, metadata.getArtifactId() ) );\r
         }\r
@@ -147,7 +145,7 @@ public class MetadataRepositoryIndex
             doc.add( Field.Text( FLD_ARTIFACTID, "" ) );\r
         }\r
 \r
-        if ( metadata.getVersion() != null && !metadata.getVersion().equals( "" ) )\r
+        if ( metadata.getVersion() != null && !"".equals( metadata.getVersion() ) )\r
         {\r
             doc.add( Field.Text( FLD_VERSION, metadata.getVersion() ) );\r
         }\r
index 9fe71b568ecd0065ae4b3b696cbd62a026a998f8..cca1e72e6545665553fd0a4c4b5646f2ecdff9fe 100644 (file)
@@ -1,14 +1,13 @@
 package org.apache.maven.repository.indexing;
 
 /*
- * Copyright 2001-2005 The Apache Software Foundation.
+ * Copyright 2005-2006 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,
@@ -31,6 +30,7 @@ import org.apache.maven.repository.digest.Digester;
 import org.codehaus.plexus.util.StringUtils;
 
 import java.io.File;
+import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.security.NoSuchAlgorithmException;
 import java.util.Iterator;
@@ -55,11 +55,9 @@ public class PomRepositoryIndex
      * @param repository      the repository where objects indexed by this class resides
      * @param digester        the digester to be used for generating checksums
      * @param artifactFactory the factory for building artifact objects
-     * @throws RepositoryIndexException
      */
     public PomRepositoryIndex( String indexPath, ArtifactRepository repository, Digester digester,
                                ArtifactFactory artifactFactory )
-        throws RepositoryIndexException
     {
         super( indexPath, repository );
         this.digester = digester;
@@ -266,6 +264,10 @@ public class PomRepositoryIndex
         {
             return digester.createChecksum( new File( file ), algorithm );
         }
+        catch ( FileNotFoundException e )
+        {
+            throw new RepositoryIndexException( e.getMessage(), e );
+        }
         catch ( IOException e )
         {
             throw new RepositoryIndexException( e.getMessage(), e );
index fd88be533734857edb0c0761453e5ba272f7b83e..52064e9d77e3793a821ba22a769ce58c5dff4133 100644 (file)
@@ -27,59 +27,58 @@ import java.util.List;
  */
 public interface RepositoryIndex
 {
-    static final String POM = "POM";
+    String POM = "POM";
 
-    static final String METADATA = "METADATA";
+    String METADATA = "METADATA";
 
-    static final String ARTIFACT = "ARTIFACT";
+    String ARTIFACT = "ARTIFACT";
 
-    static final String FLD_ID = "id";
+    String FLD_ID = "id";
 
-    static final String FLD_NAME = "name";
+    String FLD_NAME = "name";
 
-    static final String FLD_DOCTYPE = "doctype";
+    String FLD_DOCTYPE = "doctype";
 
-    static final String FLD_GROUPID = "groupId";
+    String FLD_GROUPID = "groupId";
 
-    static final String FLD_ARTIFACTID = "artifactId";
+    String FLD_ARTIFACTID = "artifactId";
 
-    static final String FLD_VERSION = "version";
+    String FLD_VERSION = "version";
 
-    static final String FLD_PACKAGING = "packaging";
+    String FLD_PACKAGING = "packaging";
 
-    static final String FLD_SHA1 = "sha1";
+    String FLD_SHA1 = "sha1";
 
-    static final String FLD_MD5 = "md5";
+    String FLD_MD5 = "md5";
 
-    static final String FLD_LASTUPDATE = "last update";
+    String FLD_LASTUPDATE = "last update";
 
-    static final String FLD_PLUGINPREFIX = "plugin prefix";
+    String FLD_PLUGINPREFIX = "plugin prefix";
 
-    static final String FLD_CLASSES = "class";
+    String FLD_CLASSES = "class";
 
-    static final String FLD_PACKAGES = "package";
+    String FLD_PACKAGES = "package";
 
-    static final String FLD_FILES = "file";
+    String FLD_FILES = "file";
 
-    static final String FLD_LICENSE_URLS = "license url";
+    String FLD_LICENSE_URLS = "license url";
 
-    static final String FLD_DEPENDENCIES = "dependency";
+    String FLD_DEPENDENCIES = "dependency";
 
-    static final String FLD_PLUGINS_BUILD = "build plugin";
+    String FLD_PLUGINS_BUILD = "build plugin";
 
-    static final String FLD_PLUGINS_REPORT = "report plugin";
+    String FLD_PLUGINS_REPORT = "report plugin";
 
-    static final String FLD_PLUGINS_ALL = "plugins_all";
+    String FLD_PLUGINS_ALL = "plugins_all";
 
-    static final String[] FIELDS = {FLD_ID, FLD_NAME, FLD_DOCTYPE, FLD_GROUPID, FLD_ARTIFACTID, FLD_VERSION,
-        FLD_PACKAGING, FLD_SHA1, FLD_MD5, FLD_LASTUPDATE, FLD_PLUGINPREFIX, FLD_CLASSES, FLD_PACKAGES, FLD_FILES,
-        FLD_LICENSE_URLS, FLD_DEPENDENCIES, FLD_PLUGINS_BUILD, FLD_PLUGINS_REPORT, FLD_PLUGINS_ALL};
+    String[] FIELDS = {FLD_ID, FLD_NAME, FLD_DOCTYPE, FLD_GROUPID, FLD_ARTIFACTID, FLD_VERSION, FLD_PACKAGING, FLD_SHA1,
+        FLD_MD5, FLD_LASTUPDATE, FLD_PLUGINPREFIX, FLD_CLASSES, FLD_PACKAGES, FLD_FILES, FLD_LICENSE_URLS,
+        FLD_DEPENDENCIES, FLD_PLUGINS_BUILD, FLD_PLUGINS_REPORT, FLD_PLUGINS_ALL};
 
-    static final List KEYWORD_FIELDS = Arrays.asList( new String[]{FLD_ID, FLD_PACKAGING, FLD_LICENSE_URLS,
-        FLD_DEPENDENCIES, FLD_PLUGINS_BUILD, FLD_PLUGINS_REPORT, FLD_PLUGINS_ALL} );
+    List KEYWORD_FIELDS = Arrays.asList( new String[]{FLD_ID, FLD_PACKAGING, FLD_LICENSE_URLS, FLD_DEPENDENCIES,
+        FLD_PLUGINS_BUILD, FLD_PLUGINS_REPORT, FLD_PLUGINS_ALL} );
 
-    static final String[] MODEL_FIELDS =
-        {FLD_PACKAGING, FLD_LICENSE_URLS, FLD_DEPENDENCIES, FLD_PLUGINS_BUILD, FLD_PLUGINS_REPORT};
+    String[] MODEL_FIELDS = {FLD_PACKAGING, FLD_LICENSE_URLS, FLD_DEPENDENCIES, FLD_PLUGINS_BUILD, FLD_PLUGINS_REPORT};
 
     /**
      * Method used to query the index status
index f28aa188579d9894d0cb6a7e6358094f61eef291..c1b00f34659148cbf27885353418a5ccb806ac1c 100644 (file)
@@ -23,11 +23,11 @@ public class RepositoryIndexSearchHit
 {\r
     private Object obj;\r
 \r
-    private boolean isHashMap = false;\r
+    private boolean isHashMap;\r
 \r
-    private boolean isMetadata = false;\r
+    private boolean isMetadata;\r
 \r
-    private boolean isModel = false;\r
+    private boolean isModel;\r
 \r
     /**\r
      * Class constructor\r
index 4bb6334480fd1f32d3b018940449ef3549c259b4..eccd574b0e84bb0ec3ccef710a77758d48345d10 100644 (file)
@@ -18,7 +18,6 @@ package org.apache.maven.repository.indexing;
 \r
 import org.apache.maven.artifact.Artifact;\r
 import org.apache.maven.artifact.factory.ArtifactFactory;\r
-import org.apache.maven.artifact.repository.metadata.RepositoryMetadata;\r
 import org.apache.maven.model.Dependency;\r
 import org.apache.maven.model.License;\r
 import org.apache.maven.model.Model;\r
@@ -144,7 +143,7 @@ public class RepositoryIndexSearchLayer
             else if ( hit.isMetadata() )\r
             {\r
                 //@todo what about metadata objects?\r
-                RepositoryMetadata metadata = (RepositoryMetadata) hit.getObject();\r
+//                RepositoryMetadata metadata = (RepositoryMetadata) hit.getObject();\r
             }\r
         }\r
 \r
@@ -212,7 +211,7 @@ public class RepositoryIndexSearchLayer
             }\r
         }\r
 \r
-        if ( values != null && values.size() > 0 )\r
+        if ( !values.isEmpty() )\r
         {\r
             resultMap.put( key, values );\r
         }\r
@@ -351,7 +350,7 @@ public class RepositoryIndexSearchLayer
             }\r
         }\r
 \r
-        if ( values.size() > 0 && values != null )\r
+        if ( !values.isEmpty() )\r
         {\r
             map.put( field, values );\r
         }\r
index a1b3981cccf2ae10804ad4b10924e6489da689ac..f1e16b0747f81c80b5e0af4ad09882caba806438 100644 (file)
@@ -1,14 +1,13 @@
 package org.apache.maven.repository.indexing;
 
 /*
- * Copyright 2001-2005 The Apache Software Foundation.
+ * Copyright 2005-2006 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,
index e86e3ec156440de4d4435897be9f0d1ac965dd39..515524121c0a415c4a31a26af6f16ac77c8d9924 100644 (file)
@@ -1,17 +1,13 @@
 package org.apache.maven.repository.indexing.query;
 
-import org.apache.lucene.queryParser.ParseException;
-import org.apache.maven.repository.indexing.RepositoryIndex;
-
 /*
- * Copyright 2001-2005 The Apache Software Foundation.
+ * Copyright 2005-2006 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,
@@ -20,6 +16,9 @@ import org.apache.maven.repository.indexing.RepositoryIndex;
  * limitations under the License.
  */
 
+import org.apache.lucene.queryParser.ParseException;
+import org.apache.maven.repository.indexing.RepositoryIndex;
+
 /**
  * Interface to label the query classes
  *
index d41ca82fb6d362cb8eedb06afa5291bcf0ae6740..34bd7fc70dfe2bbc3062c52ddc2eb7c5dbe94f4f 100644 (file)
@@ -1,20 +1,13 @@
 package org.apache.maven.repository.indexing.query;
 
-import org.apache.lucene.index.Term;
-import org.apache.lucene.queryParser.ParseException;
-import org.apache.lucene.queryParser.QueryParser;
-import org.apache.lucene.search.TermQuery;
-import org.apache.maven.repository.indexing.RepositoryIndex;
-
 /*
- * Copyright 2001-2005 The Apache Software Foundation.
+ * Copyright 2005-2006 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,
@@ -23,6 +16,12 @@ import org.apache.maven.repository.indexing.RepositoryIndex;
  * limitations under the License.
  */
 
+import org.apache.lucene.index.Term;
+import org.apache.lucene.queryParser.ParseException;
+import org.apache.lucene.queryParser.QueryParser;
+import org.apache.lucene.search.TermQuery;
+import org.apache.maven.repository.indexing.RepositoryIndex;
+
 /**
  * Class to hold a single field search condition
  *
index c38bc6bcfd86f9b28718d263ce126642b778c5c6..403232be8aea9308479af2436321d211f599bf07 100644 (file)
@@ -422,7 +422,7 @@ public class ArtifactRepositoryIndexingTest
         try
         {
             Query qry = new SinglePhraseQuery( RepositoryIndex.FLD_VERSION, "~~~~~" );
-            List artifacts = repoSearchLayer.searchAdvanced( qry );
+            repoSearchLayer.searchAdvanced( qry );
             fail( "Must throw an exception on unparseable query." );
         }
         catch ( RepositoryIndexSearchException re )
@@ -436,7 +436,7 @@ public class ArtifactRepositoryIndexingTest
         try
         {
             Query qry = new SinglePhraseQuery( RepositoryIndex.FLD_VERSION, "1.0" );
-            List artifacts = repoSearchLayer.searchAdvanced( qry );
+            repoSearchLayer.searchAdvanced( qry );
             fail( "Must throw an exception on invalid index location." );
         }
         catch ( RepositoryIndexSearchException re )
@@ -466,7 +466,7 @@ public class ArtifactRepositoryIndexingTest
         RepositoryIndexSearcher repoSearcher = factory.createDefaultRepositoryIndexSearcher( indexer );
         Query qry = new SinglePhraseQuery( RepositoryIndex.FLD_ID, RepositoryIndex.ARTIFACT + artifact.getId() );
         List artifacts = repoSearcher.search( qry );
-        assertEquals( artifacts.size(), 0 );
+        assertEquals( 0, artifacts.size() );
     }
 
     /**
index 132e1f450bfbaa6fe8d7aa4ec9f9b60dab319a74..39670273fecc7d2bac9a33022d777ec1846831f4 100644 (file)
@@ -1,5 +1,21 @@
 package org.apache.maven.repository.indexing;
 
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 import org.apache.lucene.document.DateField;
 import org.apache.lucene.document.Document;
 import org.apache.lucene.queryParser.QueryParser;
@@ -17,22 +33,6 @@ import org.codehaus.plexus.util.FileUtils;
 
 import java.io.File;
 
-/*
- * Copyright 2005-2006 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.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
 /**
  * @author Edwin Punzalan
  */
@@ -49,6 +49,8 @@ public class EclipseRepositoryIndexTest
 
     private long artifactFileTime;
 
+    private static final long TIME_DIFFERENCE = 10000L;
+
     protected void setUp()
         throws Exception
     {
@@ -84,7 +86,7 @@ public class EclipseRepositoryIndexTest
         indexer.optimize();
         indexer.close();
 
-        long historicTime = artifactFileTime - 10000L;
+        long historicTime = artifactFileTime - TIME_DIFFERENCE;
 
         artifact = getArtifact( "org.apache.maven", "maven-model", "2.0" );
         artifact.setFile( new File( repository.getBasedir(), repository.pathOf( artifact ) ) );
index 80f984fe501f59c1c433c043dc98ee55e549c466..bfc4a54bf6037dd445675db72fe0896920730188 100644 (file)
@@ -52,8 +52,6 @@ public class MetadataRepositoryIndexingTest
 \r
     private String indexPath;\r
 \r
-    private MetadataRepositoryIndex indexer;\r
-\r
     private ArtifactFactory artifactFactory;\r
 \r
     /**\r
@@ -98,7 +96,7 @@ public class MetadataRepositoryIndexingTest
         throws Exception\r
     {\r
         RepositoryIndexingFactory factory = (RepositoryIndexingFactory) lookup( RepositoryIndexingFactory.ROLE );\r
-        indexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
+        MetadataRepositoryIndex indexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
 \r
         RepositoryMetadata repoMetadata =\r
             getMetadata( "org.apache.maven", null, null, "maven-metadata.xml", MetadataRepositoryIndex.GROUP_METADATA );\r
@@ -140,8 +138,7 @@ public class MetadataRepositoryIndexingTest
         RepositoryIndexSearchLayer repoSearchLayer = factory.createRepositoryIndexSearchLayer( indexer );\r
 \r
         // search last update\r
-        org.apache.maven.repository.indexing.query.Query qry =\r
-            new SinglePhraseQuery( RepositoryIndex.FLD_LASTUPDATE, "20051212044643" );\r
+        Query qry = new SinglePhraseQuery( RepositoryIndex.FLD_LASTUPDATE, "20051212044643" );\r
         List metadataList = repoSearchLayer.searchAdvanced( qry );\r
         //assertEquals( 1, metadataList.size() );\r
         for ( Iterator iter = metadataList.iterator(); iter.hasNext(); )\r
@@ -204,7 +201,7 @@ public class MetadataRepositoryIndexingTest
         rQry.addQuery( qry2 );\r
 \r
         metadataList = repoSearchLayer.searchAdvanced( rQry );\r
-        assertEquals( metadataList.size(), 0 );\r
+        assertEquals( 0, metadataList.size() );\r
 \r
         indexer.close();\r
     }\r
@@ -219,7 +216,7 @@ public class MetadataRepositoryIndexingTest
     {\r
         //test when the object passed in the index(..) method is not a RepositoryMetadata instance\r
         RepositoryIndexingFactory factory = (RepositoryIndexingFactory) lookup( RepositoryIndexingFactory.ROLE );\r
-        indexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
+        MetadataRepositoryIndex indexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
         try\r
         {\r
             Artifact artifact = getArtifact( "org.apache.maven", "maven-artifact", "2.0.1" );\r
@@ -255,17 +252,16 @@ public class MetadataRepositoryIndexingTest
         createTestIndex();\r
 \r
         RepositoryIndexingFactory factory = (RepositoryIndexingFactory) lookup( RepositoryIndexingFactory.ROLE );\r
-        indexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
+        MetadataRepositoryIndex indexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
 \r
         RepositoryMetadata repoMetadata =\r
             getMetadata( "org.apache.maven", null, null, "maven-metadata.xml", MetadataRepositoryIndex.GROUP_METADATA );\r
         indexer.deleteDocument( RepositoryIndex.FLD_ID, (String) repoMetadata.getKey() );\r
 \r
         RepositoryIndexSearcher repoSearcher = factory.createDefaultRepositoryIndexSearcher( indexer );\r
-        org.apache.maven.repository.indexing.query.Query qry =\r
-            new SinglePhraseQuery( RepositoryIndex.FLD_ID, (String) repoMetadata.getKey() );\r
+        Query qry = new SinglePhraseQuery( RepositoryIndex.FLD_ID, (String) repoMetadata.getKey() );\r
         List metadataList = repoSearcher.search( qry );\r
-        assertEquals( metadataList.size(), 0 );\r
+        assertEquals( 0, metadataList.size() );\r
     }\r
 \r
     /**\r
@@ -285,7 +281,7 @@ public class MetadataRepositoryIndexingTest
     {\r
         RepositoryMetadata repoMetadata = null;\r
         URL url;\r
-        InputStream is = null;\r
+        InputStream is;\r
         MetadataXpp3Reader metadataReader = new MetadataXpp3Reader();\r
 \r
         //group metadata\r
index ee00f73f39e40aa20d813ed166ee3a09a22b9978..4300be481b2f7c105c1ca65b2a484fad1c6f2a04 100644 (file)
@@ -1,6 +1,6 @@
 package org.apache.maven.repository.indexing;
 
-/**
+/*
  * Copyright 2005-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -180,10 +180,6 @@ public class PomRepositoryIndexingTest
             SearchResult result = (SearchResult) artifacts.next();
             Map map = result.getFieldMatches();
             Set mapEntry = map.entrySet();
-            for ( Iterator it = mapEntry.iterator(); it.hasNext(); )
-            {
-                Map.Entry entry = (Map.Entry) it.next();
-            }
             assertEquals( "jar", (String) map.get( RepositoryIndex.FLD_PACKAGING ) );
         }
 
@@ -212,7 +208,8 @@ public class PomRepositoryIndexingTest
             SearchResult result = (SearchResult) artifacts.next();
             Map map = result.getFieldMatches();
             boolean depFound = false;
-            Iterator dependencies = ( (List) map.get( RepositoryIndex.FLD_DEPENDENCIES ) ).iterator();
+            List list = (List) map.get( RepositoryIndex.FLD_DEPENDENCIES );
+            Iterator dependencies = list.iterator();
             while ( dependencies.hasNext() )
             {
                 String dep = (String) dependencies.next();
@@ -234,7 +231,8 @@ public class PomRepositoryIndexingTest
         {
             SearchResult result = (SearchResult) artifacts.next();
             Map map = result.getFieldMatches();
-            Iterator plugins = ( (List) map.get( RepositoryIndex.FLD_PLUGINS_BUILD ) ).iterator();
+            List list = (List) map.get( RepositoryIndex.FLD_PLUGINS_BUILD );
+            Iterator plugins = list.iterator();
             boolean found = false;
             while ( plugins.hasNext() )
             {
@@ -257,7 +255,8 @@ public class PomRepositoryIndexingTest
         {
             SearchResult result = (SearchResult) artifacts.next();
             Map map = result.getFieldMatches();
-            Iterator plugins = ( (List) map.get( RepositoryIndex.FLD_PLUGINS_REPORT ) ).iterator();
+            List list = (List) map.get( RepositoryIndex.FLD_PLUGINS_REPORT );
+            Iterator plugins = list.iterator();
             boolean found = false;
             while ( plugins.hasNext() )
             {
@@ -485,7 +484,7 @@ public class PomRepositoryIndexingTest
         RepositoryIndexSearcher repoSearcher = factory.createDefaultRepositoryIndexSearcher( indexer );
         Query qry = new SinglePhraseQuery( RepositoryIndex.FLD_ID, RepositoryIndex.POM + pom.getId() );
         List artifactList = repoSearcher.search( qry );
-        assertEquals( artifactList.size(), 0 );
+        assertEquals( 0, artifactList.size() );
     }
 
     private Model getPom( String groupId, String artifactId, String version )
index 9121485f17ba979038d59edc11c2a7b1eb515210..804165bd3ebcdbca427877d38c7a6c70c86486ad 100644 (file)
@@ -28,8 +28,6 @@ import org.apache.maven.artifact.repository.metadata.SnapshotArtifactRepositoryM
 import org.apache.maven.artifact.repository.metadata.io.xpp3.MetadataXpp3Reader;\r
 import org.apache.maven.model.Model;\r
 import org.apache.maven.model.io.xpp3.MavenXpp3Reader;\r
-import org.apache.maven.repository.digest.DefaultDigester;\r
-import org.apache.maven.repository.digest.Digester;\r
 import org.codehaus.plexus.PlexusTestCase;\r
 import org.codehaus.plexus.util.FileUtils;\r
 \r
@@ -54,8 +52,6 @@ public class RepositoryIndexSearchLayerTest
 \r
     private ArtifactFactory artifactFactory;\r
 \r
-    private Digester digester;\r
-\r
     private String indexPath;\r
 \r
     /**\r
@@ -72,23 +68,11 @@ public class RepositoryIndexSearchLayerTest
         ArtifactRepositoryLayout layout = (ArtifactRepositoryLayout) lookup( ArtifactRepositoryLayout.ROLE, "default" );\r
         ArtifactRepositoryFactory repoFactory = (ArtifactRepositoryFactory) lookup( ArtifactRepositoryFactory.ROLE );\r
         repository = repoFactory.createArtifactRepository( "test", repoDir, layout, null, null );\r
-        digester = new DefaultDigester();\r
 \r
         indexPath = "target/index";\r
         FileUtils.deleteDirectory( indexPath );\r
     }\r
 \r
-    /**\r
-     * Tear down method\r
-     *\r
-     * @throws Exception\r
-     */\r
-    protected void tearDown()\r
-        throws Exception\r
-    {\r
-        super.tearDown();\r
-    }\r
-\r
     /**\r
      * Method for creating the index used for testing\r
      *\r
@@ -126,24 +110,24 @@ public class RepositoryIndexSearchLayerTest
 \r
         MetadataRepositoryIndex metaIndexer = factory.createMetadataRepositoryIndex( indexPath, repository );\r
         RepositoryMetadata repoMetadata =\r
-            getMetadata( "org.apache.maven", null, null, "maven-metadata.xml", metaIndexer.GROUP_METADATA );\r
+            getMetadata( "org.apache.maven", null, null, "maven-metadata.xml", MetadataRepositoryIndex.GROUP_METADATA );\r
         metaIndexer.index( repoMetadata );\r
         metaIndexer.optimize();\r
         metaIndexer.close();\r
 \r
         repoMetadata = getMetadata( "org.apache.maven", "maven-artifact", "2.0.1", "maven-metadata.xml",\r
-                                    metaIndexer.ARTIFACT_METADATA );\r
+                                    MetadataRepositoryIndex.ARTIFACT_METADATA );\r
         metaIndexer.index( repoMetadata );\r
         metaIndexer.optimize();\r
         metaIndexer.close();\r
 \r
         repoMetadata = getMetadata( "org.apache.maven", "maven-artifact", "2.0.1", "maven-metadata.xml",\r
-                                    metaIndexer.SNAPSHOT_METADATA );\r
+                                    MetadataRepositoryIndex.SNAPSHOT_METADATA );\r
         metaIndexer.index( repoMetadata );\r
         metaIndexer.optimize();\r
         metaIndexer.close();\r
 \r
-        repoMetadata = getMetadata( "test", null, null, "maven-metadata.xml", metaIndexer.GROUP_METADATA );\r
+        repoMetadata = getMetadata( "test", null, null, "maven-metadata.xml", MetadataRepositoryIndex.GROUP_METADATA );\r
         metaIndexer.index( repoMetadata );\r
         metaIndexer.optimize();\r
         metaIndexer.close();\r
@@ -345,14 +329,14 @@ public class RepositoryIndexSearchLayerTest
         for ( Iterator iter = returnList.iterator(); iter.hasNext(); )\r
         {\r
             SearchResult result = (SearchResult) iter.next();\r
-            assertEquals( result.getArtifact().getGroupId(), "test" );\r
+            assertEquals( "test", result.getArtifact().getGroupId() );\r
         }\r
 \r
         returnList = searchLayer.searchGeneral( "test-artifactId" );\r
         for ( Iterator iter = returnList.iterator(); iter.hasNext(); )\r
         {\r
             SearchResult result = (SearchResult) iter.next();\r
-            assertEquals( result.getArtifact().getArtifactId(), "test-artifactId" );\r
+            assertEquals( "test-artifactId", result.getArtifact().getArtifactId() );\r
         }\r
 \r
     }\r
@@ -375,7 +359,7 @@ public class RepositoryIndexSearchLayerTest
     {\r
         RepositoryMetadata repoMetadata = null;\r
         URL url;\r
-        InputStream is = null;\r
+        InputStream is;\r
         MetadataXpp3Reader metadataReader = new MetadataXpp3Reader();\r
 \r
         //group metadata\r
index 88cae6429d87805702b088c61df48022d1c98261..ef81f4799caa9adf7841d6451ecc4c2469fe7e2d 100644 (file)
@@ -1,16 +1,13 @@
 package org.apache.maven.repository.indexing.query;
 
-import junit.framework.TestCase;
-
 /*
- * Copyright 2001-2005 The Apache Software Foundation.
+ * Copyright 2005-2006 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,
@@ -19,6 +16,8 @@ import junit.framework.TestCase;
  * limitations under the License.
  */
 
+import junit.framework.TestCase;
+
 /**
  * @author Edwin Punzalan
  */
index 429b99289a6f3b8433bef3b8c594ea5bae71b800..debc162f42226824d2109ac1b6b8858742f27cd9 100644 (file)
@@ -1,4 +1,21 @@
-<?xml version="1.0"?><project>
+<?xml version="1.0"?>
+<!--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
+<project>
   <parent>
     <artifactId>maven-repository-manager</artifactId>
     <groupId>org.apache.maven.repository</groupId>
index c3e0b8044be452c1b31983426c91ff232715bda1..34bce211731a8b88bd9a421812028b720fcf027b 100644 (file)
@@ -1,18 +1,34 @@
 <?xml version="1.0" encoding="ISO-8859-1"?>
+<!--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
 <project>
   <body>
     <links>
       <item name="Maven" href="http://maven.apache.org/"/>
     </links>
 
-    <menu ref="reports" />
+    <menu ref="reports"/>
 
   </body>
   <skin>
     <groupId>org.apache.maven.skins</groupId>
     <artifactId>maven-stylus-skin</artifactId>
   </skin>
-  <publishDate format="dd MMM yyyy" />
+  <publishDate format="dd MMM yyyy"/>
   <bannerLeft>
     <name>Maven</name>
     <src>http://maven.apache.org/images/apache-maven-project-2.png</src>
index dda00d569185f0e9e580031360ba4863139d4e8f..fa429c5c6334b9d352a267375da4d0366362a76f 100644 (file)
@@ -89,6 +89,8 @@ public class DefaultProxyManager
      */
     private Map failuresCache = new HashMap();
 
+    private static final int MS_PER_SEC = 1000;
+
     public void setConfiguration( ProxyConfiguration config )
     {
         this.config = config;
@@ -413,7 +415,7 @@ public class DefaultProxyManager
                     }
                     else
                     {
-                        long repoTimestamp = target.lastModified() + repository.getCachePeriod() * 1000;
+                        long repoTimestamp = target.lastModified() + repository.getCachePeriod() * MS_PER_SEC;
                         wagon.getIfNewer( path, temp, repoTimestamp );
                     }
 
index 68e3e2fa89eb0d1959017b47e88126b1eec75e1b..aec5ec0686f7c0280d350f8aa00f19d9771ddf10 100644 (file)
@@ -29,7 +29,7 @@ import java.io.File;
  */
 public interface ProxyManager
 {
-    static String ROLE = ProxyManager.class.getName();
+    String ROLE = ProxyManager.class.getName();
 
     /**
      * Used to retrieve a cached path or retrieve one if the cache does not contain it yet.
index 71c76db8582c0a23d150d610d731f50f019b3ef8..8e336b1c0e888c9c796a2b2c95f9b715baa688f2 100644 (file)
@@ -1,13 +1,13 @@
 package org.apache.maven.repository.proxy.configuration;
 
 /*
- * Copyright 2003-2004 The Apache Software Foundation.
+ * Copyright 2005-2006 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
+ *      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,
@@ -50,70 +50,70 @@ public class MavenProxyPropertyLoader
 
         config.setRepositoryCachePath( getMandatoryProperty( props, REPO_LOCAL_STORE ) );
 
-        {//just get the first proxy and break
-            String propertyList = props.getProperty( PROXY_LIST );
-            if ( propertyList != null )
+        //just get the first proxy and break
+        String propertyList = props.getProperty( PROXY_LIST );
+        if ( propertyList != null )
+        {
+            StringTokenizer tok = new StringTokenizer( propertyList, "," );
+            while ( tok.hasMoreTokens() )
             {
-                StringTokenizer tok = new StringTokenizer( propertyList, "," );
-                while ( tok.hasMoreTokens() )
+                String key = tok.nextToken();
+                if ( StringUtils.isNotEmpty( key ) )
                 {
-                    String key = tok.nextToken();
-                    if ( StringUtils.isNotEmpty( key ) )
+                    String host = getMandatoryProperty( props, "proxy." + key + ".host" );
+                    int port = Integer.parseInt( getMandatoryProperty( props, "proxy." + key + ".port" ) );
+
+                    // the username and password isn't required
+                    String username = props.getProperty( "proxy." + key + ".username" );
+                    String password = props.getProperty( "proxy." + key + ".password" );
+
+                    if ( StringUtils.isNotEmpty( username ) )
                     {
-                        String host = getMandatoryProperty( props, "proxy." + key + ".host" );
-                        int port = Integer.parseInt( getMandatoryProperty( props, "proxy." + key + ".port" ) );
-
-                        // the username and password isn't required
-                        String username = props.getProperty( "proxy." + key + ".username" );
-                        String password = props.getProperty( "proxy." + key + ".password" );
-
-                        if ( StringUtils.isNotEmpty( username ) )
-                        {
-                            config.setHttpProxy( host, port, username, password );
-                        }
-                        else
-                        {
-                            config.setHttpProxy( host, port );
-                        }
-
-                        //accept only one proxy configuration
-                        break;
+                        config.setHttpProxy( host, port, username, password );
                     }
+                    else
+                    {
+                        config.setHttpProxy( host, port );
+                    }
+
+                    //accept only one proxy configuration
+                    break;
                 }
             }
         }
 
         List repositories = new ArrayList();
-        { //get the remote repository list
-            String repoList = getMandatoryProperty( props, REPO_LIST );
 
-            StringTokenizer tok = new StringTokenizer( repoList, "," );
-            while ( tok.hasMoreTokens() )
-            {
-                String key = tok.nextToken();
+        //get the remote repository list
+        String repoList = getMandatoryProperty( props, REPO_LIST );
 
-                Properties repoProps = getSubset( props, "repo." + key + "." );
-                String url = getMandatoryProperty( props, "repo." + key + ".url" );
-                String proxyKey = repoProps.getProperty( "proxy" );
+        StringTokenizer tok = new StringTokenizer( repoList, "," );
+        while ( tok.hasMoreTokens() )
+        {
+            String key = tok.nextToken();
 
-                boolean cacheFailures =
-                    Boolean.valueOf( repoProps.getProperty( "cache.failures", "false" ) ).booleanValue();
-                boolean hardFail = Boolean.valueOf( repoProps.getProperty( "hardfail", "true" ) ).booleanValue();
-                long cachePeriod = Long.parseLong( repoProps.getProperty( "cache.period", "0" ) );
+            Properties repoProps = getSubset( props, "repo." + key + "." );
+            String url = getMandatoryProperty( props, "repo." + key + ".url" );
+            String proxyKey = repoProps.getProperty( "proxy" );
 
-                ProxyRepository repository =
-                    new ProxyRepository( key, url, new DefaultRepositoryLayout(), cacheFailures, cachePeriod );
+            boolean cacheFailures =
+                Boolean.valueOf( repoProps.getProperty( "cache.failures", "false" ) ).booleanValue();
+            boolean hardFail = Boolean.valueOf( repoProps.getProperty( "hardfail", "true" ) ).booleanValue();
+            long cachePeriod = Long.parseLong( repoProps.getProperty( "cache.period", "0" ) );
 
-                repository.setHardfail( hardFail );
+            ProxyRepository repository =
+                new ProxyRepository( key, url, new DefaultRepositoryLayout(), cacheFailures, cachePeriod );
 
-                if ( StringUtils.isNotEmpty( proxyKey ) )
-                {
-                    repository.setProxied( true );
-                }
+            repository.setHardfail( hardFail );
 
-                repositories.add( repository );
+            if ( StringUtils.isNotEmpty( proxyKey ) )
+            {
+                repository.setProxied( true );
             }
+
+            repositories.add( repository );
         }
+
         config.setRepositories( repositories );
 
         validateDirectories( config );
@@ -161,7 +161,7 @@ public class MavenProxyPropertyLoader
     private String getMandatoryProperty( Properties props, String key )
         throws ValidationException
     {
-        final String value = props.getProperty( key );
+        String value = props.getProperty( key );
 
         if ( value == null )
         {
index b54f02a4f687464eb180479474fdf4dd02f9d865..d873c0af2e57b7b617ecd327a4620267154b67ad 100644 (file)
@@ -74,7 +74,7 @@ public class ProxyConfiguration
         proxyInfo.setHost( host );
         proxyInfo.setPort( port );
 
-        setHttpProxy( proxyInfo );
+        httpProxy = proxyInfo;
     }
 
     public void setHttpProxy( String host, int port, String username, String password )
index 6f1ca364b052e41f39612cd8f1245d2149428a19..dbd5401f8dc925ff557f72384cd1d319b17ff782 100644 (file)
@@ -1,13 +1,13 @@
 package org.apache.maven.repository.proxy.configuration;
 
 /*
- * Copyright 2003-2004 The Apache Software Foundation.
+ * Copyright 2005-2006 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
+ *      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,
index 9e2ae73e9e2e7a4ef9686ec5f3aeacb89e649783..24499f023a47f4d41fc68750505caa0df3db7777 100644 (file)
@@ -29,22 +29,22 @@ public class ProxyRepository
     extends DefaultArtifactRepository
 {
     // zero caches forever
-    private long cachePeriod = 0;
+    private long cachePeriod;
 
-    private boolean cacheFailures = false;
+    private boolean cacheFailures;
 
     private boolean hardfail = true;
 
-    private boolean proxied = false;
+    private boolean proxied;
 
     public ProxyRepository( String id, String url, ArtifactRepositoryLayout layout, boolean cacheFailures,
                             long cachePeriod )
     {
         this( id, url, layout );
 
-        setCacheFailures( cacheFailures );
+        this.cacheFailures = cacheFailures;
 
-        setCachePeriod( cachePeriod );
+        this.cachePeriod = cachePeriod;
     }
 
     public ProxyRepository( String id, String url, ArtifactRepositoryLayout layout )
index f3f8b27ba518958f0510a9295d84e865e7e14eba..a15c595effdbc1791fe3281ee8b39a1b3558c92d 100644 (file)
@@ -1,15 +1,5 @@
 package org.apache.maven.repository.proxy;
 
-import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;
-import org.apache.maven.artifact.repository.layout.DefaultRepositoryLayout;
-import org.apache.maven.repository.proxy.configuration.ProxyConfiguration;
-import org.apache.maven.repository.proxy.repository.ProxyRepository;
-import org.apache.maven.wagon.ResourceDoesNotExistException;
-import org.codehaus.plexus.PlexusTestCase;
-import org.codehaus.plexus.component.repository.exception.ComponentLookupException;
-
-import java.io.File;
-
 /*
  * Copyright 2005-2006 The Apache Software Foundation.
  *
@@ -26,6 +16,16 @@ import java.io.File;
  * limitations under the License.
  */
 
+import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;
+import org.apache.maven.artifact.repository.layout.DefaultRepositoryLayout;
+import org.apache.maven.repository.proxy.configuration.ProxyConfiguration;
+import org.apache.maven.repository.proxy.repository.ProxyRepository;
+import org.apache.maven.wagon.ResourceDoesNotExistException;
+import org.codehaus.plexus.PlexusTestCase;
+import org.codehaus.plexus.component.repository.exception.ComponentLookupException;
+
+import java.io.File;
+
 /**
  * @author Edwin Punzalan
  */
@@ -73,11 +73,11 @@ public class DefaultProxyManagerTest
                     file.getAbsolutePath().startsWith( proxy.getConfiguration().getRepositoryCachePath() ) );
 
         //test cache
-        file = proxy.get( "/commons-logging/commons-logging/1.0/commons-logging-1.0.jar" );
+        proxy.get( "/commons-logging/commons-logging/1.0/commons-logging-1.0.jar" );
 
         try
         {
-            file = proxy.get( "/commons-logging/commons-logging/2.0/commons-logging-2.0.jar" );
+            proxy.get( "/commons-logging/commons-logging/2.0/commons-logging-2.0.jar" );
             fail( "Expected ResourceDoesNotExistException exception not thrown" );
         }
         catch ( ResourceDoesNotExistException e )
index 768e505b88ee88ec2527150f117e9615b1e15005..b0f6d1a7aede3ded2338858fff9863952e0726b8 100644 (file)
@@ -73,11 +73,11 @@ public class LegacyProxyManagerTest
                     file.getAbsolutePath().startsWith( proxy.getConfiguration().getRepositoryCachePath() ) );
 
         //test cache
-        file = proxy.get( "/commons-logging/jars/commons-logging-1.0.jar" );
+        proxy.get( "/commons-logging/jars/commons-logging-1.0.jar" );
 
         try
         {
-            file = proxy.get( "/commons-logging/jars/commons-logging-2.0.jar" );
+            proxy.get( "/commons-logging/jars/commons-logging-2.0.jar" );
             fail( "Expected ResourceDoesNotExistException exception not thrown" );
         }
         catch ( ResourceDoesNotExistException e )
index 404aed04a801b423bf4b1051af50887efce82710..88fee50695f07d9127e8c81110e1073a55f2c9a9 100644 (file)
@@ -23,7 +23,6 @@ import org.codehaus.plexus.util.FileUtils;
 import java.io.File;
 import java.io.FileInputStream;
 import java.io.IOException;
-import java.util.Iterator;
 
 
 /**
@@ -32,6 +31,8 @@ import java.util.Iterator;
 public class MavenProxyPropertyLoaderTest
     extends PlexusTestCase
 {
+    private static final int DEFAULT_CACHE_PERIOD = 3600;
+
     public void testLoadValidMavenProxyConfiguration()
         throws ValidationException, IOException
     {
@@ -50,51 +51,34 @@ public class MavenProxyPropertyLoaderTest
 
             assertEquals( "Count repositories", 4, config.getRepositories().size() );
 
-            int idx = 0;
-            for ( Iterator repos = config.getRepositories().iterator(); repos.hasNext(); )
-            {
-                idx++;
-
-                ProxyRepository repo = (ProxyRepository) repos.next();
-
-                //switch is made to check for ordering
-                switch ( idx )
-                {
-                    case 1:
-                        assertEquals( "Repository name not as expected", "local-repo", repo.getKey() );
-                        assertEquals( "Repository url does not match its name", "file://target", repo.getUrl() );
-                        assertEquals( "Repository cache period check failed", 0, repo.getCachePeriod() );
-                        assertFalse( "Repository failure caching check failed", repo.isCacheFailures() );
-                        break;
-                    case 2:
-                        assertEquals( "Repository name not as expected", "www-ibiblio-org", repo.getKey() );
-                        assertEquals( "Repository url does not match its name", "http://www.ibiblio.org/maven2",
-                                      repo.getUrl() );
-                        assertEquals( "Repository cache period check failed", 3600, repo.getCachePeriod() );
-                        assertTrue( "Repository failure caching check failed", repo.isCacheFailures() );
-                        break;
-                    case 3:
-                        assertEquals( "Repository name not as expected", "dist-codehaus-org", repo.getKey() );
-                        assertEquals( "Repository url does not match its name", "http://dist.codehaus.org",
-                                      repo.getUrl() );
-                        assertEquals( "Repository cache period check failed", 3600, repo.getCachePeriod() );
-                        assertTrue( "Repository failure caching check failed", repo.isCacheFailures() );
-                        break;
-                    case 4:
-                        assertEquals( "Repository name not as expected", "private-example-com", repo.getKey() );
-                        assertEquals( "Repository url does not match its name", "http://private.example.com/internal",
-                                      repo.getUrl() );
-                        assertEquals( "Repository cache period check failed", 3600, repo.getCachePeriod() );
-                        assertFalse( "Repository failure caching check failed", repo.isCacheFailures() );
-                        break;
-                    default:
-                        fail( "Unexpected order count" );
-                }
-            }
+            ProxyRepository repo = (ProxyRepository) config.getRepositories().get( 0 );
+            assertEquals( "Repository name not as expected", "local-repo", repo.getKey() );
+            assertEquals( "Repository url does not match its name", "file://target", repo.getUrl() );
+            assertEquals( "Repository cache period check failed", 0, repo.getCachePeriod() );
+            assertFalse( "Repository failure caching check failed", repo.isCacheFailures() );
+
+            repo = (ProxyRepository) config.getRepositories().get( 1 );
+            assertEquals( "Repository name not as expected", "www-ibiblio-org", repo.getKey() );
+            assertEquals( "Repository url does not match its name", "http://www.ibiblio.org/maven2", repo.getUrl() );
+            assertEquals( "Repository cache period check failed", DEFAULT_CACHE_PERIOD, repo.getCachePeriod() );
+            assertTrue( "Repository failure caching check failed", repo.isCacheFailures() );
+
+            repo = (ProxyRepository) config.getRepositories().get( 2 );
+            assertEquals( "Repository name not as expected", "dist-codehaus-org", repo.getKey() );
+            assertEquals( "Repository url does not match its name", "http://dist.codehaus.org", repo.getUrl() );
+            assertEquals( "Repository cache period check failed", DEFAULT_CACHE_PERIOD, repo.getCachePeriod() );
+            assertTrue( "Repository failure caching check failed", repo.isCacheFailures() );
+
+            repo = (ProxyRepository) config.getRepositories().get( 3 );
+            assertEquals( "Repository name not as expected", "private-example-com", repo.getKey() );
+            assertEquals( "Repository url does not match its name", "http://private.example.com/internal",
+                          repo.getUrl() );
+            assertEquals( "Repository cache period check failed", DEFAULT_CACHE_PERIOD, repo.getCachePeriod() );
+            assertFalse( "Repository failure caching check failed", repo.isCacheFailures() );
         }
-        //make sure to delete the test directory after tests
         finally
         {
+            //make sure to delete the test directory after tests
             FileUtils.deleteDirectory( "target/remote-repo1" );
         }
     }
index 491e8bf19e5c0d50e1a813e3536c5e89e708cbd7..53f583a98cbac3b2fb24cf2a38d0b404cfbec88b 100644 (file)
@@ -32,6 +32,10 @@ public class ProxyConfigurationTest
 {\r
     private ProxyConfiguration config;\r
 \r
+    private static final int DEFAULT_CACHE_PERIOD = 3600;\r
+\r
+    private static final int DEFAULT_PORT = 80;\r
+\r
     protected void setUp()\r
         throws Exception\r
     {\r
@@ -60,9 +64,9 @@ public class ProxyConfigurationTest
         ArtifactRepositoryLayout legacyLayout = new LegacyRepositoryLayout();\r
         ProxyRepository repo2 = new ProxyRepository( "repo2", "http://www.ibiblio.org/maven", legacyLayout );\r
         repo2.setCacheFailures( false );\r
-        repo2.setCachePeriod( 3600 );\r
+        repo2.setCachePeriod( DEFAULT_CACHE_PERIOD );\r
         repo2.setProxied( true );\r
-        config.setHttpProxy( "some.local.proxy", 80, "username", "password" );\r
+        config.setHttpProxy( "some.local.proxy", DEFAULT_PORT, "username", "password" );\r
         config.addRepository( repo2 );\r
         assertEquals( 2, config.getRepositories().size() );\r
 \r
@@ -80,14 +84,14 @@ public class ProxyConfigurationTest
         assertEquals( "http://www.ibiblio.org/maven", repo.getUrl() );\r
         assertFalse( repo.isCacheFailures() );\r
         assertTrue( repo.isHardfail() );\r
-        assertEquals( 3600, repo.getCachePeriod() );\r
+        assertEquals( DEFAULT_CACHE_PERIOD, repo.getCachePeriod() );\r
         assertEquals( repo2, repo );\r
         assertTrue( repo.isProxied() );\r
 \r
         ProxyInfo proxyInfo = config.getHttpProxy();\r
         assertNotNull( proxyInfo );\r
         assertEquals( "some.local.proxy", proxyInfo.getHost() );\r
-        assertEquals( 80, proxyInfo.getPort() );\r
+        assertEquals( DEFAULT_PORT, proxyInfo.getPort() );\r
         assertEquals( "username", proxyInfo.getUserName() );\r
         assertEquals( "password", proxyInfo.getPassword() );\r
 \r
@@ -96,7 +100,7 @@ public class ProxyConfigurationTest
             repositories.add( new ProxyRepository( "repo", "url", defLayout ) );\r
             fail( "Expected UnsupportedOperationException not thrown." );\r
         }\r
-        catch ( java.lang.UnsupportedOperationException e )\r
+        catch ( UnsupportedOperationException e )\r
         {\r
             assertTrue( true );\r
         }\r
index 02809c60ef2bdfc00693bf175093683cf9c352b0..1c894d6af6ce281162206220cd728d3bbcc3e3bb 100644 (file)
@@ -132,7 +132,7 @@ public class Cache
      * @param key   the object to map the valued object
      * @param value the object to cache
      */
-    public Object put( Object key, Object value )
+    public void put( Object key, Object value )
     {
         Object old = null;
 
@@ -148,8 +148,6 @@ public class Cache
         }
 
         manageCache();
-
-        return old;
     }
 
     /**
index e4d7d6602031b1ef7907992b485600f865b425a4..26ff14da57b27f2592e29d12333f513af45d6880 100644 (file)
@@ -37,6 +37,10 @@ import java.util.StringTokenizer;
  */
 public class ArtifactUtils
 {
+    private ArtifactUtils()
+    {
+    }
+
     /**
      * Method used to build an artifact and then set its repository and file fields with the proper values
      *
@@ -245,6 +249,7 @@ public class ArtifactUtils
                 // contains artifactId, version, classifier, and extension.
                 String avceGlob = tokens.nextToken();
 
+                //noinspection CollectionDeclaredAsConcreteClass
                 LinkedList avceTokenList = new LinkedList();
 
                 StringTokenizer avceTokenizer = new StringTokenizer( avceGlob, "-" );
index f138a67e25e0f0b3e77be2db31f45cf187592a44..4974fb05d1d52aa03de572ec34b864d0ef8d2d47 100644 (file)
@@ -1,16 +1,5 @@
 package org.apache.maven.repository.digest;
 
-import org.codehaus.plexus.util.IOUtil;
-
-import java.io.File;
-import java.io.FileInputStream;
-import java.io.IOException;
-import java.io.InputStream;
-import java.security.MessageDigest;
-import java.security.NoSuchAlgorithmException;
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
-
 /*
  * Copyright 2005-2006 The Apache Software Foundation.
  *
@@ -27,6 +16,17 @@ import java.util.regex.Pattern;
  * limitations under the License.
  */
 
+import org.codehaus.plexus.util.IOUtil;
+
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.IOException;
+import java.io.InputStream;
+import java.security.MessageDigest;
+import java.security.NoSuchAlgorithmException;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
+
 /**
  * Create a digest for a file.
  *
index 6c0576723c1b4aa0281c505441ccdd85ee9df98a..8a1359f41838936a229c9b7572ad48749d532d65 100644 (file)
@@ -1,5 +1,21 @@
 package org.apache.maven.repository;
 
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.factory.ArtifactFactory;
 import org.codehaus.plexus.PlexusTestCase;
index cad86049f9ef7262c7e90e3c6ee31524b8b32551..6e275db52162bd348a2f3ffa307e463e11cd910b 100644 (file)
@@ -1,3 +1,19 @@
+<!--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
index e05cf20b4caed2dee983beac6ec0266ade313e51..fe4a615a692c8f1a4433e6d5490518a9d8160760 100644 (file)
@@ -17,8 +17,8 @@ package org.apache.maven.repository.manager.web.action;
  */\r
 \r
 import com.opensymphony.xwork.Action;\r
-import org.apache.maven.repository.manager.web.job.DiscovererScheduler;\r
 import org.apache.maven.repository.manager.web.execution.DiscovererExecution;\r
+import org.apache.maven.repository.manager.web.job.DiscovererScheduler;\r
 \r
 /**\r
  * This is the Action class of index.jsp, which is the initial page of the web application.\r
@@ -53,6 +53,7 @@ public class BaseAction
         }\r
         catch ( Exception e )\r
         {\r
+            // TODO: better exception handling!\r
             e.printStackTrace();\r
             return ERROR;\r
         }\r
index 5b182cc84f6d8b555faf565d785998313b701960..459a8d4084440a31fca97a6a07eb6fd42bb46c4d 100644 (file)
@@ -1,5 +1,21 @@
 package org.apache.maven.repository.manager.web.action;\r
 \r
+/*\r
+ * Copyright 2005-2006 The Apache Software Foundation.\r
+ *\r
+ * Licensed under the Apache License, Version 2.0 (the "License");\r
+ * you may not use this file except in compliance with the License.\r
+ * You may obtain a copy of the License at\r
+ *\r
+ *      http://www.apache.org/licenses/LICENSE-2.0\r
+ *\r
+ * Unless required by applicable law or agreed to in writing, software\r
+ * distributed under the License is distributed on an "AS IS" BASIS,\r
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+ * See the License for the specific language governing permissions and\r
+ * limitations under the License.\r
+ */\r
+\r
 import com.opensymphony.xwork.Action;\r
 import org.apache.maven.artifact.repository.ArtifactRepository;\r
 import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;\r
index a5e916085e4093ada5c960ea37e33ecf3d17252b..409dcc179aa47b684b7d5a36244d27fc51d32762 100644 (file)
@@ -20,14 +20,13 @@ import com.opensymphony.xwork.Action;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;
 import org.apache.maven.repository.indexing.ArtifactRepositoryIndex;
-import org.apache.maven.repository.indexing.DefaultRepositoryIndexSearcher;
+import org.apache.maven.repository.indexing.RepositoryIndex;
 import org.apache.maven.repository.indexing.RepositoryIndexException;
 import org.apache.maven.repository.indexing.RepositoryIndexSearchException;
-import org.apache.maven.repository.indexing.RepositoryIndexingFactory;
 import org.apache.maven.repository.indexing.RepositoryIndexSearchLayer;
+import org.apache.maven.repository.indexing.RepositoryIndexingFactory;
 import org.apache.maven.repository.indexing.query.SinglePhraseQuery;
 import org.apache.maven.repository.manager.web.job.Configuration;
-import org.apache.maven.repository.indexing.RepositoryIndex;
 
 import java.io.File;
 import java.net.MalformedURLException;
index 62d223d982ff4652c593d7b44701e88b4020d731..870fe06fd683131cad8a888804432a95f042e9dc 100644 (file)
@@ -1,5 +1,21 @@
 package org.apache.maven.repository.manager.web.action;\r
 \r
+/*\r
+ * Copyright 2005-2006 The Apache Software Foundation.\r
+ *\r
+ * Licensed under the Apache License, Version 2.0 (the "License");\r
+ * you may not use this file except in compliance with the License.\r
+ * You may obtain a copy of the License at\r
+ *\r
+ *      http://www.apache.org/licenses/LICENSE-2.0\r
+ *\r
+ * Unless required by applicable law or agreed to in writing, software\r
+ * distributed under the License is distributed on an "AS IS" BASIS,\r
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+ * See the License for the specific language governing permissions and\r
+ * limitations under the License.\r
+ */\r
+\r
 import com.opensymphony.xwork.Action;\r
 import org.apache.maven.artifact.Artifact;\r
 import org.apache.maven.artifact.repository.ArtifactRepository;\r
@@ -11,6 +27,7 @@ import java.util.ArrayList;
 import java.util.Collections;\r
 import java.util.Iterator;\r
 import java.util.List;\r
+import java.util.Map;\r
 import java.util.TreeMap;\r
 \r
 /**\r
@@ -38,15 +55,15 @@ public class RepositoryBrowseAction
 \r
     private String group;\r
 \r
-    private TreeMap artifactMap;\r
+    private Map artifactMap;\r
 \r
     private String folder;\r
 \r
     private int idx;\r
 \r
     public String execute()\r
-        throws Exception\r
     {\r
+        // TODO! fix hardcoded path\r
         String path = "E:/jeprox/maven-repository-manager/trunk/maven-repository-discovery/src/test/repository";\r
 \r
         ArtifactRepository repository =\r
@@ -58,21 +75,19 @@ public class RepositoryBrowseAction
 \r
         artifactMap = new TreeMap();\r
 \r
-        String groupId;\r
-\r
         while ( iterator.hasNext() )\r
         {\r
             Artifact artifact = (Artifact) iterator.next();\r
 \r
-            groupId = artifact.getGroupId();\r
+            String groupId = artifact.getGroupId();\r
 \r
             String key = groupId.replace( '.', '/' ) + "/" + artifact.getArtifactId() + "/" + artifact.getVersion();\r
 \r
-            ArrayList artifactList;\r
+            List artifactList;\r
 \r
             if ( artifactMap.containsKey( key ) )\r
             {\r
-                artifactList = (ArrayList) artifactMap.get( key );\r
+                artifactList = (List) artifactMap.get( key );\r
             }\r
             else\r
             {\r
@@ -87,28 +102,28 @@ public class RepositoryBrowseAction
         }\r
 \r
         //set the index for folder level to be displayed\r
-        setIdx( 1 );\r
+        idx = 1;\r
 \r
-        setFolder( "" );\r
+        folder = "";\r
 \r
         return SUCCESS;\r
     }\r
 \r
+    // TODO! is this method needed?\r
     public String doEdit()\r
-        throws Exception\r
     {\r
-        setIdx( getIdx() + 1 );\r
+        idx = idx + 1;\r
 \r
         //set folder to "" if we are at the root directory\r
-        if ( getIdx() == 1 )\r
+        if ( idx == 1 )\r
         {\r
-            setFolder( "" );\r
+            folder = "";\r
         }\r
 \r
         return SUCCESS;\r
     }\r
 \r
-    public TreeMap getArtifactMap()\r
+    public Map getArtifactMap()\r
     {\r
         return artifactMap;\r
     }\r
index 7b3de21a3e7dfc18862ad0ca5231a774014ea3f6..465ebe412ffb28bc14b4d30af3151eb5b1c3b898 100644 (file)
@@ -16,29 +16,29 @@ package org.apache.maven.repository.manager.web.execution;
  * limitations under the License.
  */
 
+import org.apache.lucene.index.IndexReader;
+import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;
 import org.apache.maven.artifact.repository.DefaultArtifactRepositoryFactory;
 import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;
 import org.apache.maven.artifact.repository.metadata.RepositoryMetadata;
-import org.apache.maven.artifact.Artifact;
-import org.apache.maven.repository.indexing.RepositoryIndexException;
+import org.apache.maven.model.Model;
+import org.apache.maven.repository.discovery.ArtifactDiscoverer;
+import org.apache.maven.repository.discovery.MetadataDiscoverer;
 import org.apache.maven.repository.indexing.ArtifactRepositoryIndex;
 import org.apache.maven.repository.indexing.MetadataRepositoryIndex;
 import org.apache.maven.repository.indexing.PomRepositoryIndex;
+import org.apache.maven.repository.indexing.RepositoryIndexException;
 import org.apache.maven.repository.indexing.RepositoryIndexingFactory;
 import org.apache.maven.repository.manager.web.job.Configuration;
-import org.apache.maven.repository.discovery.ArtifactDiscoverer;
-import org.apache.maven.repository.discovery.MetadataDiscoverer;
-import org.apache.maven.model.Model;
-import org.apache.lucene.index.IndexReader;
 import org.codehaus.plexus.logging.AbstractLogEnabled;
 
-import java.util.List;
-import java.util.Iterator;
-import java.util.Properties;
 import java.io.File;
 import java.net.MalformedURLException;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Properties;
 
 /**
  * This is the class that executes the discoverer and indexer.
@@ -80,8 +80,6 @@ public class DiscovererExecution
 
     private ArtifactRepositoryLayout layout;
 
-    private Properties props;
-
     private String indexPath;
 
     private String blacklistedPatterns;
@@ -101,24 +99,16 @@ public class DiscovererExecution
     public void executeDiscovererIfIndexDoesNotExist()
         throws MalformedURLException, RepositoryIndexException
     {
-        props = config.getProperties();
+        Properties props = config.getProperties();
         indexPath = props.getProperty( "index.path" );
 
         File indexDir = new File( indexPath );
-        boolean isExisting;
+        boolean isExisting = false;
 
         if ( IndexReader.indexExists( indexDir ) )
         {
             isExisting = true;
         }
-        else if ( !indexDir.exists() )
-        {
-            isExisting = false;
-        }
-        else
-        {
-            isExisting = false;
-        }
 
         if ( !isExisting )
         {
@@ -132,12 +122,12 @@ public class DiscovererExecution
     public void executeDiscoverer()
         throws MalformedURLException, RepositoryIndexException
     {
-        props = config.getProperties();
+        Properties props = config.getProperties();
         indexPath = props.getProperty( "index.path" );
         layout = config.getLayout();
         blacklistedPatterns = props.getProperty( "blacklist.patterns" );
-        includeSnapshots = new Boolean( props.getProperty( "include.snapshots" ) ).booleanValue();
-        convertSnapshots = new Boolean( props.getProperty( "convert.snapshots" ) ).booleanValue();
+        includeSnapshots = Boolean.valueOf( props.getProperty( "include.snapshots" ) ).booleanValue();
+        convertSnapshots = Boolean.valueOf( props.getProperty( "convert.snapshots" ) ).booleanValue();
 
         try
         {
@@ -149,11 +139,11 @@ public class DiscovererExecution
         }
 
         getLogger().info( "[DiscovererExecution] Started discovery and indexing.." );
-        if ( props.getProperty( "layout" ).equals( "default" ) )
+        if ( "default".equals( props.getProperty( "layout" ) ) )
         {
             executeDiscovererInDefaultRepo();
         }
-        else if ( props.getProperty( "layout" ).equals( "legacy" ) )
+        else if ( "legacy".equals( props.getProperty( "layout" ) ) )
         {
             executeDiscovererInLegacyRepo();
         }
@@ -210,17 +200,7 @@ public class DiscovererExecution
         for ( Iterator iter = artifacts.iterator(); iter.hasNext(); )
         {
             Artifact artifact = (Artifact) iter.next();
-            try
-            {
-                artifactIndex.indexArtifact( artifact );
-            }
-            catch ( Exception e )
-            {
-                if ( e instanceof RepositoryIndexException )
-                {
-                    throw (RepositoryIndexException) e;
-                }
-            }
+            artifactIndex.indexArtifact( artifact );
 
             if ( artifactIndex.isOpen() )
             {
@@ -248,17 +228,8 @@ public class DiscovererExecution
         for ( Iterator iter = metadataList.iterator(); iter.hasNext(); )
         {
             RepositoryMetadata repoMetadata = (RepositoryMetadata) iter.next();
-            try
-            {
-                metadataIndex.index( repoMetadata );
-            }
-            catch ( Exception e )
-            {
-                if ( e instanceof RepositoryIndexException )
-                {
-                    throw (RepositoryIndexException) e;
-                }
-            }
+            metadataIndex.index( repoMetadata );
+
             if ( metadataIndex.isOpen() )
             {
                 metadataIndex.optimize();
@@ -281,17 +252,7 @@ public class DiscovererExecution
         for ( Iterator iter = models.iterator(); iter.hasNext(); )
         {
             Model model = (Model) iter.next();
-            try
-            {
-                pomIndex.indexPom( model );
-            }
-            catch ( Exception e )
-            {
-                if ( e instanceof RepositoryIndexException )
-                {
-                    throw (RepositoryIndexException) e;
-                }
-            }
+            pomIndex.indexPom( model );
 
             if ( pomIndex.isOpen() )
             {
index c0fe67f7986567fd3ead92a31bdc67aeb80b4bd6..0e147321c24aab34863e2ce3a12b4a06fe6c7586 100644 (file)
@@ -1,12 +1,5 @@
 package org.apache.maven.repository.manager.web.job;\r
 \r
-import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;\r
-import org.apache.maven.artifact.repository.layout.DefaultRepositoryLayout;\r
-import org.apache.maven.artifact.repository.layout.LegacyRepositoryLayout;\r
-import org.codehaus.plexus.personality.plexus.lifecycle.phase.Initializable;\r
-import org.codehaus.plexus.personality.plexus.lifecycle.phase.InitializationException;\r
-\r
-import java.util.Properties;\r
 /*\r
  * Copyright 2005-2006 The Apache Software Foundation.\r
  *\r
@@ -23,6 +16,14 @@ import java.util.Properties;
  * limitations under the License.\r
  */\r
 \r
+import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;\r
+import org.apache.maven.artifact.repository.layout.DefaultRepositoryLayout;\r
+import org.apache.maven.artifact.repository.layout.LegacyRepositoryLayout;\r
+import org.codehaus.plexus.personality.plexus.lifecycle.phase.Initializable;\r
+import org.codehaus.plexus.personality.plexus.lifecycle.phase.InitializationException;\r
+\r
+import java.util.Properties;\r
+\r
 /**\r
  * This class contains the configuration values to be used by the scheduler\r
  */\r
index f7a27ce5dad49939b2cbbb8f2975be0410bbdef3..9a93a115bf766b93beebe39857ab5335e64fc98a 100644 (file)
@@ -16,32 +16,14 @@ package org.apache.maven.repository.manager.web.job;
  * limitations under the License.\r
  */\r
 \r
-import org.apache.maven.artifact.Artifact;\r
-import org.apache.maven.artifact.repository.ArtifactRepository;\r
-import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;\r
-import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;\r
-import org.apache.maven.artifact.repository.metadata.RepositoryMetadata;\r
-import org.apache.maven.model.Model;\r
-import org.apache.maven.repository.discovery.ArtifactDiscoverer;\r
-import org.apache.maven.repository.discovery.DefaultArtifactDiscoverer;\r
-import org.apache.maven.repository.discovery.DefaultMetadataDiscoverer;\r
-import org.apache.maven.repository.discovery.LegacyArtifactDiscoverer;\r
-import org.apache.maven.repository.discovery.MetadataDiscoverer;\r
-import org.apache.maven.repository.indexing.ArtifactRepositoryIndex;\r
-import org.apache.maven.repository.indexing.MetadataRepositoryIndex;\r
-import org.apache.maven.repository.indexing.PomRepositoryIndex;\r
 import org.apache.maven.repository.indexing.RepositoryIndexException;\r
-import org.apache.maven.repository.indexing.RepositoryIndexingFactory;\r
 import org.apache.maven.repository.manager.web.execution.DiscovererExecution;\r
 import org.codehaus.plexus.scheduler.AbstractJob;\r
 import org.quartz.JobDataMap;\r
 import org.quartz.JobExecutionContext;\r
 import org.quartz.JobExecutionException;\r
 \r
-import java.io.File;\r
 import java.net.MalformedURLException;\r
-import java.util.Iterator;\r
-import java.util.List;\r
 \r
 /**\r
  * This class is the discoverer job that is executed by the scheduler.\r
@@ -53,7 +35,7 @@ public class DiscovererJob
 {\r
     public static final String ROLE = DiscovererJob.class.getName();\r
 \r
-    public static String MAP_DISCOVERER_EXECUTION = "EXECUTION";\r
+    public static final String MAP_DISCOVERER_EXECUTION = "EXECUTION";\r
 \r
     /**\r
      * Execute the discoverer and the indexer.\r
@@ -76,10 +58,12 @@ public class DiscovererJob
         }\r
         catch ( RepositoryIndexException e )\r
         {\r
+            // TODO!\r
             e.printStackTrace();\r
         }\r
         catch ( MalformedURLException me )\r
         {\r
+            // TODO!\r
             me.printStackTrace();\r
         }\r
 \r
index bc708234cde45b68bf8c34bd0e2d6d7ce74b4229..8d66dfe0d5012a0b8a9218a51f554cd6efbcf3df 100644 (file)
@@ -16,26 +16,18 @@ package org.apache.maven.repository.manager.web.job;
  * limitations under the License.\r
  */\r
 \r
-import java.io.File;\r
-import java.io.IOException;\r
-import java.net.MalformedURLException;\r
-import java.text.ParseException;\r
-import java.util.Properties;\r
-\r
-import org.apache.maven.artifact.repository.ArtifactRepository;\r
-import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;\r
-import org.apache.maven.artifact.repository.DefaultArtifactRepositoryFactory;\r
-import org.apache.maven.repository.discovery.ArtifactDiscoverer;\r
-import org.apache.maven.repository.discovery.MetadataDiscoverer;\r
-import org.apache.maven.repository.indexing.RepositoryIndexingFactory;\r
 import org.apache.maven.repository.manager.web.execution.DiscovererExecution;\r
 import org.codehaus.plexus.logging.AbstractLogEnabled;\r
+import org.codehaus.plexus.scheduler.AbstractJob;\r
 import org.codehaus.plexus.scheduler.Scheduler;\r
 import org.quartz.CronTrigger;\r
 import org.quartz.JobDataMap;\r
 import org.quartz.JobDetail;\r
 import org.quartz.SchedulerException;\r
 \r
+import java.text.ParseException;\r
+import java.util.Properties;\r
+\r
 /**\r
  * This class sets the job to be executed in the plexus-quartz scheduler\r
  *\r
@@ -54,8 +46,6 @@ public class DiscovererScheduler
      */\r
     private Scheduler scheduler;\r
 \r
-    private Properties props;\r
-\r
     /**\r
      * @plexus.requirement\r
      */\r
@@ -64,17 +54,16 @@ public class DiscovererScheduler
     /**\r
      * Method that sets the schedule in the plexus-quartz scheduler\r
      *\r
-     * @throws IOException\r
      * @throws ParseException\r
      * @throws SchedulerException\r
      */\r
     public void setSchedule()\r
-        throws IOException, ParseException, SchedulerException\r
+        throws ParseException, SchedulerException\r
     {\r
-        props = config.getProperties();\r
+        Properties props = config.getProperties();\r
         JobDetail jobDetail = new JobDetail( "discovererJob", "DISCOVERER", DiscovererJob.class );\r
         JobDataMap dataMap = new JobDataMap();\r
-        dataMap.put( DiscovererJob.LOGGER, getLogger() );\r
+        dataMap.put( AbstractJob.LOGGER, getLogger() );\r
         dataMap.put( DiscovererJob.MAP_DISCOVERER_EXECUTION, execution );\r
         jobDetail.setJobDataMap( dataMap );\r
 \r
index 6838e8ed0b01955b0ae14d1cbd7a5afd5bfa5844..03131ae9f517b301a99537406420e805203e22c8 100644 (file)
@@ -1,3 +1,19 @@
+<!--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
 <decorators defaultdir="/WEB-INF/jsp/decorators">
   <decorator name="default" page="default.jsp">
     <pattern>/*</pattern>
index bf950fd658fde90cfc775faf5a31e42a935b6805..a07fe5948d8cf55704770b4ae6946cd45476c706 100644 (file)
@@ -1,3 +1,19 @@
+<%--\r
+  ~ Copyright 2005-2006 The Apache Software Foundation.\r
+  ~\r
+  ~ Licensed under the Apache License, Version 2.0 (the "License");\r
+  ~ you may not use this file except in compliance with the License.\r
+  ~ You may obtain a copy of the License at\r
+  ~\r
+  ~      http://www.apache.org/licenses/LICENSE-2.0\r
+  ~\r
+  ~ Unless required by applicable law or agreed to in writing, software\r
+  ~ distributed under the License is distributed on an "AS IS" BASIS,\r
+  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+  ~ See the License for the specific language governing permissions and\r
+  ~ limitations under the License.\r
+  --%>\r
+\r
 <%@ taglib uri="webwork" prefix="ww" %>\r
 \r
 <html>\r
 \r
 <body>\r
 <h3><a href="<ww:url value="browse!edit.action"><ww:param name="idx" value="0"/></ww:url>">basedir</a> /\r
-  <ww:set name="previousFolder" value="''"/>\r
-  <ww:set name="counter" value="0"/>\r
+  <ww:set name="previousFolder" value="''" />\r
+  <ww:set name="counter" value="0" />\r
   <ww:if test="folder != ''">\r
-    <ww:set name="folderHeader" value="folder.split('/')"/>\r
+    <ww:set name="folderHeader" value="folder.split('/')" />\r
     <ww:iterator value="#folderHeader">\r
-      <ww:set name="counter" value="#counter + 1"/>\r
+      <ww:set name="counter" value="#counter + 1" />\r
       <ww:if test="#previousFolder == ''">\r
-        <ww:set name="previousFolder" value="top"/>\r
+        <ww:set name="previousFolder" value="top" />\r
       </ww:if>\r
       <ww:else>\r
-        <ww:set name="previousFolder" value="#previousFolder + '/' + top"/>\r
+        <ww:set name="previousFolder" value="#previousFolder + '/' + top" />\r
       </ww:else>\r
       <ww:if test="idx > (#counter + 1)"><a href="<ww:url value="browse!edit.action"><ww:param name="idx"><ww:property\r
-          value="#counter"/></ww:param><ww:param name="folder"></ww:param></ww:url>"></ww:if><ww:property/></a> /\r
+          value="#counter" /></ww:param><ww:param name="folder"></ww:param></ww:url>"></ww:if><ww:property /></a> /\r
     </ww:iterator>\r
   </ww:if>\r
 </h3>\r
-<br/>\r
+<br />\r
 \r
 <ww:set name="previousFolder" value="'the previous folder'"/>\r
 <ww:set name="in" value="idx" scope="page"/>\r
 <ww:if test="idx == 1 || (folder != '' and  #groupName.startsWith(folder))">\r
   <%\r
 \r
+\r
   int ctr = 1;\r
 \r
+\r
   %>\r
   <ww:set name="groupFolder" value="#groupName.split('/')"/>\r
 <ww:iterator value="#groupFolder">\r
   <%\r
 \r
+\r
 if (ctr == ((Integer)pageContext.getAttribute("in")).intValue()) {\r
+\r
   %>\r
 <ww:if test="top != #previousFolder">\r
   <ww:set name="previousFolder" value="top"/>\r
@@ -57,32 +77,32 @@ if (ctr == ((Integer)pageContext.getAttribute("in")).intValue()) {
   </ww:iterator>\r
 \r
   <ww:if test="folder != ''">\r
-    <ww:set name="previousFolder" value="''"/>\r
-    <ww:set name="artifactList" value="artifactMap.get(folder)"/>\r
+    <ww:set name="previousFolder" value="''" />\r
+    <ww:set name="artifactList" value="artifactMap.get(folder)" />\r
     <ww:iterator value="#artifactList">\r
       <table border="1">\r
         <tr align="left">\r
           <th>Group ID</th>\r
-          <td><ww:property value="groupId"/></td>\r
+          <td><ww:property value="groupId" /></td>\r
         </tr>\r
         <tr align="left">\r
           <th>Artifact ID</th>\r
-          <td><ww:property value="artifactId"/></td>\r
+          <td><ww:property value="artifactId" /></td>\r
         </tr>\r
         <tr align="left">\r
           <th>Version</th>\r
-          <td><ww:property value="version"/></td>\r
+          <td><ww:property value="version" /></td>\r
         </tr>\r
         <tr align="left">\r
           <th>Derivatives</th>\r
-          <td><ww:property value="groupId"/></td>\r
+          <td><ww:property value="groupId" /></td>\r
         </tr>\r
         <tr align="left">\r
           <th>Parent</th>\r
-          <td><ww:property value="folder"/></td>\r
+          <td><ww:property value="folder" /></td>\r
         </tr>\r
       </table>\r
-      <br/>\r
+      <br />\r
     </ww:iterator>\r
   </ww:if>\r
 </body>\r
index c4818db2d5b51ce718b9fe317cc226a3f45f98bf..23a934220cfb76f254cefaf30f7c58d65c17bd5d 100644 (file)
@@ -1,3 +1,19 @@
+<%--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  --%>
+
 <%@ taglib uri="http://www.opensymphony.com/sitemesh/decorator" prefix="decorator" %>
 <html>
 <head>
index 158fe9f61498211115daac88c3ee3d822d648d05..6049fe788645066722738f4ecf27b83ebcb984a1 100644 (file)
@@ -12,7 +12,6 @@
   ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   ~ See the License for the specific language governing permissions and
   ~ limitations under the License.
-  ~
   --%>
 
 <script type="text/javascript">
index 20287171255bbfe739521c249c2f40105d6a46cc..0ae0ab76856419e00a5540053e84879ce277a209 100644 (file)
@@ -1,3 +1,19 @@
+<%--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  --%>
+
 <%@ taglib uri="webwork" prefix="ww" %>
 <html>
 <head>
index 5a46febfc59bc470f1fc1eae4f36c1e1f56d60af..847a49e3d902b9a5a94634035f4b5a59dae7ffa7 100644 (file)
@@ -1,3 +1,19 @@
+<!--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+
 <!DOCTYPE web-app PUBLIC
     "-//Sun Microsystems, Inc.//DTD Web Application 2.3//EN"
     "http://java.sun.com/dtd/web-app_2_3.dtd" >
index 8adb0098b4b713e91be37ab0507ef3c2161b736b..9dc7da470c51eca8b6d0a09fd9a7ba0db1472197 100644 (file)
@@ -1,3 +1,19 @@
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 body {
     margin: 0px;
     padding: 0px;
index 886d4fc0fd9d78bcbcc311665b159ea6c86b3e60..d52c165120747685dc449f1f76c85dffb9de637d 100644 (file)
@@ -1,3 +1,19 @@
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 body {
     padding: 0px 0px 10px 0px;
 }
index 5695a5ac05cbc43be97285078bae7c82431e911f..9a6106d2d1784e9da220e0382b4877a9e29a403c 100644 (file)
@@ -1,3 +1,19 @@
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 #banner, #footer, #leftcol, #breadcrumbs, .docs #toc, .docs .courtesylinks, #leftColumn, #navColumn {
     display: none !important;
 }
index d13661b87366056c935e72826e30ed9fc4af13f1..8f0702b3af96522fbc2d0176aeac35c089bcb404 100644 (file)
@@ -1,3 +1,19 @@
+/*
+ * Copyright 2005-2006 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 #sidebar {
     float: right;
     font-size: small;
index 66a198330e9e910a4ab7f75c2517834cfd339cfb..867a05e7887862c87290995312eee48fbbbc87e1 100644 (file)
@@ -1 +1,17 @@
+<%--
+  ~ Copyright 2005-2006 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.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  --%>
+
 <%response.sendRedirect( request.getContextPath() + "/index.action" );%>
\ No newline at end of file