svn commit: r930242 - in /archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src: main/java/org/apache/maven/archiva/database/browsing/ test/java/org/apache/maven/archiva/database/browsing/

[oching]

Author: oching
Date: Fri Apr  2 11:18:43 2010
New Revision: 930242

URL: http://svn.apache.org/viewvc?rev=930242&view=rev
Log:
[MRM-1321] Snapshot is linking to oldest version in Archiva GUI
submitted by Daniel Triphaus
o do not reverse the order of the list containing the snapshot versions

additional change made:
o updated unit test to correct behavior

Modified:
    
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/main/java/org/apache/maven/archiva/database/browsing/DefaultRepositoryBrowsing.java
    
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/test/java/org/apache/maven/archiva/database/browsing/RepositoryBrowsingTest.java

Modified: 
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/main/java/org/apache/maven/archiva/database/browsing/DefaultRepositoryBrowsing.java
URL: 
http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/main/java/org/apache/maven/archiva/database/browsing/DefaultRepositoryBrowsing.java?rev=930242&r1=930241&r2=930242&view=diff
==============================================================================
--- 
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/main/java/org/apache/maven/archiva/database/browsing/DefaultRepositoryBrowsing.java
 (original)
+++ 
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/main/java/org/apache/maven/archiva/database/browsing/DefaultRepositoryBrowsing.java
 Fri Apr  2 11:18:43 2010
@@ -343,7 +343,6 @@ public class DefaultRepositoryBrowsing
         {
             final List<String> versions = (List<String>) dao.query( new 
UniqueVersionConstraint( groupId, artifactId ) );
             Collections.sort( versions );
-            Collections.reverse( versions );
 
             for ( String uniqueVersion : versions )
             {

Modified: 
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/test/java/org/apache/maven/archiva/database/browsing/RepositoryBrowsingTest.java
URL: 
http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/test/java/org/apache/maven/archiva/database/browsing/RepositoryBrowsingTest.java?rev=930242&r1=930241&r2=930242&view=diff
==============================================================================
--- 
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/test/java/org/apache/maven/archiva/database/browsing/RepositoryBrowsingTest.java
 (original)
+++ 
archiva/branches/archiva-1.3.x/archiva-modules/archiva-database/src/test/java/org/apache/maven/archiva/database/browsing/RepositoryBrowsingTest.java
 Fri Apr  2 11:18:43 2010
@@ -189,7 +189,7 @@ public class RepositoryBrowsingTest
         assertNotNull( "Artifact should not be null.", artifact );
         assertEquals( "org.apache.archiva", artifact.getGroupId() );
         assertEquals( "archiva-indexer", artifact.getArtifactId() );
-        assertEquals( "1.0-20070522.143249-1", artifact.getVersion() );       
+        assertEquals( "1.0-20070522.153141-2", artifact.getVersion() );       
         assertEquals( "jar", artifact.getPackaging() );
         
         String repoId = browser.getRepositoryId( USER_GUEST, GUEST_REPO_IDS, 
"org.apache.archiva", "archiva-indexer", "1.0-SNAPSHOT" );