You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2012/01/10 23:31:31 UTC
svn commit: r1229771 -
/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
Author: rfscholte
Date: Tue Jan 10 22:31:31 2012
New Revision: 1229771
URL: http://svn.apache.org/viewvc?rev=1229771&view=rev
Log:
Add generics
Modified:
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
Modified: maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java?rev=1229771&r1=1229770&r2=1229771&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java (original)
+++ maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/CheckDependencySnapshotsPhase.java Tue Jan 10 22:31:31 2012
@@ -91,10 +91,8 @@ public class CheckDependencySnapshotsPha
Map originalVersions = releaseDescriptor.getOriginalVersions( reactorProjects );
- for ( Iterator i = reactorProjects.iterator(); i.hasNext(); )
+ for ( MavenProject project : reactorProjects )
{
- MavenProject project = (MavenProject) i.next();
-
checkProject( project, originalVersions, releaseDescriptor );
}
}
@@ -110,12 +108,13 @@ public class CheckDependencySnapshotsPha
private void checkProject( MavenProject project, Map originalVersions, ReleaseDescriptor releaseDescriptor )
throws ReleaseFailureException, ReleaseExecutionException
{
- Map artifactMap = ArtifactUtils.artifactMapByVersionlessId( project.getArtifacts() );
+ @SuppressWarnings("unchecked")
+ Map<String, Artifact> artifactMap = ArtifactUtils.artifactMapByVersionlessId( project.getArtifacts() );
- Set snapshotDependencies = new HashSet();
- Set snapshotReportDependencies = new HashSet();
- Set snapshotExtensionsDependencies = new HashSet();
- Set snapshotPluginDependencies = new HashSet();
+ Set<Artifact> snapshotDependencies = new HashSet<Artifact>();
+ Set<Artifact> snapshotReportDependencies = new HashSet<Artifact>();
+ Set<Artifact> snapshotExtensionsDependencies = new HashSet<Artifact>();
+ Set<Artifact> snapshotPluginDependencies = new HashSet<Artifact>();
if ( project.getParentArtifact() != null )
{
@@ -127,12 +126,11 @@ public class CheckDependencySnapshotsPha
try
{
- Set dependencyArtifacts = project.createArtifacts( artifactFactory, null, null );
+ @SuppressWarnings("unchecked")
+ Set<Artifact> dependencyArtifacts = project.createArtifacts( artifactFactory, null, null );
- for ( Iterator i = dependencyArtifacts.iterator(); i.hasNext(); )
+ for ( Artifact artifact : dependencyArtifacts )
{
- Artifact artifact = (Artifact) i.next();
-
if ( checkArtifact( artifact, originalVersions, artifactMap, releaseDescriptor ) )
{
snapshotDependencies.add( getArtifactFromMap( artifact, artifactMap ) );
@@ -144,10 +142,11 @@ public class CheckDependencySnapshotsPha
throw new ReleaseExecutionException( "Failed to create dependency artifacts", e );
}
- for ( Iterator i = project.getPluginArtifacts().iterator(); i.hasNext(); )
+ @SuppressWarnings("unchecked")
+ Set<Artifact> pluginArtifacts = project.getPluginArtifacts();
+
+ for ( Artifact artifact : pluginArtifacts )
{
- Artifact artifact = (Artifact) i.next();
-
if ( checkArtifact( artifact, originalVersions, artifactMap, releaseDescriptor ) )
{
boolean addToFailures;
@@ -210,10 +209,11 @@ public class CheckDependencySnapshotsPha
}
}
- for ( Iterator i = project.getReportArtifacts().iterator(); i.hasNext(); )
+ @SuppressWarnings("unchecked")
+ Set<Artifact> reportArtifacts = project.getReportArtifacts();
+
+ for ( Artifact artifact : reportArtifacts )
{
- Artifact artifact = (Artifact) i.next();
-
if ( checkArtifact( artifact, originalVersions, artifactMap, releaseDescriptor ) )
{
//snapshotDependencies.add( artifact );
@@ -221,10 +221,11 @@ public class CheckDependencySnapshotsPha
}
}
- for ( Iterator i = project.getExtensionArtifacts().iterator(); i.hasNext(); )
+ @SuppressWarnings("unchecked")
+ Set<Artifact> extensionArtifacts = project.getExtensionArtifacts();
+
+ for ( Artifact artifact : extensionArtifacts )
{
- Artifact artifact = (Artifact) i.next();
-
if ( checkArtifact( artifact, originalVersions, artifactMap, releaseDescriptor ) )
{
snapshotExtensionsDependencies.add( artifact );
@@ -257,17 +258,17 @@ public class CheckDependencySnapshotsPha
}
}
- private static boolean checkArtifact( Artifact artifact, Map originalVersions, Map artifactMapByVersionlessId, ReleaseDescriptor releaseDescriptor )
+ private static boolean checkArtifact( Artifact artifact, Map originalVersions, Map<String, Artifact> artifactMapByVersionlessId, ReleaseDescriptor releaseDescriptor )
{
Artifact checkArtifact = getArtifactFromMap( artifact, artifactMapByVersionlessId );
return checkArtifact( checkArtifact, originalVersions, releaseDescriptor );
}
- private static Artifact getArtifactFromMap( Artifact artifact, Map artifactMapByVersionlessId )
+ private static Artifact getArtifactFromMap( Artifact artifact, Map<String, Artifact> artifactMapByVersionlessId )
{
String versionlessId = ArtifactUtils.versionlessKey( artifact );
- Artifact checkArtifact = (Artifact) artifactMapByVersionlessId.get( versionlessId );
+ Artifact checkArtifact = artifactMapByVersionlessId.get( versionlessId );
if ( checkArtifact == null)
{
@@ -307,16 +308,14 @@ public class CheckDependencySnapshotsPha
this.prompter = prompter;
}
- private StringBuffer printSnapshotDependencies( Set snapshotsSet, StringBuffer message )
+ private StringBuffer printSnapshotDependencies( Set<Artifact> snapshotsSet, StringBuffer message )
{
- List snapshotsList = new ArrayList( snapshotsSet );
+ List<Artifact> snapshotsList = new ArrayList<Artifact>( snapshotsSet );
Collections.sort( snapshotsList );
- for ( Iterator i = snapshotsList.iterator(); i.hasNext(); )
+ for ( Artifact artifact : snapshotsList )
{
- Artifact artifact = (Artifact) i.next();
-
message.append( " " );
message.append( artifact );
@@ -327,8 +326,8 @@ public class CheckDependencySnapshotsPha
return message;
}
- private void resolveSnapshots( Set projectDependencies, Set reportDependencies, Set extensionDependencies,
- Set pluginDependencies, ReleaseDescriptor releaseDescriptor )
+ private void resolveSnapshots( Set<Artifact> projectDependencies, Set<Artifact> reportDependencies, Set<Artifact> extensionDependencies,
+ Set<Artifact> pluginDependencies, ReleaseDescriptor releaseDescriptor )
throws ReleaseExecutionException
{
try
@@ -339,7 +338,7 @@ public class CheckDependencySnapshotsPha
if ( result.toLowerCase( Locale.ENGLISH ).startsWith( "y" ) )
{
- Map resolvedSnapshots = null;
+ Map<String, Map<String, String>> resolvedSnapshots = null;
prompter.showMessage( RESOLVE_SNAPSHOT_TYPE_MESSAGE );
result = prompter.prompt( RESOLVE_SNAPSHOT_TYPE_PROMPT,
Arrays.asList( new String[]{"0", "1", "2", "3"} ), "1" );
@@ -395,18 +394,18 @@ public class CheckDependencySnapshotsPha
}
}
- private Map processSnapshot( Set snapshotSet )
+ private Map<String, Map<String, String>> processSnapshot( Set<Artifact> snapshotSet )
throws PrompterException, VersionParseException
{
- Map resolvedSnapshots = new HashMap();
- Iterator iterator = snapshotSet.iterator();
+ Map<String, Map<String, String>> resolvedSnapshots = new HashMap<String, Map<String, String>>();
+ Iterator<Artifact> iterator = snapshotSet.iterator();
while ( iterator.hasNext() )
{
- Artifact currentArtifact = (Artifact) iterator.next();
+ Artifact currentArtifact = iterator.next();
String versionlessKey = ArtifactUtils.versionlessKey( currentArtifact );
- Map versionMap = new HashMap();
+ Map<String, String> versionMap = new HashMap<String, String>();
VersionInfo versionInfo = new DefaultVersionInfo( currentArtifact.getVersion() );
versionMap.put( ReleaseDescriptor.ORIGINAL_VERSION, versionInfo.toString() );
@@ -440,4 +439,4 @@ public class CheckDependencySnapshotsPha
return resolvedSnapshots;
}
-}
+}
\ No newline at end of file