]> source.dussan.org Git - archiva.git/commitdiff
simplify debug code using vargars
authorOlivier Lamy <olamy@apache.org>
Mon, 15 Oct 2012 07:45:34 +0000 (07:45 +0000)
committerOlivier Lamy <olamy@apache.org>
Mon, 15 Oct 2012 07:45:34 +0000 (07:45 +0000)
git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1398206 13f79535-47bb-0310-9956-ffa450edef68

archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/metadata/repository/storage/maven2/RepositoryModelResolver.java

index a1c7009fc8da6da36a41993c7cad091248a0925f..142ba26b0973f3321ae984d68da6b9c99fe660bb 100644 (file)
@@ -91,7 +91,7 @@ public class RepositoryModelResolver
         this( new File( managedRepository.getLocation() ), pathTranslator );
 
         this.managedRepository = managedRepository;
-        
+
         this.wagonFactory = wagonFactory;
 
         this.remoteRepositories = remoteRepositories;
@@ -112,7 +112,7 @@ public class RepositoryModelResolver
         if ( !model.exists() )
         {
             /**
-             * 
+             *
              */
             // is a SNAPSHOT ? so we can try to find locally before asking remote repositories.
             if ( StringUtils.contains( version, VersionUtil.SNAPSHOT ) )
@@ -132,21 +132,22 @@ public class RepositoryModelResolver
                     boolean success = getModelFromProxy( remoteRepository, groupId, artifactId, version, filename );
                     if ( success && model.exists() )
                     {
-                        log.info(
-                            "Model '{}' successfully retrieved from remote repository '{}'", 
-                                model.getAbsolutePath(), remoteRepository.getId() );
+                        log.info( "Model '{}' successfully retrieved from remote repository '{}'",
+                                  model.getAbsolutePath(), remoteRepository.getId() );
                         break;
                     }
                 }
                 catch ( ResourceDoesNotExistException e )
                 {
-                    log.info( "An exception was caught while attempting to retrieve model '{}' from remote repository '{}'.Reason:{}",
-                                 new Object[]{ model.getAbsolutePath(), remoteRepository.getId(), e.getMessage() } );
+                    log.info(
+                        "An exception was caught while attempting to retrieve model '{}' from remote repository '{}'.Reason:{}",
+                        new Object[]{ model.getAbsolutePath(), remoteRepository.getId(), e.getMessage() } );
                 }
                 catch ( Exception e )
                 {
-                    log.warn( "An exception was caught while attempting to retrieve model '{}' from remote repository '{}'.Reason:{}",
-                                 new Object[]{ model.getAbsolutePath(), remoteRepository.getId(), e.getMessage() } );
+                    log.warn(
+                        "An exception was caught while attempting to retrieve model '{}' from remote repository '{}'.Reason:{}",
+                        new Object[]{ model.getAbsolutePath(), remoteRepository.getId(), e.getMessage() } );
 
                     continue;
                 }
@@ -176,11 +177,10 @@ public class RepositoryModelResolver
                         StringUtils.replaceChars( groupId, '.', '/' ) + '/' + artifactId + '/' + version + '/'
                             + artifactId + '-' + StringUtils.remove( version, "-SNAPSHOT" ) + '-' + lastVersion + '-'
                             + buildNumber + ".pom";
-                    if ( log.isDebugEnabled() )
-                    {
-                        log.debug( "use snapshot path {} for maven coordinate {}", snapshotPath,
-                                   groupId + ":" + artifactId + ":" + version );
-                    }
+
+                    log.debug( "use snapshot path {} for maven coordinate {}:{}:{}", snapshotPath, groupId, artifactId,
+                               version );
+
                     File model = new File( basedir, snapshotPath );
                     //model = pathTranslator.toFile( basedir, groupId, artifactId, lastVersion, filename );
                     if ( model.exists() )
@@ -401,12 +401,12 @@ public class RepositoryModelResolver
         }
         catch ( ConnectionException e )
         {
-            log.error( "Could not connect to {}:{} ",remoteRepository.getName(), e.getMessage() );
+            log.error( "Could not connect to {}:{} ", remoteRepository.getName(), e.getMessage() );
             connected = false;
         }
         catch ( AuthenticationException e )
         {
-            log.error( "Could not connect to {}:{} ",remoteRepository.getName(), e.getMessage() );
+            log.error( "Could not connect to {}:{} ", remoteRepository.getName(), e.getMessage() );
             connected = false;
         }
 
@@ -464,8 +464,8 @@ public class RepositoryModelResolver
                 {
                     if ( newLocation.exists() )
                     {
-                        log.error( "Tried to copy file {} to {} but file with this name already exists.", 
-                                        fileToMove.getName(), newLocation.getAbsolutePath() );
+                        log.error( "Tried to copy file {} to {} but file with this name already exists.",
+                                   fileToMove.getName(), newLocation.getAbsolutePath() );
                     }
                     else
                     {