Apache
/
archiva
mirror of https://github.com/apache/archiva.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[MRM-425 and MRM-426] 

- Added and modified code for handling snapshots (if the versions of a specific artifact snapshot are only timestamped versions, add a 
generic snapshot which is pointing to the latest timestamp version) in DefaultRepositoryBrowsing and ProjectModelToDatabaseConsumer.
- Updated pom validations in ProjectModelToDatabaseConsumer - handling of timestamped versions were considered
- Added isUniqueSnapshot(..) and isGenericSnapshot(..) in VersionUtil
- Added new attribute 'modelVersion' in DependencyTreeTag to get the in-pom version. Did not use the version attribute so as to retain the 
actual version being browsed. Also updated DependencyTree
- Updated the ff. pages for the version to be displayed: artifactInfo.jspf, showArtifact.jsp, dependencyTree.jsp and artifactDecorator.jsp
- Updated the version in SearchResultHit



git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@558795 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Maria Odea B. Ching 2007-07-23 16:48:30 +00:00
parent d972255f00
commit 488303a9e8
11 changed files with 236 additions and 103 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
archiva-base/archiva-common/src/main/java/org/apache/maven/archiva/common/utils/VersionUtil.java
View File

@ -151,4 +151,20 @@ public class VersionUtil
return version;
}
}
public static boolean isUniqueSnapshot( String version )
{
Matcher m = UNIQUE_SNAPSHOT_PATTERN.matcher( version );
if( m.matches() )
{
return true;
}
return false;
}
public static boolean isGenericSnapshot( String version )
{
return version.endsWith( SNAPSHOT );
}
}

71
archiva-base/archiva-consumers/archiva-database-consumers/src/main/java/org/apache/maven/archiva/consumers/database/ProjectModelToDatabaseConsumer.java
View File

