removed usage of BidirectionalRepositoryLayout in database cleanup consumers, made...

git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@585068 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumer.java b/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumer.java
index 6d462c2fbac38e81960bee178bdc7f1a24d29e74..efdff176521a34d11c034ac6da8423937cffd5fa 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumer.java
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumer.java
@@ -28,9 +28,6 @@ import org.apache.maven.archiva.database.ArchivaDatabaseException;
 import org.apache.maven.archiva.repository.ManagedRepositoryContent;
 import org.apache.maven.archiva.repository.RepositoryContentFactory;
 import org.apache.maven.archiva.repository.RepositoryException;
-import org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayout;
-import org.apache.maven.archiva.repository.layout.LayoutException;
-import org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory;
 
 import java.util.List;
 import java.io.File;
@@ -64,11 +61,6 @@ public class DatabaseCleanupRemoveArtifactConsumer
      * @plexus.requirement role-hint="jdo"
      */
     private ArtifactDAO artifactDAO;
-
-    /**
-     * @plexus.requirement
-     */
-    private BidirectionalRepositoryLayoutFactory layoutFactory;
     
     /**
      * @plexus.requirement
@@ -98,9 +90,9 @@ public class DatabaseCleanupRemoveArtifactConsumer
        {
                ManagedRepositoryContent repositoryContent = 
                        repositoryFactory.getManagedRepositoryContent( artifact.getModel().getRepositoryId() );
-        
-               File file = new File( repositoryContent.getRepoRoot(), toPath( artifact ) );
-               
+                       
+               File file = new File( repositoryContent.getRepoRoot(), repositoryContent.toPath( artifact ) );
+                               
                if( !file.exists() )
                {                            
                        artifactDAO.deleteArtifact( artifact );
@@ -136,30 +128,9 @@ public class DatabaseCleanupRemoveArtifactConsumer
     {
         this.artifactDAO = artifactDAO;
     }
-
-    public void setBidirectionalRepositoryLayoutFactory( BidirectionalRepositoryLayoutFactory layoutFactory )
-    {
-        this.layoutFactory = layoutFactory;
-    }
     
     public void setRepositoryFactory( RepositoryContentFactory repositoryFactory )
     {
         this.repositoryFactory = repositoryFactory;
-    }
-
-    private String toPath( ArchivaArtifact artifact )
-    {
-        try
-        {
-            BidirectionalRepositoryLayout layout = layoutFactory.getLayout( artifact );
-
-            return layout.toPath( artifact );
-        }
-        catch ( LayoutException e )
-        {
-            getLogger().warn( "Unable to calculate path for artifact: " + artifact );
-            return null;
-        }
-    }
-    
+    }     
 }

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumer.java b/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumer.java
index feb0ec67291a3cb3f42bc2e556414413e139b6ed..34343977456e19a524383b23b989e89988f3f433 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumer.java
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumer.java
@@ -28,14 +28,10 @@ import org.apache.maven.archiva.model.ArchivaProjectModel;
 import org.apache.maven.archiva.repository.ManagedRepositoryContent;
 import org.apache.maven.archiva.repository.RepositoryContentFactory;
 import org.apache.maven.archiva.repository.RepositoryException;
-import org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayout;
-import org.apache.maven.archiva.repository.layout.LayoutException;
-import org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory;
 import org.apache.maven.archiva.database.ProjectModelDAO;
 import org.apache.maven.archiva.database.ArchivaDatabaseException;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.io.File;
 
 /**
@@ -68,11 +64,6 @@ public class DatabaseCleanupRemoveProjectConsumer
      * @plexus.requirement role-hint="jdo"
      */
     private ProjectModelDAO projectModelDAO;
