From 8a6d97d0fa0b338148fe6d35fc93c526d4727ce6 Mon Sep 17 00:00:00 2001
From: Brett Porter <brett@apache.org>
Date: Fri, 4 Aug 2006 11:10:50 +0000
Subject: [PATCH] [MRM-138] move local repository to a permanent location

git-svn-id: https://svn.apache.org/repos/asf/maven/repository-manager/trunk@428694 13f79535-47bb-0310-9956-ffa450edef68
---
 .../src/main/mdo/configuration.mdo                | 13 +++++++++++++
 .../ConfiguredRepositoryFactory.java              |  8 ++++++++
 .../DefaultConfiguredRepositoryFactory.java       |  6 ++++++
 maven-repository-webapp/pom.xml                   |  6 ++++++
 .../manager/web/action/ShowArtifactAction.java    | 15 ++-------------
 5 files changed, 35 insertions(+), 13 deletions(-)

diff --git a/maven-repository-configuration/src/main/mdo/configuration.mdo b/maven-repository-configuration/src/main/mdo/configuration.mdo
index 66ed605dd..423745d24 100644
--- a/maven-repository-configuration/src/main/mdo/configuration.mdo
+++ b/maven-repository-configuration/src/main/mdo/configuration.mdo
@@ -24,6 +24,14 @@
             <multiplicity>*</multiplicity>
           </association>
         </field>
+        <field>
+          <name>localRepository</name>
+          <version>1.0.0</version>
+          <type>String</type>
+          <description>
+            The location of the local repository.
+          </description>
+        </field>
         <field>
           <name>indexPath</name>
           <version>1.0.0</version>
@@ -67,6 +75,11 @@
         <codeSegment>
           <version>1.0.0</version>
           <code><![CDATA[
+    public Configuration()
+    {
+        localRepository = new java.io.File( System.getProperty( "user.home" ), ".m2/repository" ).getAbsolutePath();
+    }
+
     public boolean isValid()
     {
         boolean valid = true;
diff --git a/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/ConfiguredRepositoryFactory.java b/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/ConfiguredRepositoryFactory.java
index c6e0b011f..8e1641392 100644
--- a/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/ConfiguredRepositoryFactory.java
+++ b/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/ConfiguredRepositoryFactory.java
@@ -44,4 +44,12 @@ public interface ConfiguredRepositoryFactory
      * @return the artifact repositories
      */
     List createRepositories( Configuration configuration );
+
+    /**
+     * Create a local repository from the given configuration.
+     *
+     * @param configuration the configuration
+     * @return the local artifact repository
+     */
+    ArtifactRepository createLocalRepository( Configuration configuration );
 }
diff --git a/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/DefaultConfiguredRepositoryFactory.java b/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/DefaultConfiguredRepositoryFactory.java
index 7442242ff..54edb11e7 100644
--- a/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/DefaultConfiguredRepositoryFactory.java
+++ b/maven-repository-core/src/main/java/org/apache/maven/repository/configuration/DefaultConfiguredRepositoryFactory.java
@@ -65,4 +65,10 @@ public class DefaultConfiguredRepositoryFactory
 
         return repositories;
     }
+
+    public ArtifactRepository createLocalRepository( Configuration configuration )
+    {
+        ArtifactRepositoryLayout layout = (ArtifactRepositoryLayout) repositoryLayouts.get( "default" );
+        return repoFactory.createArtifactRepository( "local", configuration.getLocalRepository(), layout, null, null );
+    }
 }
diff --git a/maven-repository-webapp/pom.xml b/maven-repository-webapp/pom.xml
index 360ae9c89..e2d3da487 100644
--- a/maven-repository-webapp/pom.xml
+++ b/maven-repository-webapp/pom.xml
@@ -64,6 +64,12 @@
       <artifactId>plexus-log4j-logging</artifactId>
       <version>1.1-alpha-2</version>
     </dependency>
+    <dependency>
+      <groupId>org.apache.maven.wagon</groupId>
+      <artifactId>wagon-file</artifactId>
+      <version>1.0-beta-1</version>
+      <scope>runtime</scope>
+    </dependency>
     <dependency>
       <groupId>org.codehaus.plexus</groupId>
       <artifactId>plexus-container-default</artifactId>
diff --git a/maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/ShowArtifactAction.java b/maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/ShowArtifactAction.java
index defa8134f..b739f6aa9 100644
--- a/maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/ShowArtifactAction.java
+++ b/maven-repository-webapp/src/main/java/org/apache/maven/repository/manager/web/action/ShowArtifactAction.java
@@ -28,11 +28,9 @@ import org.apache.maven.repository.configuration.Configuration;
 import org.apache.maven.repository.configuration.ConfigurationStore;
 import org.apache.maven.repository.configuration.ConfigurationStoreException;
 import org.apache.maven.repository.configuration.ConfiguredRepositoryFactory;
-import org.apache.maven.repository.configuration.RepositoryConfiguration;
 import org.codehaus.plexus.util.StringUtils;
 import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
 
-import java.io.File;
 import java.io.IOException;
 import java.util.List;
 
@@ -101,23 +99,14 @@ public class ShowArtifactAction
 
         Artifact artifact = artifactFactory.createProjectArtifact( groupId, artifactId, version );
         // TODO: maybe we can decouple the assembly parts of the project builder from the repository handling to get rid of the temp repo
-        MavenProject project = projectBuilder.buildFromRepository( artifact, repositories, getLocalRepository() );
+        ArtifactRepository localRepository = repositoryFactory.createLocalRepository( configuration );
+        MavenProject project = projectBuilder.buildFromRepository( artifact, repositories, localRepository );
 
         model = project.getModel();
 
         return SUCCESS;
     }
 
-    private ArtifactRepository getLocalRepository()
-        throws IOException
-    {
-        // TODO: do we want this to be configurable?
-        RepositoryConfiguration configuration = new RepositoryConfiguration();
-        configuration.setId( "local" );
-        configuration.setDirectory( File.createTempFile( "repository", "local" ).getAbsolutePath() );
-        return repositoryFactory.createRepository( configuration );
-    }
-
     public Model getModel()
     {
         return model;