This is an automated email from the ASF dual-hosted git repository.
martin_s pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/archiva.git
The following commit(s) were added to refs/heads/master by this push:
new b942314 Removing unused repository API method
b942314 is described below
commit b942314aa27330a179acf17300fea6849bf89d4d
Author: Martin Stockhammer <[email protected]>
AuthorDate: Sun Jun 7 19:32:53 2020 +0200
Removing unused repository API method
---
.../archiva/repository/RepositoryContent.java | 11 --------
.../mock/ManagedRepositoryContentMock.java | 6 -----
.../mock/RemoteRepositoryContentMock.java | 6 -----
.../scanner/mock/ManagedRepositoryContentMock.java | 24 -----------------
.../mock/ManagedRepositoryContentMock.java | 25 ------------------
.../mock/RemoteRepositoryContentMock.java | 6 -----
.../content/AbstractDefaultRepositoryContent.java | 7 -----
.../content/ManagedDefaultRepositoryContent.java | 30 ----------------------
.../content/RemoteDefaultRepositoryContent.java | 19 --------------
9 files changed, 134 deletions(-)
diff --git
a/archiva-modules/archiva-base/archiva-repository-api/src/main/java/org/apache/archiva/repository/RepositoryContent.java
b/archiva-modules/archiva-base/archiva-repository-api/src/main/java/org/apache/archiva/repository/RepositoryContent.java
index fd92be1..c30550e 100644
---
a/archiva-modules/archiva-base/archiva-repository-api/src/main/java/org/apache/archiva/repository/RepositoryContent.java
+++
b/archiva-modules/archiva-base/archiva-repository-api/src/main/java/org/apache/archiva/repository/RepositoryContent.java
@@ -31,17 +31,6 @@ public interface RepositoryContent
/**
- * Given a repository relative path to a filename, return the {@link
ArtifactReference} object suitable for the path.
- *
- * @param path the path relative to the repository base dir for the
artifact.
- * @return the {@link ArtifactReference} representing the path. (or null
if path cannot be converted to
- * a {@link ArtifactReference})
- * @throws LayoutException if there was a problem converting the path to
an artifact.
- */
- ArtifactReference toArtifactReference( String path )
- throws LayoutException;
-
- /**
* Given an {@link ArtifactReference}, return the relative path to the
artifact.
*
* @param reference the artifact reference to use.
diff --git
a/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
b/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
index bab8373..2a2bf39 100644
---
a/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
+++
b/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
@@ -322,12 +322,6 @@ public class ManagedRepositoryContentMock implements
BaseRepositoryContentLayout
}
@Override
- public ArtifactReference toArtifactReference( String path ) throws
LayoutException
- {
- return null;
- }
-
- @Override
public String toPath( ArtifactReference reference )
{
return null;
diff --git
a/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
b/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
index bdf951f..1b9a857 100644
---
a/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
+++
b/archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
@@ -60,12 +60,6 @@ public class RemoteRepositoryContentMock implements
RemoteRepositoryContent
}
@Override
- public ArtifactReference toArtifactReference( String path ) throws
LayoutException
- {
- return null;
- }
-
- @Override
public String toPath( ArtifactReference reference )
{
return null;
diff --git
a/archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/mock/ManagedRepositoryContentMock.java
b/archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/mock/ManagedRepositoryContentMock.java
index 5deb975..054d095 100644
---
a/archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/mock/ManagedRepositoryContentMock.java
+++
b/archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/mock/ManagedRepositoryContentMock.java
@@ -336,30 +336,6 @@ public class ManagedRepositoryContentMock implements
BaseRepositoryContentLayout
private Map<ArtifactReference, String> refs = new HashMap<>();
- @Override
- public ArtifactReference toArtifactReference( String path ) throws
LayoutException
- {
- if ( StringUtils.isBlank( path ) )
- {
- throw new LayoutException( "Unable to convert blank path." );
- }
-
- ArtifactMetadata metadata = getArtifactForPath("test-repository",
path);
-
- ArtifactReference artifact = new ArtifactReference();
- artifact.setGroupId( metadata.getNamespace() );
- artifact.setArtifactId( metadata.getProject() );
- artifact.setVersion( metadata.getVersion() );
- MavenArtifactFacet facet = (MavenArtifactFacet) metadata.getFacet(
MavenArtifactFacet.FACET_ID );
- if ( facet != null )
- {
- artifact.setClassifier( facet.getClassifier() );
- artifact.setType( facet.getType() );
- }
- refs.put(artifact, path);
- return artifact;
- }
-
public ArtifactMetadata getArtifactForPath( String repoId, String
relativePath )
{
String[] parts = relativePath.replace( '\\', '/' ).split( "/" );
diff --git
a/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
b/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
index 7048470..3730c20 100644
---
a/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
+++
b/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/ManagedRepositoryContentMock.java
@@ -369,31 +369,6 @@ public class ManagedRepositoryContentMock implements
BaseRepositoryContentLayout
private Map<ArtifactReference, String> refs = new HashMap<>();
- @Override
- public ArtifactReference toArtifactReference( String path ) throws
LayoutException
- {
- if ( StringUtils.isBlank( path ) )
- {
- throw new LayoutException( "Unable to convert blank path." );
- }
-
- ArtifactMetadata metadata = getArtifactForPath("test-repository",
path);
-
- ArtifactReference artifact = new ArtifactReference();
- artifact.setGroupId( metadata.getNamespace() );
- artifact.setArtifactId( metadata.getProject() );
- artifact.setVersion( metadata.getVersion() );
- artifact.setProjectVersion( metadata.getProjectVersion( ) );
- MavenArtifactFacet facet = (MavenArtifactFacet) metadata.getFacet(
MavenArtifactFacet.FACET_ID );
- if ( facet != null )
- {
- artifact.setClassifier( facet.getClassifier() );
- artifact.setType( facet.getType() );
- }
- refs.put(artifact, path);
- return artifact;
- }
-
public ArtifactMetadata getArtifactForPath( String repoId, String
relativePath )
{
String[] parts = relativePath.replace( '\\', '/' ).split( "/" );
diff --git
a/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
b/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
index 057d09a..168ed3d 100644
---
a/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
+++
b/archiva-modules/archiva-maven/archiva-maven-proxy/src/test/java/org/apache/archiva/repository/mock/RemoteRepositoryContentMock.java
@@ -66,12 +66,6 @@ public class RemoteRepositoryContentMock implements
RemoteRepositoryContent
}
@Override
- public ArtifactReference toArtifactReference( String path ) throws
LayoutException
- {
- return null;
- }
-
- @Override
public String toPath( ArtifactReference reference )
{
String baseVersion;
diff --git
a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/AbstractDefaultRepositoryContent.java
b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/AbstractDefaultRepositoryContent.java
index c43d401..2f2b2f7 100644
---
a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/AbstractDefaultRepositoryContent.java
+++
b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/AbstractDefaultRepositoryContent.java
@@ -78,13 +78,6 @@ public abstract class AbstractDefaultRepositoryContent
implements RepositoryCont
}
@Override
- public ArtifactReference toArtifactReference( String path )
- throws LayoutException
- {
- return defaultPathParser.toArtifactReference( path );
- }
-
- @Override
public ItemSelector toItemSelector( String path ) throws LayoutException
{
return defaultPathParser.toItemSelector( path );
diff --git
a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/ManagedDefaultRepositoryContent.java
b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/ManagedDefaultRepositoryContent.java
index eedd366..7341747 100644
---
a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/ManagedDefaultRepositoryContent.java
+++
b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/ManagedDefaultRepositoryContent.java
@@ -1597,36 +1597,6 @@ public class ManagedDefaultRepositoryContent
return repository.getRoot().getStorage( );
}
- /**
- * Convert a path to an artifact reference.
- *
- * @param path the path to convert. (relative or full location path)
- * @throws LayoutException if the path cannot be converted to an artifact
reference.
- */
- @Override
- public ArtifactReference toArtifactReference( String path )
- throws LayoutException
- {
- String repoPath = convertUriToPath( repository.getLocation( ) );
- if ( ( path != null ) && path.startsWith( repoPath ) &&
repoPath.length( ) > 0 )
- {
- return super.toArtifactReference( path.substring( repoPath.length(
) + 1 ) );
- }
- else
- {
- repoPath = path;
- if ( repoPath != null )
- {
- while ( repoPath.startsWith( "/" ) )
- {
- repoPath = repoPath.substring( 1 );
- }
- }
- return super.toArtifactReference( repoPath );
- }
- }
-
-
public void setFiletypes( FileTypes filetypes )
{
this.filetypes = filetypes;
diff --git
a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/RemoteDefaultRepositoryContent.java
b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/RemoteDefaultRepositoryContent.java
index 0e7ce73..abdefbf 100644
---
a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/RemoteDefaultRepositoryContent.java
+++
b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/maven/content/RemoteDefaultRepositoryContent.java
@@ -73,25 +73,6 @@ public class RemoteDefaultRepositoryContent
this.repository = repository;
}
- /**
- * Convert a path to an artifact reference.
- *
- * @param path the path to convert. (relative or full url path)
- * @throws LayoutException if the path cannot be converted to an artifact
reference.
- */
- @Override
- public ArtifactReference toArtifactReference( String path )
- throws LayoutException
- {
-
- if ( ( path != null ) && repository.getLocation()!=null &&
path.startsWith( repository.getLocation().toString() ) )
- {
- return super.toArtifactReference( path.substring(
repository.getLocation().toString().length( ) ) );
- }
-
- return super.toArtifactReference( path );
- }
-
@Override
public RepositoryURL toURL( ArtifactReference reference )
{