You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2015/01/04 18:33:56 UTC

svn commit: r1649372 - /maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependencyManagementRenderer.java

Author: michaelo
Date: Sun Jan  4 17:33:56 2015
New Revision: 1649372

URL: http://svn.apache.org/r1649372
Log:
[MPIR-247] "Comparison method violates its general contract!" while generating site

Modified:
    maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependencyManagementRenderer.java

Modified: maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependencyManagementRenderer.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependencyManagementRenderer.java?rev=1649372&r1=1649371&r2=1649372&view=diff
==============================================================================
--- maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependencyManagementRenderer.java (original)
+++ maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependencyManagementRenderer.java Sun Jan  4 17:33:56 2015
@@ -233,7 +233,7 @@ public class DependencyManagementRendere
 
                 List<ArtifactVersion> versions =
                     artifactMetadataSource.retrieveAvailableVersions( artifact, localRepository, remoteRepositories );
-    
+
                 // only use versions from range
                 for ( Iterator<ArtifactVersion> iter = versions.iterator(); iter.hasNext(); )
                 {
@@ -246,9 +246,9 @@ public class DependencyManagementRendere
                 // select latest, assuming pom information will be the most accurate
                 if ( versions.size() > 0 )
                 {
-                    Collections.sort( versions );
-                    
-                    artifact.setVersion( versions.get( versions.size() - 1 ).toString() );
+                    ArtifactVersion maxArtifactVersion = Collections.max( versions );
+
+                    artifact.setVersion( maxArtifactVersion.toString() );
                     log.debug( "DependencyManagement resolved: " + artifact.getId() );
                 }
             }