You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by br...@apache.org on 2007/02/12 16:26:28 UTC

svn commit: r506484 - /maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java

Author: brett
Date: Mon Feb 12 07:26:28 2007
New Revision: 506484

URL: http://svn.apache.org/viewvc?view=rev&rev=506484
Log:
don't swallow exception

Modified:
    maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java

Modified: maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java
URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java?view=diff&rev=506484&r1=506483&r2=506484
==============================================================================
--- maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java (original)
+++ maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigurationAction.java Mon Feb 12 07:26:28 2007
@@ -96,31 +96,24 @@
     public String save()
         throws ConfigurationStoringException, ContinuumStoreException
     {
+        ConfigurationService configuration = getContinuum().getConfiguration();
 
-        try
-        {
-            ConfigurationService configuration = getContinuum().getConfiguration();
+        configuration.setWorkingDirectory( new File( workingDirectory ) );
 
-            configuration.setWorkingDirectory( new File( workingDirectory ) );
+        configuration.setBuildOutputDirectory( new File( buildOutputDirectory ) );
 
-            configuration.setBuildOutputDirectory( new File( buildOutputDirectory ) );
+        if ( StringUtils.isNotEmpty( deploymentRepositoryDirectory ) )
+        {
+            configuration.setDeploymentRepositoryDirectory( new File( deploymentRepositoryDirectory ) );
+        }
 
-            if ( StringUtils.isNotEmpty( deploymentRepositoryDirectory ) )
-            {
-                configuration.setDeploymentRepositoryDirectory( new File( deploymentRepositoryDirectory ) );
-            }
+        configuration.setUrl( baseUrl );
 
-            configuration.setUrl( baseUrl );
+        configuration.setInitialized( true );
 
-            configuration.setInitialized( true );
-            configuration.store();
-        }
-        catch ( Exception e )
-        {
-            e.printStackTrace();
-        }
-        return SUCCESS;
+        configuration.store();
 
+        return SUCCESS;
     }
 
     public String getWorkingDirectory()