You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ep...@apache.org on 2006/01/03 10:55:41 UTC

svn commit: r365594 - in /maven/repository-manager/trunk/maven-repository-indexer/src: main/java/org/apache/maven/repository/indexing/ test/java/org/apache/maven/repository/indexing/

Author: epunzalan
Date: Tue Jan  3 01:55:29 2006
New Revision: 365594

URL: http://svn.apache.org/viewcvs?rev=365594&view=rev
Log:
Updated ArtifactRepositoryIndexSearcher to use factory

Modified:
    maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexSearcher.java
    maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java
    maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearcher.java
    maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexingTest.java

Modified: maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexSearcher.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexSearcher.java?rev=365594&r1=365593&r2=365594&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexSearcher.java (original)
+++ maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexSearcher.java Tue Jan  3 01:55:29 2006
@@ -33,8 +33,6 @@
 /**
  * This class searches the index for existing artifacts that contains the
  * specified query string.
- *
- * @plexus.component role="org.apache.maven.repository.indexing.RepositoryIndexSearcher" role-hint="artifact"
  */
 public class ArtifactRepositoryIndexSearcher
     implements RepositoryIndexSearcher
@@ -47,10 +45,15 @@
 
     private static final String VERSION = "version";
 
-    /**
-     * @plexus.requirement
-     */
     private ArtifactFactory factory;
+    
+    private ArtifactRepositoryIndex index;
+
+    public ArtifactRepositoryIndexSearcher( ArtifactRepositoryIndex index, ArtifactFactory factory )
+    {
+        this.factory = factory;
+        this.index = index;
+    }
 
     /**
      * Search the artifact that contains the query string in the specified
@@ -59,7 +62,7 @@
      * @param queryString
      * @param searchField
      */
-    public List search( RepositoryIndex index, String queryString, String searchField )
+    public List search( String queryString, String searchField )
         throws RepositoryIndexSearchException
     {
         List artifactList = new ArrayList();

Modified: maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java?rev=365594&r1=365593&r2=365594&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java (original)
+++ maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/DefaultRepositoryIndexingFactory.java Tue Jan  3 01:55:29 2006
@@ -17,6 +17,7 @@
  * limitations under the License.
  */
 
+import org.apache.maven.artifact.factory.ArtifactFactory;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.repository.digest.Digester;
 
@@ -31,9 +32,12 @@
     /** @plexus.requirement */
     private Digester digester;
     
-    public ArtifactRepositoryIndexSearcher createArtifactRepositoryIndexSearcher(ArtifactRepositoryIndex index)
+    /** @plexus.requirement */
+    private ArtifactFactory artifactFactory;
+    
+    public ArtifactRepositoryIndexSearcher createArtifactRepositoryIndexSearcher( ArtifactRepositoryIndex index )
     {
-        return null;
+        return new ArtifactRepositoryIndexSearcher( index, artifactFactory );
     }
 
     public ArtifactRepositoryIndex createArtifactRepositoryIndex( String indexPath, ArtifactRepository repository )

Modified: maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearcher.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearcher.java?rev=365594&r1=365593&r2=365594&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearcher.java (original)
+++ maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndexSearcher.java Tue Jan  3 01:55:29 2006
@@ -23,8 +23,6 @@
  */
 public interface RepositoryIndexSearcher
 {
-    String ROLE = RepositoryIndexSearcher.class.getName();
-
     /**
      * Search the artifact that contains the query string in the specified
      * search field.
@@ -33,6 +31,6 @@
      * @param queryString
      * @param searchField
      */
-    List search( RepositoryIndex index, String queryString, String searchField )
+    List search( String queryString, String searchField )
         throws RepositoryIndexSearchException;
 }

Modified: maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexingTest.java
URL: http://svn.apache.org/viewcvs/maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexingTest.java?rev=365594&r1=365593&r2=365594&view=diff
==============================================================================
--- maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexingTest.java (original)
+++ maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndexingTest.java Tue Jan  3 01:55:29 2006
@@ -162,36 +162,34 @@
         createTestIndex();
 
         RepositoryIndexingFactory factory = (RepositoryIndexingFactory) lookup( RepositoryIndexingFactory.ROLE );
-        ArtifactRepositoryIndex indexer = indexer = factory.createArtifactRepositoryIndex( indexPath, repository );
+        ArtifactRepositoryIndex indexer = factory.createArtifactRepositoryIndex( indexPath, repository );
+        RepositoryIndexSearcher repoSearcher = factory.createArtifactRepositoryIndexSearcher( indexer );
 
-        RepositoryIndexSearcher repoSearcher =
-            (RepositoryIndexSearcher) lookup( RepositoryIndexSearcher.ROLE, "artifact" );
-
-        List artifacts = repoSearcher.search( indexer, "test", GROUPID );
+        List artifacts = repoSearcher.search( "test", GROUPID );
         assertEquals( 1, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "test", ARTIFACTID );
+        artifacts = repoSearcher.search( "test", ARTIFACTID );
         assertEquals( 1, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "1.0", VERSION );
+        artifacts = repoSearcher.search( "1.0", VERSION );
         assertEquals( 1, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "App", CLASSES );
+        artifacts = repoSearcher.search( "App", CLASSES );
         assertEquals( 1, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "groupId", PACKAGES );
+        artifacts = repoSearcher.search( "groupId", PACKAGES );
         assertEquals( 1, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "pom.xml", FILES );
+        artifacts = repoSearcher.search( "pom.xml", FILES );
         assertEquals( 3, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "org.apache.maven", GROUPID );
+        artifacts = repoSearcher.search( "org.apache.maven", GROUPID );
         assertEquals( 2, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "maven-artifact", ARTIFACTID );
+        artifacts = repoSearcher.search( "maven-artifact", ARTIFACTID );
         assertEquals( 1, artifacts.size() );
 
-        artifacts = repoSearcher.search( indexer, "2", VERSION );
+        artifacts = repoSearcher.search( "2", VERSION );
         assertEquals( 2, artifacts.size() );
 
         indexer.close();