-
-    /**
-     * @plexus.requirement
-     */
-    private BidirectionalRepositoryLayoutFactory layoutFactory;
     
     /**
      * @plexus.requirement
@@ -108,8 +99,8 @@ public class DatabaseCleanupRemoveProjectConsumer
                ManagedRepositoryContent repositoryContent = 
                        repositoryFactory.getManagedRepositoryContent( artifact.getModel().getRepositoryId() );
                
-               File file = new File( repositoryContent.getRepoRoot(), toPath( artifact ) );
-               
+               File file = new File( repositoryContent.getRepoRoot(), repositoryContent.toPath( artifact ) );
+                               
                if( !file.exists() )
                {                       
                        ArchivaProjectModel projectModel = projectModelDAO.getProjectModel( 
@@ -144,31 +135,11 @@ public class DatabaseCleanupRemoveProjectConsumer
     {
         return false;
     }    
-
-    private String toPath( ArchivaArtifact artifact )
-    {
-        try
-        {
-            BidirectionalRepositoryLayout layout = layoutFactory.getLayout( artifact );
-
-            return layout.toPath( artifact );
-        }
-        catch ( LayoutException e )
-        {
-            getLogger().warn( "Unable to calculate path for artifact: " + artifact );
-            return null;
-        }
-    }
-
+    
     public void setProjectModelDAO( ProjectModelDAO projectModelDAO )
     {
         this.projectModelDAO = projectModelDAO;
     }
-
-    public void setBidirectionalRepositoryLayoutFactory( BidirectionalRepositoryLayoutFactory layoutFactory )
-    {
-        this.layoutFactory = layoutFactory;
-    }
     
     public void setRepositoryFactory( RepositoryContentFactory repositoryFactory )
     {

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/AbstractDatabaseCleanupTest.java b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/AbstractDatabaseCleanupTest.java
index e49ff497b271051abeebb771e219be4dfab228bf..f4bb42990dde81f860365c06654fc86407fd5ab9 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/AbstractDatabaseCleanupTest.java
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/AbstractDatabaseCleanupTest.java
@@ -25,7 +25,6 @@ import org.apache.maven.archiva.configuration.ArchivaConfiguration;
 import org.apache.maven.archiva.configuration.Configuration;
 import org.apache.maven.archiva.configuration.ManagedRepositoryConfiguration;
 import org.apache.maven.archiva.repository.RepositoryContentFactory;
-import org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory;
 import org.apache.maven.archiva.model.ArchivaArtifact;
 import org.apache.maven.archiva.model.ArchivaArtifactModel;
 import org.apache.maven.archiva.model.ArchivaProjectModel;
@@ -39,8 +38,6 @@ public class AbstractDatabaseCleanupTest
     extends PlexusTestCase
 {
     ArchivaConfiguration archivaConfig;
-
-    BidirectionalRepositoryLayoutFactory layoutFactory;
     
     RepositoryContentFactory repositoryFactory;
 
@@ -73,9 +70,6 @@ public class AbstractDatabaseCleanupTest
         repo.setLocation( new File( getBasedir(), "src/test/resources/test-repo" ).toString() );
 
         archivaConfig.save( configuration );
-
-        // set bidirectional repository layout factory
-        layoutFactory = (BidirectionalRepositoryLayoutFactory) lookup( BidirectionalRepositoryLayoutFactory.class );
         
         repositoryFactory = (RepositoryContentFactory) lookup( RepositoryContentFactory.class );
     }

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.java b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.java
index 1e4932129f2888d1aa5be862064f40e6f19a1d72..08747a8d42f8986fdc858b40a9c73b54ebf6dbc3 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.java
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.java
@@ -50,8 +50,6 @@ public class DatabaseCleanupRemoveArtifactConsumerTest
 
         dbCleanupRemoveArtifactConsumer.setArtifactDAO( artifactDAOMock );
         
-        dbCleanupRemoveArtifactConsumer.setBidirectionalRepositoryLayoutFactory( layoutFactory );
-        
         dbCleanupRemoveArtifactConsumer.setRepositoryFactory( repositoryFactory );
     }
 

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.java b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.java
index 646d3b0c8245881e5e8fca833c4cc1b5ec8ce459..0e454e08e19ccd5b52b32f951dba0241ad5cc9db 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.java
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/java/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.java
@@ -51,8 +51,6 @@ public class DatabaseCleanupRemoveProjectConsumerTest
 
         dbCleanupRemoveProjectConsumer.setProjectModelDAO( projectModelDAOMock );
         
-        dbCleanupRemoveProjectConsumer.setBidirectionalRepositoryLayoutFactory( layoutFactory );
-        
         dbCleanupRemoveProjectConsumer.setRepositoryFactory( repositoryFactory );
     }
 

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.xml b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.xml
index 1a2bc20935836ba1473bf8070b7716029e756ce9..712381c1c0570a3fea1e0728aca146e3ae82436d 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.xml
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveArtifactConsumerTest.xml
@@ -20,24 +20,6 @@
 
 <component-set>
   <components>
-    <component>
-      <role>org.apache.maven.archiva.consumers.DatabaseCleanupConsumer</role>
-      <role-hint>not-present-remove-db-artifact</role-hint>
-      <implementation>org.apache.maven.archiva.consumers.database.DatabaseCleanupRemoveArtifactConsumer</implementation>
-      <requirements>
-        <requirement>
-          <role>org.apache.maven.archiva.configuration.ArchivaConfiguration</role>
-          <role-hint>database-cleanup</role-hint>
-        </requirement>
-        <requirement>
-          <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory</role>
-        </requirement>
-        <requirement>
-          <role>org.apache.maven.archiva.database.ArtifactDAO</role>
-          <role-hint>jdo</role-hint>
-        </requirement>
-      </requirements>
-    </component>
     <component>
       <role>org.apache.maven.archiva.configuration.ArchivaConfiguration</role>
       <role-hint>database-cleanup</role-hint>
@@ -60,25 +42,6 @@
         </properties>
       </configuration>
     </component>
-    <component>
-      <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory</role>
-      <implementation>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory</implementation>
-      <requirements>
-        <requirement>
-          <role>org.apache.maven.archiva.configuration.ArchivaConfiguration</role>
-          <role-hint>database-cleanup</role-hint>
-        </requirement>
-        <requirement>
-          <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayout</role>
-          <field-name>layouts</field-name>
-        </requirement>
-      </requirements>
-    </component>
-    <component>
-      <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayout</role>
-      <role-hint>default</role-hint>
-      <implementation>org.apache.maven.archiva.repository.layout.DefaultBidirectionalRepositoryLayout</implementation>
-    </component>
     
     <component>
       <role>org.apache.maven.archiva.repository.RepositoryContentFactory</role>      

diff --git a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.xml b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.xml
index ded6624478a5c70dc3a5b82080b08bc5811fe8cb..712381c1c0570a3fea1e0728aca146e3ae82436d 100644 (file)
--- a/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.xml
+++ b/archiva-base/archiva-consumers/archiva-database-consumers/src/test/resources/org/apache/maven/archiva/consumers/database/DatabaseCleanupRemoveProjectConsumerTest.xml
@@ -42,25 +42,6 @@
         </properties>
       </configuration>
     </component>
-    <component>
-      <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory</role>
-      <implementation>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayoutFactory</implementation>
-      <requirements>
-        <requirement>
-          <role>org.apache.maven.archiva.configuration.ArchivaConfiguration</role>
-          <role-hint>database-cleanup</role-hint>
-        </requirement>
-        <requirement>
-          <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayout</role>
-          <field-name>layouts</field-name>
-        </requirement>
-      </requirements>
-    </component>
-    <component>
-      <role>org.apache.maven.archiva.repository.layout.BidirectionalRepositoryLayout</role>
-      <role-hint>default</role-hint>
-      <implementation>org.apache.maven.archiva.repository.layout.DefaultBidirectionalRepositoryLayout</implementation>
-    </component>
     
     <component>
       <role>org.apache.maven.archiva.repository.RepositoryContentFactory</role>