@ -42,6 +42,7 @@ import org.apache.maven.archiva.repository.project.ProjectModelException;
import org.apache.maven.archiva.repository.project.ProjectModelFilter;
import org.apache.maven.archiva.repository.project.ProjectModelReader;
import org.apache.maven.archiva.repository.project.filters.EffectiveProjectModelFilter;
import org.apache.maven.archiva.common.utils.VersionUtil;
import java.io.File;
import java.util.ArrayList;
@ -52,7 +53,6 @@ import java.util.List;
*
* @author <a href="mailto:joakim@erdfelt.com">Joakim Erdfelt</a>
* @version $Id$
*
* @plexus.component role="org.apache.maven.archiva.consumers.DatabaseUnprocessedArtifactConsumer"
* role-hint="update-db-project"
* instantiation-strategy="per-lookup"
@ -102,8 +102,7 @@ public class ProjectModelToDatabaseConsumer
private ProjectModelFilter expressionModelFilter;
/**
* @plexus.requirement
* role="org.apache.maven.archiva.repository.project.ProjectModelFilter"
* @plexus.requirement role="org.apache.maven.archiva.repository.project.ProjectModelFilter"
* role-hint="effective"
*/
private EffectiveProjectModelFilter effectiveModelFilter;
@ -167,13 +166,24 @@ public class ProjectModelToDatabaseConsumer
// Resolve the project model
model = effectiveModelFilter.filter( model );
// The version should be updated to the filename version if it is a unique snapshot
FilenameParts parts = RepositoryLayoutUtils.splitFilename( artifactFile.getName(), null );
if ( model.getVersion().equals( VersionUtil.getBaseVersion( parts.version ) ) &&
VersionUtil.isUniqueSnapshot( parts.version ) )
{
model.setVersion( parts.version );
}
if ( isValidModel( model, artifact ) )
{
getLogger().info( "Add project model " + model + " to database." );
dao.getProjectModelDAO().saveProjectModel( model );
}
else
{
getLogger().warn( "Invalid or corrupt pom. Project model " + model + " was not added in the database." );
getLogger().warn(
"Invalid or corrupt pom. Project model " + model + " was not added in the database." );
}
}
@ -191,9 +201,8 @@ public class ProjectModelToDatabaseConsumer
catch ( Throwable t )
{
// Catch the other errors in the process to allow the rest of the process to complete.
getLogger().error(
"Unable to process model " + artifactFile + " due to : " + t.getClass().getName()
+ " : " + t.getMessage(), t );
getLogger().error( "Unable to process model " + artifactFile + " due to : " + t.getClass().getName() +
" : " + t.getMessage(), t );
}
}
@ -279,51 +288,26 @@ public class ProjectModelToDatabaseConsumer
try
{
FilenameParts parts = RepositoryLayoutUtils.splitFilename( artifactFile.getName(), null );
if ( !parts.artifactId.equalsIgnoreCase( model.getArtifactId() ) )
{
getLogger().warn(
"Project Model " + model + " artifactId: " + model.getArtifactId()
+ " does not match the pom file's artifactId: " + parts.artifactId );
getLogger().warn( "Project Model " + model + " artifactId: " + model.getArtifactId() +
" does not match the pom file's artifactId: " + parts.artifactId );
addProblem( artifact, "Project Model " + model + " artifactId: " + model.getArtifactId()
+ " does not match the pom file's artifactId: " + parts.artifactId );
addProblem( artifact, "Project Model " + model + " artifactId: " + model.getArtifactId() +
" does not match the pom file's artifactId: " + parts.artifactId );
return false;
}
if ( !parts.version.equalsIgnoreCase( model.getVersion() ) )
if ( !parts.version.equalsIgnoreCase( model.getVersion() ) &&
!VersionUtil.getBaseVersion( parts.version ).equalsIgnoreCase( model.getVersion() ) )
{
getLogger().warn(
"Project Model " + model + " version: " + model.getVersion()
+ " does not match the pom file's version: " + parts.version );
getLogger().warn( "Project Model " + model + " version: " + model.getVersion() +
" does not match the pom file's version: " + parts.version );
addProblem( artifact, "Project Model " + model + " version: " + model.getVersion()
+ " does not match the pom file's version: " + parts.version );
return false;
}
String constructedFilename;
if( parts.classifier != null )
{
constructedFilename = model.getArtifactId() + "-" + model.getVersion() + "-" +
parts.classifier.trim() + ".pom";
}
else
{
constructedFilename = model.getArtifactId() + "-" + model.getVersion() + ".pom";
}
//check if the file name matches the values indicated in the pom
if ( !artifactFile.getName().equalsIgnoreCase( constructedFilename ) )
{
getLogger().warn(
"Artifact " + artifact + " does not match the artifactId and/or version "
+ "specified in the project model " + model );
addProblem( artifact, "Artifact " + artifact + " does not match the artifactId and/or version "
+ "specified in the project model " + model );
addProblem( artifact, "Project Model " + model + " version: " + model.getVersion() +
" does not match the pom file's version: " + parts.version );
return false;
}
@ -361,4 +345,5 @@ public class ProjectModelToDatabaseConsumer
throw new ConsumerException( emsg, e );
}
}
}

2
archiva-base/archiva-indexer/src/main/java/org/apache/maven/archiva/indexer/search/SearchResultHit.java
View File

