You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ol...@apache.org on 2008/06/13 15:24:15 UTC

svn commit: r667525 - /continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/CheckoutProjectContinuumAction.java

Author: olamy
Date: Fri Jun 13 06:24:14 2008
New Revision: 667525

URL: http://svn.apache.org/viewvc?rev=667525&view=rev
Log:
fix checkout with empty password

Modified:
    continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/CheckoutProjectContinuumAction.java

Modified: continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/CheckoutProjectContinuumAction.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/CheckoutProjectContinuumAction.java?rev=667525&r1=667524&r2=667525&view=diff
==============================================================================
--- continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/CheckoutProjectContinuumAction.java (original)
+++ continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/CheckoutProjectContinuumAction.java Fri Jun 13 06:24:14 2008
@@ -90,8 +90,8 @@
 
         try
         {
-            String scmUserName = getString( context, KEY_SCM_USERNAME );
-            String scmPassword = getString( context, KEY_SCM_PASSWORD );
+            String scmUserName = getString( context, KEY_SCM_USERNAME, "" );
+            String scmPassword = getString( context, KEY_SCM_PASSWORD, "" );
             ContinuumScmConfiguration config = createScmConfiguration( project, workingDirectory, scmUserName, scmPassword );
 
             String tag = config.getTag();
@@ -135,6 +135,8 @@
             result.setSuccess( false );
 
             result.setProviderMessage( e.getMessage() + ": " + getValidationMessages( e ) );
+            
+            getLogger().error( e.getMessage(), e);
         }
         catch ( NoSuchScmProviderException e )
         {
@@ -144,6 +146,8 @@
             result.setSuccess( false );
 
             result.setProviderMessage( e.getMessage() );
+            
+            getLogger().error( e.getMessage(), e);
         }
         catch ( ScmException e )
         {
@@ -152,6 +156,8 @@
             result.setSuccess( false );
 
             result.setException( ContinuumUtils.throwableMessagesToString( e ) );
+            
+            getLogger().error( e.getMessage(), e);
         }
         catch ( Throwable t )
         {
@@ -162,6 +168,8 @@
             result.setSuccess( false );
 
             result.setException( ContinuumUtils.throwableMessagesToString( t ) );
+            
+            getLogger().error( t.getMessage(), t);
         }
         finally
         {