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()