@ -91,7 +91,7 @@ public class SearchResultHit
{
this.artifacts.add( artifact );
String ver = artifact.getBaseVersion();
String ver = artifact.getVersion();
if ( !this.versions.contains( ver ) )
{

194
archiva-database/src/main/java/org/apache/maven/archiva/database/browsing/DefaultRepositoryBrowsing.java
View File

@ -32,17 +32,20 @@ import org.apache.maven.archiva.database.constraints.UniqueVersionConstraint;
import org.apache.maven.archiva.database.updater.DatabaseUpdater;
import org.apache.maven.archiva.model.ArchivaArtifact;
import org.apache.maven.archiva.model.ArchivaProjectModel;
import org.apache.maven.archiva.common.utils.VersionUtil;
import org.codehaus.plexus.logging.AbstractLogEnabled;
import java.util.Collections;
import java.util.List;
import java.util.Iterator;
import java.util.Map;
import java.util.HashMap;
/**
* DefaultRepositoryBrowsing
*
* @author <a href="mailto:joakim@erdfelt.com">Joakim Erdfelt</a>
* @version $Id$
*
* @plexus.component role="org.apache.maven.archiva.database.browsing.RepositoryBrowsing"
*/
public class DefaultRepositoryBrowsing
@ -77,6 +80,8 @@ public class DefaultRepositoryBrowsing
BrowsingResults results = new BrowsingResults( groupId, artifactId );
processSnapshots( versions );
results.setVersions( versions );
return results;
@ -101,60 +106,50 @@ public class DefaultRepositoryBrowsing
public ArchivaProjectModel selectVersion( String groupId, String artifactId, String version )
throws ObjectNotFoundException, ArchivaDatabaseException
{
ArchivaArtifact pomArtifact = getArtifact( groupId, artifactId, version );
ArchivaProjectModel model;
version = pomArtifact.getVersion();
if ( !pomArtifact.getModel().isProcessed() )
{
// Process it.
dbUpdater.updateUnprocessed( pomArtifact );
}
model = getProjectModel( groupId, artifactId, version );
return model;
}
private ArchivaArtifact getArtifact( String groupId, String artifactId, String version )
throws ObjectNotFoundException, ArchivaDatabaseException
{
ArchivaArtifact pomArtifact = null;
try
{
pomArtifact = dao.getArtifactDAO().getArtifact( groupId, artifactId, version, null, "pom" );
}
catch ( ObjectNotFoundException e )
{
pomArtifact = handleGenericSnapshots( groupId, artifactId, version, pomArtifact );
}
if ( pomArtifact == null )
{
throw new ObjectNotFoundException( "Unable to find artifact [" + groupId + ":" + artifactId + ":"
+ version + "]" );
}
}
catch ( ObjectNotFoundException e )
{
throw e;
}
ArchivaProjectModel model;
if ( pomArtifact.getModel().isProcessed() )
{
// It's been processed. return it.
model = dao.getProjectModelDAO().getProjectModel( groupId, artifactId, version );
return model;
}
// Process it.
dbUpdater.updateUnprocessed( pomArtifact );
// Find it.
try
{
model = dao.getProjectModelDAO().getProjectModel( groupId, artifactId, version );
if ( model == null )
{
throw new ObjectNotFoundException( "Unable to find project model for [" + groupId + ":" + artifactId
+ ":" + version + "]" );
}
return model;
}
catch ( ObjectNotFoundException e )
{
throw e;
throw new ObjectNotFoundException(
"Unable to find artifact [" + groupId + ":" + artifactId + ":" + version + "]" );
}
return pomArtifact;
}
public List getUsedBy( String groupId, String artifactId, String version )
throws ArchivaDatabaseException
{
ProjectsByArtifactUsageConstraint constraint = new ProjectsByArtifactUsageConstraint( groupId, artifactId,
version );
ProjectsByArtifactUsageConstraint constraint =
new ProjectsByArtifactUsageConstraint( groupId, artifactId, version );
List results = dao.getProjectModelDAO().queryProjectModels( constraint );
if ( results == null )
{
@ -164,4 +159,123 @@ public class DefaultRepositoryBrowsing
return results;
}
/**
* Add generic (*-SNAPSHOT) snapshot versions in the list for artifacts with only unique version (timestamped)
* snapshots.
* <p/>
* Ex.
* artifact1 has the ff. versions retrieved from the db:
* - 1.0
* - 1.1-20061118.060401-2
* - 1.1-20061118.060402-3
* - 2.2-20071007.070101-1
* - 2.2-20071007.070110-2
* - 2.2-SNAPSHOT
* <p/>
* This method will add a '1.1-SNAPSHOT' in the list since there is no generic snapshot entry for it.
* When this version is browsed, the pom of the latest snapshot will be displayed.
*
* @param versions
*/
private void processSnapshots( List versions )
{
Map snapshots = new HashMap();
getLogger().info( "Processing snapshots." );
for ( Iterator iter = versions.iterator(); iter.hasNext(); )
{
String version = (String) iter.next();
if ( VersionUtil.isSnapshot( version ) )
{
String baseVersion = VersionUtil.getBaseVersion( version );
if ( !snapshots.containsKey( baseVersion ) )
{
snapshots.put( baseVersion, baseVersion );
}
}
}
for ( Iterator it = ( snapshots.entrySet() ).iterator(); it.hasNext(); )
{
String baseVersion = (String) ( (Map.Entry) it.next() ).getValue();
if ( !versions.contains( baseVersion ) )
{
versions.add( baseVersion );
}
}
}
/**
* Handles querying of generic (*-SNAPSHOT) snapshot version.
* Process:
* - Get all the timestamped/unique versions of the artifact from the db
* - Sort the queried project models
* - Reverse the list of queried project models to get the latest timestamp version
* - Loop through the list and get the first one to match the generic (*-SNAPHOT) version
*
* @param groupId
* @param artifactId
* @param version
* @param pomArtifact
* @throws ArchivaDatabaseException
*/
private ArchivaArtifact handleGenericSnapshots( String groupId, String artifactId, String version,
ArchivaArtifact pomArtifact )
throws ArchivaDatabaseException
{
if ( VersionUtil.isGenericSnapshot( version ) )
{
List versions = dao.query( new UniqueVersionConstraint( groupId, artifactId ) );
Collections.sort( versions );
Collections.reverse( versions );
for ( Iterator iter = versions.iterator(); iter.hasNext(); )
{
String uniqueVersion = (String) iter.next();
if ( VersionUtil.getBaseVersion( uniqueVersion ).equals( version ) )
{
getLogger().info( "Retrieving artifact with version " + uniqueVersion );
pomArtifact = dao.getArtifactDAO().getArtifact( groupId, artifactId, uniqueVersion, null, "pom" );
return pomArtifact;
}
}
}
return null;
}
/**
* Get the project model from the database.
*
* @param groupId
* @param artifactId
* @param version
* @return
* @throws ArchivaDatabaseException
*/
private ArchivaProjectModel getProjectModel( String groupId, String artifactId, String version )
throws ArchivaDatabaseException
{
try
{
ArchivaProjectModel model = dao.getProjectModelDAO().getProjectModel( groupId, artifactId, version );
if ( model == null )
{
throw new ObjectNotFoundException(
"Unable to find project model for [" + groupId + ":" + artifactId + ":" + version + "]" );
}
return model;
}
catch ( ObjectNotFoundException e )
{
throw e;
}
}
}

12
archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/tags/DependencyTree.java
View File

@ -113,7 +113,7 @@ public class DependencyTree
}
}
public List gatherTreeList( String groupId, String artifactId, String version, String nodevar,
public List gatherTreeList( String groupId, String artifactId, String modelVersion, String nodevar,
PageContext pageContext )
throws JspException
{
@ -124,7 +124,7 @@ public class DependencyTree
throw new JspException( emsg );
}
DependencyGraph graph = fetchGraph( groupId, artifactId, version );
DependencyGraph graph = fetchGraph( groupId, artifactId, modelVersion );
if ( graph == null )
{
@ -213,18 +213,20 @@ public class DependencyTree
}
}
private DependencyGraph fetchGraph( String groupId, String artifactId, String version )
private DependencyGraph fetchGraph( String groupId, String artifactId, String modelVersion )
{
// TODO Cache the results to disk, in XML format, in the same place as the artifact is located.
VersionedReference projectRef = new VersionedReference();
projectRef.setGroupId( groupId );
projectRef.setArtifactId( artifactId );
projectRef.setVersion( version );
projectRef.setVersion( modelVersion );
try
{
return graphFactory.getGraph( projectRef );
DependencyGraph depGraph = graphFactory.getGraph( projectRef );
return depGraph;
}
catch ( GraphTaskException e )
{

9
archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/tags/DependencyTreeTag.java
View File

@ -68,6 +68,8 @@ public class DependencyTreeTag
private TreeEntry currentTreeEntry;
private String modelVersion;
public int doAfterBody()
throws JspException
{
@ -123,7 +125,7 @@ public class DependencyTreeTag
nodevar = "node";
}
this.tree = deptree.gatherTreeList( groupId, artifactId, version, nodevar, pageContext );
this.tree = deptree.gatherTreeList( groupId, artifactId, modelVersion, nodevar, pageContext );
if ( CollectionUtils.isEmpty( this.tree ) )
{
@ -172,6 +174,11 @@ public class DependencyTreeTag
this.version = version;
}
public void setModelVersion( String modelVersion )
{
this.modelVersion = modelVersion;
}
private void exposeVariables()
throws JspException
{

8
archiva-web/archiva-webapp/src/main/resources/META-INF/taglib.tld
View File

@ -126,6 +126,14 @@
<description><![CDATA[The version]]></description>
</attribute>
<attribute>
<name>modelVersion</name>
<required>false</required>
<rtexprvalue>true</rtexprvalue>
<description><![CDATA[The version of the project model. Used to verify the dependency graph for generic snapshots not yet in the repo.]]></description>
</attribute>
<attribute>
<name>nodevar</name>
<required>false</required>

3
archiva-web/archiva-webapp/src/main/webapp/WEB-INF/jsp/artifact/dependencyTree.jsp
View File

@ -21,7 +21,8 @@
<%@ taglib prefix="my" tagdir="/WEB-INF/tags" %>
<%@ taglib prefix="archiva" uri="http://maven.apache.org/archiva" %>
<archiva:dependency-tree groupId="${groupId}" artifactId="${artifactId}" version="${version}">
<archiva:dependency-tree groupId="${groupId}" artifactId="${artifactId}" version="${version}"
modelVersion="${model.version}">
<my:showArtifactLink groupId="${node.groupId}" artifactId="${node.artifactId}"
version="${node.version}"/>
</archiva:dependency-tree>

2
archiva-web/archiva-webapp/src/main/webapp/WEB-INF/jsp/decorators/artifactDecorator.jsp
View File

@ -137,7 +137,7 @@
</div>
<div class="sidebar3">
<archiva:downloadArtifact groupId="${groupId}" artifactId="${artifactId}" version="${Version}" />
<archiva:downloadArtifact groupId="${groupId}" artifactId="${artifactId}" version="${model.version}" />
</div>
<decorator:body />

4
archiva-web/archiva-webapp/src/main/webapp/WEB-INF/jsp/include/artifactInfo.jspf
View File

@ -31,7 +31,7 @@
</ww:url>
</c:set>
<a href="${url}">${model.artifactId}</a> /
<strong>${model.version}</strong>
<strong>${version}</strong>
<%-- TODO: new versions?
(<strong class="statusFailed">Newer version available:</strong>
@ -110,7 +110,7 @@
&lt;dependency>
&lt;groupId>${model.groupId}&lt;/groupId>
&lt;artifactId>${model.artifactId}&lt;/artifactId>
&lt;version>${model.version}&lt;/version><c:if test="${model.packaging != 'jar'}">
&lt;version>${version}&lt;/version><c:if test="${model.packaging != 'jar'}">
&lt;type>${model.packaging}&lt;/type></c:if>
&lt;/dependency>
</pre>

2
archiva-web/archiva-webapp/src/main/webapp/WEB-INF/jsp/showArtifact.jsp
View File

@ -133,7 +133,7 @@
</div>
<div class="sidebar3">
<archiva:downloadArtifact groupId="${groupId}" artifactId="${artifactId}" version="${Version}" />
<archiva:downloadArtifact groupId="${model.groupId}" artifactId="${model.artifactId}" version="${model.version}" />
</div>
<%-- TODO: perhaps using ajax? --%>