continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From br...@apache.org
Subject svn commit: r1372260 [29/30] - in /continuum/trunk: ./ continuum-api/ continuum-api/src/main/java/org/apache/continuum/builder/distributed/ continuum-api/src/main/java/org/apache/continuum/builder/distributed/executor/ continuum-api/src/main/java/org/a...
Date Mon, 13 Aug 2012 04:29:28 GMT
Modified: continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java?rev=1372260&r1=1372259&r2=1372260&view=diff
==============================================================================
--- continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java (original)
+++ continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumServiceImpl.java Mon Aug 13 04:29:09 2012
@@ -19,18 +19,8 @@ package org.apache.maven.continuum.xmlrp
  * under the License.
  */
 
-import java.io.File;
-import java.lang.reflect.Field;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Properties;
-import java.util.Set;
-
+import net.sf.dozer.util.mapping.DozerBeanMapperSingletonWrapper;
+import net.sf.dozer.util.mapping.MapperIF;
 import org.apache.commons.lang.StringEscapeUtils;
 import org.apache.commons.lang.StringUtils;
 import org.apache.continuum.buildagent.NoBuildAgentException;
@@ -90,8 +80,17 @@ import org.codehaus.plexus.redback.role.
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import net.sf.dozer.util.mapping.DozerBeanMapperSingletonWrapper;
-import net.sf.dozer.util.mapping.MapperIF;
+import java.io.File;
+import java.lang.reflect.Field;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Properties;
+import java.util.Set;
 
 /**
  * @author <a href="mailto:evenisse@apache.org">Emmanuel Venisse</a>
@@ -117,7 +116,7 @@ public class ContinuumServiceImpl
 
     private static final String PROJECT_ARTIFACT_ID_VALID_EXPRESSION = "[A-Za-z0-9\\-]*";
 
-    private static final String PROJECT_EXECUTOR_OR_BUILDDEF_TYPE_VALID_EXPRESSION = "maven2|maven-1|ant|shell";    
+    private static final String PROJECT_EXECUTOR_OR_BUILDDEF_TYPE_VALID_EXPRESSION = "maven2|maven-1|ant|shell";
 
     private static final String SCHEDULE_CRON_VALID_EXPRESSION = "[A-Z0-9\\s*/,-?#]*";
 
@@ -189,8 +188,8 @@ public class ContinuumServiceImpl
 
         List<ProjectSummary> projectsList = new ArrayList<ProjectSummary>();
 
-        Collection<org.apache.maven.continuum.model.project.Project> projects =
-            continuum.getProjectsInGroup( projectGroupId );
+        Collection<org.apache.maven.continuum.model.project.Project> projects = continuum.getProjectsInGroup(
+            projectGroupId );
         if ( projects != null )
         {
             for ( org.apache.maven.continuum.model.project.Project project : projects )
@@ -237,27 +236,26 @@ public class ContinuumServiceImpl
     public ProjectSummary updateProject( ProjectSummary project )
         throws ContinuumException
     {
-        if ( StringUtils.isNotBlank( project.getName() ) && 
-                        !project.getName().matches( NAME_VALID_EXPRESSION ) )
-                        //!GenericValidator.matchRegexp( project.getName(), NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( project.getName() ) && !project.getName().matches( NAME_VALID_EXPRESSION ) )
+        //!GenericValidator.matchRegexp( project.getName(), NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Name contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( project.getScmTag() ) && 
-                        !project.getScmTag().matches( PROJECT_SCM_TAG_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( project.getScmTag() ) && !project.getScmTag().matches(
+            PROJECT_SCM_TAG_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Scm Tag contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( project.getScmUrl() ) &&
-                        !project.getScmUrl().matches( PROJECT_SCM_URL_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( project.getScmUrl() ) && !project.getScmUrl().matches(
+            PROJECT_SCM_URL_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Scm Url contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( project.getVersion() ) &&
-                        !project.getVersion().matches( PROJECT_VERSION_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( project.getVersion() ) && !project.getVersion().matches(
+            PROJECT_VERSION_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Version contains invalid characters" );
         }
@@ -311,7 +309,8 @@ public class ContinuumServiceImpl
         throws ContinuumException
     {
         ProjectGroup result = null;
-        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroup( projectGroupId );
+        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroup(
+            projectGroupId );
         try
         {
             if ( isAuthorized( ContinuumRoleConstants.CONTINUUM_VIEW_GROUP_OPERATION, projectGroup.getName() ) )
@@ -365,7 +364,8 @@ public class ContinuumServiceImpl
     private ProjectGroupSummary getPGSummary( int projectGroupId )
         throws ContinuumException
     {
-        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroup( projectGroupId );
+        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroup(
+            projectGroupId );
 
         checkViewProjectGroupAuthorization( projectGroup.getName() );
         return populateProjectGroupSummary( projectGroup );
@@ -376,7 +376,8 @@ public class ContinuumServiceImpl
     {
         checkViewProjectGroupAuthorization( getProjectGroupName( projectGroupId ) );
 
-        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroup( projectGroupId );
+        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroup(
+            projectGroupId );
         return populateProjectGroupSummary( projectGroup );
     }
 
@@ -385,8 +386,8 @@ public class ContinuumServiceImpl
     {
         checkViewProjectGroupAuthorization( getProjectGroupName( projectGroupId ) );
 
-        org.apache.maven.continuum.model.project.ProjectGroup projectGroup =
-            continuum.getProjectGroupWithProjects( projectGroupId );
+        org.apache.maven.continuum.model.project.ProjectGroup projectGroup = continuum.getProjectGroupWithProjects(
+            projectGroupId );
         return populateProjectGroupWithAllDetails( projectGroup );
     }
 
@@ -423,8 +424,8 @@ public class ContinuumServiceImpl
             throw new ContinuumException( "ProjectGroup Name contains invalid characters" );
         }
 
-        org.apache.maven.continuum.model.project.ProjectGroup pg =
-            continuum.getProjectGroupWithProjects( projectGroup.getId() );
+        org.apache.maven.continuum.model.project.ProjectGroup pg = continuum.getProjectGroupWithProjects(
+            projectGroup.getId() );
 
         // need to administer roles since they are based off of this
         // todo convert everything like to work off of string keys
@@ -457,14 +458,12 @@ public class ContinuumServiceImpl
     public ProjectGroupSummary addProjectGroup( String groupName, String groupId, String description )
         throws Exception
     {
-        if ( StringUtils.isNotBlank( groupId ) &&
-                        !groupId.matches( PROJECTGROUP_ID_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( groupId ) && !groupId.matches( PROJECTGROUP_ID_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "ProjectGroup Id contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( groupName ) &&
-                        !groupName.matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( groupName ) && !groupName.matches( NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "ProjectGroup Name contains invalid characters" );
         }
@@ -487,14 +486,14 @@ public class ContinuumServiceImpl
     public ProjectNotifier updateNotifier( int projectid, ProjectNotifier newNotifier )
         throws ContinuumException
     {
-        if ( StringUtils.isNotBlank( newNotifier.getType() ) &&
-             !newNotifier.getType().matches( NOTIFIER_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( newNotifier.getType() ) && !newNotifier.getType().matches(
+            NOTIFIER_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Notifier Type can only be 'irc, jabber, msn, mail or wagon" );
         }
 
-        org.apache.maven.continuum.model.project.ProjectNotifier notifier =
-            continuum.getNotifier( projectid, newNotifier.getId() );
+        org.apache.maven.continuum.model.project.ProjectNotifier notifier = continuum.getNotifier( projectid,
+                                                                                                   newNotifier.getId() );
         notifier.setConfiguration( newNotifier.getConfiguration() );
         notifier.setFrom( newNotifier.getFrom() );
         notifier.setRecipientType( newNotifier.getRecipientType() );
@@ -510,8 +509,8 @@ public class ContinuumServiceImpl
     public ProjectNotifier addNotifier( int projectid, ProjectNotifier newNotifier )
         throws ContinuumException
     {
-        if ( StringUtils.isNotBlank( newNotifier.getType() ) &&
-                        !newNotifier.getType().matches( NOTIFIER_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( newNotifier.getType() ) && !newNotifier.getType().matches(
+            NOTIFIER_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Notifier Type can only be 'irc, jabber, msn, mail or wagon'" );
         }
@@ -546,14 +545,14 @@ public class ContinuumServiceImpl
     public ProjectNotifier updateGroupNotifier( int projectgroupid, ProjectNotifier newNotifier )
         throws ContinuumException
     {
-        if ( StringUtils.isNotBlank( newNotifier.getType() ) &&
-             !newNotifier.getType().matches( NOTIFIER_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( newNotifier.getType() ) && !newNotifier.getType().matches(
+            NOTIFIER_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Notifier Type can only be 'irc, jabber, msn, mail, or wagon" );
         }
 
-        org.apache.maven.continuum.model.project.ProjectNotifier notifier =
-            continuum.getGroupNotifier( projectgroupid, newNotifier.getId() );
+        org.apache.maven.continuum.model.project.ProjectNotifier notifier = continuum.getGroupNotifier( projectgroupid,
+                                                                                                        newNotifier.getId() );
         notifier.setConfiguration( newNotifier.getConfiguration() );
         notifier.setFrom( newNotifier.getFrom() );
         notifier.setRecipientType( newNotifier.getRecipientType() );
@@ -572,8 +571,8 @@ public class ContinuumServiceImpl
         org.apache.maven.continuum.model.project.ProjectNotifier notifier =
             new org.apache.maven.continuum.model.project.ProjectNotifier();
 
-        if ( StringUtils.isNotBlank( newNotifier.getType() ) &&
-                        !newNotifier.getType().matches( NOTIFIER_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( newNotifier.getType() ) && !newNotifier.getType().matches(
+            NOTIFIER_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Notifier Type can only be 'irc, jabber, msn, mail or wagon" );
         }
@@ -608,8 +607,8 @@ public class ContinuumServiceImpl
 
         checkViewProjectGroupAuthorization( ps.getProjectGroup().getName() );
 
-        List<org.apache.maven.continuum.model.project.BuildDefinition> bds =
-            continuum.getBuildDefinitionsForProject( projectId );
+        List<org.apache.maven.continuum.model.project.BuildDefinition> bds = continuum.getBuildDefinitionsForProject(
+            projectId );
 
         List<BuildDefinition> result = new ArrayList<BuildDefinition>();
         for ( org.apache.maven.continuum.model.project.BuildDefinition bd : bds )
@@ -658,8 +657,8 @@ public class ContinuumServiceImpl
         ProjectSummary ps = getProjectSummary( projectId );
 
         checkModifyProjectBuildDefinitionAuthorization( ps.getProjectGroup().getName() );
-        org.apache.maven.continuum.model.project.BuildDefinition newbd =
-            continuum.getBuildDefinition( buildDef.getId() );
+        org.apache.maven.continuum.model.project.BuildDefinition newbd = continuum.getBuildDefinition(
+            buildDef.getId() );
         org.apache.maven.continuum.model.project.BuildDefinition bd = populateBuildDefinition( buildDef, newbd );
         bd = continuum.updateBuildDefinitionForProject( projectId, bd );
         return populateBuildDefinition( bd );
@@ -669,8 +668,8 @@ public class ContinuumServiceImpl
         throws ContinuumException
     {
         checkModifyGroupBuildDefinitionAuthorization( getProjectGroupName( projectGroupId ) );
-        org.apache.maven.continuum.model.project.BuildDefinition newbd =
-            continuum.getBuildDefinition( buildDef.getId() );
+        org.apache.maven.continuum.model.project.BuildDefinition newbd = continuum.getBuildDefinition(
+            buildDef.getId() );
         org.apache.maven.continuum.model.project.BuildDefinition bd = populateBuildDefinition( buildDef, newbd );
         bd = continuum.updateBuildDefinitionForProjectGroup( projectGroupId, bd );
         return populateBuildDefinition( bd );
@@ -734,10 +733,8 @@ public class ContinuumServiceImpl
         ProjectSummary ps = getProjectSummary( projectId );
         checkBuildProjectInGroupAuthorization( ps.getProjectGroup().getName() );
 
-        continuum.buildProject( projectId,
-                                new org.apache.continuum.utils.build.BuildTrigger(
-                                                                                   ContinuumProjectState.TRIGGER_SCHEDULED,
-                                                                                   "" ) );
+        continuum.buildProject( projectId, new org.apache.continuum.utils.build.BuildTrigger(
+            ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
         return 0;
     }
 
@@ -748,11 +745,8 @@ public class ContinuumServiceImpl
         ProjectSummary ps = getProjectSummary( projectId );
         checkBuildProjectInGroupAuthorization( ps.getProjectGroup().getName() );
 
-        continuum.buildProject( projectId,
-                                buildDefinitionId,
-                                new org.apache.continuum.utils.build.BuildTrigger(
-                                                                                   ContinuumProjectState.TRIGGER_SCHEDULED,
-                                                                                   "" ) );
+        continuum.buildProject( projectId, buildDefinitionId, new org.apache.continuum.utils.build.BuildTrigger(
+            ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
         return 0;
     }
 
@@ -762,10 +756,8 @@ public class ContinuumServiceImpl
         ProjectSummary ps = getProjectSummary( projectId );
         checkBuildProjectInGroupAuthorization( ps.getProjectGroup().getName() );
 
-        continuum.buildProject( projectId,
-                                new org.apache.continuum.utils.build.BuildTrigger(
-                                                                                   ContinuumProjectState.TRIGGER_SCHEDULED,
-                                                                                   "" ) );
+        continuum.buildProject( projectId, new org.apache.continuum.utils.build.BuildTrigger(
+            ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
         return 0;
     }
 
@@ -776,7 +768,8 @@ public class ContinuumServiceImpl
         checkBuildProjectInGroupAuthorization( ps.getProjectGroup().getName() );
 
         buildProjectWithBuildDefinition( projectId, buildDefinitionId,
-                                         new org.apache.continuum.utils.build.BuildTrigger( ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
+                                         new org.apache.continuum.utils.build.BuildTrigger(
+                                             ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
         return 0;
     }
 
@@ -810,10 +803,8 @@ public class ContinuumServiceImpl
         ProjectGroupSummary pg = getProjectGroupSummary( projectGroupId );
         checkBuildProjectInGroupAuthorization( pg.getName() );
 
-        continuum.buildProjectGroup( projectGroupId,
-                                     new org.apache.continuum.utils.build.BuildTrigger(
-                                                                                        ContinuumProjectState.TRIGGER_SCHEDULED,
-                                                                                        "" ) );
+        continuum.buildProjectGroup( projectGroupId, new org.apache.continuum.utils.build.BuildTrigger(
+            ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
 
         return 0;
     }
@@ -824,11 +815,9 @@ public class ContinuumServiceImpl
         ProjectGroupSummary pg = getProjectGroupSummary( projectGroupId );
         checkBuildProjectInGroupAuthorization( pg.getName() );
 
-        continuum.buildProjectGroupWithBuildDefinition( projectGroupId,
-                                                        buildDefinitionId,
+        continuum.buildProjectGroupWithBuildDefinition( projectGroupId, buildDefinitionId,
                                                         new org.apache.continuum.utils.build.BuildTrigger(
-                                                                                                           ContinuumProjectState.TRIGGER_SCHEDULED,
-                                                                                                           "" ) );
+                                                            ContinuumProjectState.TRIGGER_SCHEDULED, "" ) );
 
         return 0;
     }
@@ -860,8 +849,8 @@ public class ContinuumServiceImpl
     public ProjectScmRoot getProjectScmRootByProject( int projectId )
         throws ContinuumException
     {
-        org.apache.continuum.model.project.ProjectScmRoot projectScmRoot =
-            continuum.getProjectScmRootByProject( projectId );
+        org.apache.continuum.model.project.ProjectScmRoot projectScmRoot = continuum.getProjectScmRootByProject(
+            projectId );
 
         checkViewProjectGroupAuthorization( projectScmRoot.getProjectGroup().getName() );
 
@@ -878,8 +867,8 @@ public class ContinuumServiceImpl
         ProjectSummary ps = getProjectSummary( projectId );
         checkViewProjectGroupAuthorization( ps.getProjectGroup().getName() );
 
-        org.apache.maven.continuum.model.project.BuildResult buildResult =
-            continuum.getLatestBuildResultForProject( projectId );
+        org.apache.maven.continuum.model.project.BuildResult buildResult = continuum.getLatestBuildResultForProject(
+            projectId );
 
         if ( buildResult != null )
         {
@@ -921,7 +910,8 @@ public class ContinuumServiceImpl
     public int removeBuildResult( BuildResult br )
         throws ContinuumException
     {
-        checkModifyProjectGroupAuthorization( getProjectSummary( br.getProject().getId() ).getProjectGroup().getName() );
+        checkModifyProjectGroupAuthorization( getProjectSummary(
+            br.getProject().getId() ).getProjectGroup().getName() );
         continuum.removeBuildResult( br.getId() );
         return 0;
     }
@@ -965,14 +955,11 @@ public class ContinuumServiceImpl
         ContinuumProjectBuildingResult result = null;
         try
         {
-            result =
-                continuum.addMavenTwoProject( url,
-                                              projectGroupId,
-                                              true, // checkProtocol
-                                              false, // useCredentialsCache
-                                              true, // recursiveProjects
-                                              continuum.getBuildDefinitionService().getDefaultMavenTwoBuildDefinitionTemplate().getId(),
-                                              checkoutInSingleDirectory );
+            result = continuum.addMavenTwoProject( url, projectGroupId, true, // checkProtocol
+                                                   false, // useCredentialsCache
+                                                   true, // recursiveProjects
+                                                   continuum.getBuildDefinitionService().getDefaultMavenTwoBuildDefinitionTemplate().getId(),
+                                                   checkoutInSingleDirectory );
         }
         catch ( BuildDefinitionServiceException e )
         {
@@ -990,18 +977,15 @@ public class ContinuumServiceImpl
         ContinuumProjectBuildingResult result = null;
         try
         {
-            result =
-                continuum.addMavenTwoProject( url,
-                                              projectGroupId,
-                                              true, // checkProtocol
-                                              false, // useCredentialsCache
-                                              false, // recursiveProjects
-                                              continuum.getBuildDefinitionService().getDefaultMavenTwoBuildDefinitionTemplate().getId(),
-                                              true ); // a multi-module project added as a single project is always
-                                                      // checked out
-                                                      // in a single directory, regardless the value set for
-                                                      // checkoutInSingleDirectory
-                                                      // variable
+            result = continuum.addMavenTwoProject( url, projectGroupId, true, // checkProtocol
+                                                   false, // useCredentialsCache
+                                                   false, // recursiveProjects
+                                                   continuum.getBuildDefinitionService().getDefaultMavenTwoBuildDefinitionTemplate().getId(),
+                                                   true ); // a multi-module project added as a single project is always
+            // checked out
+            // in a single directory, regardless the value set for
+            // checkoutInSingleDirectory
+            // variable
         }
         catch ( BuildDefinitionServiceException e )
         {
@@ -1021,14 +1005,10 @@ public class ContinuumServiceImpl
         ContinuumProjectBuildingResult result = null;
         try
         {
-            result =
-                continuum.addMavenTwoProject( url,
-                                              projectGroupId,
-                                              checkProtocol,
-                                              useCredentialsCache,
-                                              recursiveProjects,
-                                              continuum.getBuildDefinitionService().getDefaultMavenTwoBuildDefinitionTemplate().getId(),
-                                              checkoutInSingleDirectory );
+            result = continuum.addMavenTwoProject( url, projectGroupId, checkProtocol, useCredentialsCache,
+                                                   recursiveProjects,
+                                                   continuum.getBuildDefinitionService().getDefaultMavenTwoBuildDefinitionTemplate().getId(),
+                                                   checkoutInSingleDirectory );
         }
         catch ( BuildDefinitionServiceException e )
         {
@@ -1061,9 +1041,8 @@ public class ContinuumServiceImpl
         checkAddProjectGroupAuthorization();
         org.apache.maven.continuum.model.project.Project newProject =
             new org.apache.maven.continuum.model.project.Project();
-        int projectId =
-            continuum.addProject( populateProject( project, newProject ),
-                                  ContinuumBuildExecutorConstants.ANT_BUILD_EXECUTOR, projectGroupId );
+        int projectId = continuum.addProject( populateProject( project, newProject ),
+                                              ContinuumBuildExecutorConstants.ANT_BUILD_EXECUTOR, projectGroupId );
         return getProjectSummary( projectId );
     }
 
@@ -1077,9 +1056,8 @@ public class ContinuumServiceImpl
         checkAddProjectGroupAuthorization();
         org.apache.maven.continuum.model.project.Project newProject =
             new org.apache.maven.continuum.model.project.Project();
-        int projectId =
-            continuum.addProject( populateProject( project, newProject ),
-                                  ContinuumBuildExecutorConstants.SHELL_BUILD_EXECUTOR, projectGroupId );
+        int projectId = continuum.addProject( populateProject( project, newProject ),
+                                              ContinuumBuildExecutorConstants.SHELL_BUILD_EXECUTOR, projectGroupId );
         return getProjectSummary( projectId );
     }
 
@@ -1177,8 +1155,8 @@ public class ContinuumServiceImpl
         throws ContinuumException
     {
         checkManageProfilesAuthorization();
-        org.apache.maven.continuum.model.system.Profile profile = 
-                        continuum.getProfileService().getProfileWithName( profileName );
+        org.apache.maven.continuum.model.system.Profile profile = continuum.getProfileService().getProfileWithName(
+            profileName );
         return profile != null ? populateProfile( profile ) : null;
     }
 
@@ -1194,8 +1172,8 @@ public class ContinuumServiceImpl
     public int updateProfile( Profile profile )
         throws ContinuumException
     {
-        org.apache.maven.continuum.model.system.Profile newProfile =
-            continuum.getProfileService().getProfile( profile.getId() );
+        org.apache.maven.continuum.model.system.Profile newProfile = continuum.getProfileService().getProfile(
+            profile.getId() );
 
         continuum.getProfileService().updateProfile( populateProfile( profile, newProfile ) );
         return 0;
@@ -1240,16 +1218,18 @@ public class ContinuumServiceImpl
     {
         try
         {
-            List<org.apache.maven.continuum.model.system.Installation> buildAgentInstallations = distributedBuildManager.getAvailableInstallations( url );
+            List<org.apache.maven.continuum.model.system.Installation> buildAgentInstallations =
+                distributedBuildManager.getAvailableInstallations( url );
 
             List<Installation> convertedBuildAgentInstallations = new ArrayList<Installation>();
             for ( Object buildAgentInstallation : buildAgentInstallations )
             {
-                convertedBuildAgentInstallations.add( populateInstallation( (org.apache.maven.continuum.model.system.Installation) buildAgentInstallation ) );
+                convertedBuildAgentInstallations.add( populateInstallation(
+                    (org.apache.maven.continuum.model.system.Installation) buildAgentInstallation ) );
             }
             return convertedBuildAgentInstallations;
         }
-        catch (Exception e)
+        catch ( Exception e )
         {
             throw new ContinuumException( "Can't load installations", e );
         }
@@ -1363,7 +1343,7 @@ public class ContinuumServiceImpl
     {
         return isProjectInPrepareBuildQueue( projectId, -1 );
     }
-    
+
     public boolean isProjectInPrepareBuildQueue( int projectId, int buildDefinitionId )
         throws ContinuumException
     {
@@ -1561,7 +1541,6 @@ public class ContinuumServiceImpl
                     }
                 }
 
-                
                 if ( dbm.isProjectInAnyBuildQueue( projectId, buildDefinitionId ) )
                 {
                     dbm.removeFromBuildQueue( buildAgentUrl, projectId, buildDefinitionId );
@@ -1598,8 +1577,8 @@ public class ContinuumServiceImpl
 
                 if ( parallelBuildsManager.isInPrepareBuildQueue( projectId ) )
                 {
-                    org.apache.continuum.model.project.ProjectScmRoot scmRoot =
-                        continuum.getProjectScmRootByProject( projectId );
+                    org.apache.continuum.model.project.ProjectScmRoot scmRoot = continuum.getProjectScmRootByProject(
+                        projectId );
                     parallelBuildsManager.removeProjectFromPrepareBuildQueue( scmRoot.getProjectGroup().getId(),
                                                                               scmRoot.getId() );
                 }
@@ -1632,8 +1611,8 @@ public class ContinuumServiceImpl
     public ContinuumReleaseResult getReleaseResult( int releaseId )
         throws ContinuumException
     {
-        org.apache.continuum.model.release.ContinuumReleaseResult releaseResult =
-            continuum.getContinuumReleaseResult( releaseId );
+        org.apache.continuum.model.release.ContinuumReleaseResult releaseResult = continuum.getContinuumReleaseResult(
+            releaseId );
         checkViewProjectGroupAuthorization( getProjectGroupName( releaseResult.getProjectGroup().getId() ) );
         return populateReleaseResult( releaseResult );
     }
@@ -1663,8 +1642,8 @@ public class ContinuumServiceImpl
     public String getReleaseOutput( int releaseId )
         throws ContinuumException
     {
-        org.apache.continuum.model.release.ContinuumReleaseResult releaseResult =
-            continuum.getContinuumReleaseResult( releaseId );
+        org.apache.continuum.model.release.ContinuumReleaseResult releaseResult = continuum.getContinuumReleaseResult(
+            releaseId );
         checkViewProjectGroupAuthorization( getProjectGroupName( releaseResult.getProjectGroup().getId() ) );
 
         return continuum.getReleaseOutput( releaseId );
@@ -1683,8 +1662,9 @@ public class ContinuumServiceImpl
         {
             org.apache.continuum.model.repository.RepositoryPurgeConfiguration newPurge =
                 new org.apache.continuum.model.repository.RepositoryPurgeConfiguration();
-            return populateRepositoryPurgeConfiguration( continuum.getPurgeConfigurationService().addRepositoryPurgeConfiguration( populateRepositoryPurgeConfiguration( repoPurge,
-                                                                                                                                                                         newPurge ) ) );
+            return populateRepositoryPurgeConfiguration(
+                continuum.getPurgeConfigurationService().addRepositoryPurgeConfiguration(
+                    populateRepositoryPurgeConfiguration( repoPurge, newPurge ) ) );
         }
         catch ( RepositoryServiceException e )
         {
@@ -1705,8 +1685,8 @@ public class ContinuumServiceImpl
         {
             org.apache.continuum.model.repository.RepositoryPurgeConfiguration purge =
                 new org.apache.continuum.model.repository.RepositoryPurgeConfiguration();
-            continuum.getPurgeConfigurationService().updateRepositoryPurgeConfiguration( populateRepositoryPurgeConfiguration( repoPurge,
-                                                                                                                               purge ) );
+            continuum.getPurgeConfigurationService().updateRepositoryPurgeConfiguration(
+                populateRepositoryPurgeConfiguration( repoPurge, purge ) );
             return 0;
         }
         catch ( RepositoryServiceException e )
@@ -1763,7 +1743,8 @@ public class ContinuumServiceImpl
         List<RepositoryPurgeConfiguration> r = new ArrayList<RepositoryPurgeConfiguration>();
         for ( Object repoPurgeConfig : repoPurgeConfigs )
         {
-            r.add( populateRepositoryPurgeConfiguration( (org.apache.continuum.model.repository.RepositoryPurgeConfiguration) repoPurgeConfig ) );
+            r.add( populateRepositoryPurgeConfiguration(
+                (org.apache.continuum.model.repository.RepositoryPurgeConfiguration) repoPurgeConfig ) );
         }
         return r;
     }
@@ -1777,8 +1758,9 @@ public class ContinuumServiceImpl
         {
             org.apache.continuum.model.repository.DirectoryPurgeConfiguration newPurge =
                 new org.apache.continuum.model.repository.DirectoryPurgeConfiguration();
-            return populateDirectoryPurgeConfiguration( continuum.getPurgeConfigurationService().addDirectoryPurgeConfiguration( populateDirectoryPurgeConfiguration( dirPurge,
-                                                                                                                                                                      newPurge ) ) );
+            return populateDirectoryPurgeConfiguration(
+                continuum.getPurgeConfigurationService().addDirectoryPurgeConfiguration(
+                    populateDirectoryPurgeConfiguration( dirPurge, newPurge ) ) );
         }
         catch ( RepositoryServiceException e )
         {
@@ -1799,8 +1781,8 @@ public class ContinuumServiceImpl
         {
             org.apache.continuum.model.repository.DirectoryPurgeConfiguration purge =
                 new org.apache.continuum.model.repository.DirectoryPurgeConfiguration();
-            continuum.getPurgeConfigurationService().updateDirectoryPurgeConfiguration( populateDirectoryPurgeConfiguration( dirPurge,
-                                                                                                                             purge ) );
+            continuum.getPurgeConfigurationService().updateDirectoryPurgeConfiguration(
+                populateDirectoryPurgeConfiguration( dirPurge, purge ) );
             return 0;
         }
         catch ( RepositoryServiceException e )
@@ -1857,7 +1839,8 @@ public class ContinuumServiceImpl
         List<DirectoryPurgeConfiguration> d = new ArrayList<DirectoryPurgeConfiguration>();
         for ( Object dirPurgeConfig : dirPurgeConfigs )
         {
-            d.add( populateDirectoryPurgeConfiguration( (org.apache.continuum.model.repository.DirectoryPurgeConfiguration) dirPurgeConfig ) );
+            d.add( populateDirectoryPurgeConfiguration(
+                (org.apache.continuum.model.repository.DirectoryPurgeConfiguration) dirPurgeConfig ) );
         }
         return d;
     }
@@ -1921,8 +1904,8 @@ public class ContinuumServiceImpl
         {
             org.apache.continuum.model.repository.LocalRepository newRepository =
                 new org.apache.continuum.model.repository.LocalRepository();
-            return populateLocalRepository( continuum.getRepositoryService().addLocalRepository( populateLocalRepository( repository,
-                                                                                                                          newRepository ) ) );
+            return populateLocalRepository( continuum.getRepositoryService().addLocalRepository(
+                populateLocalRepository( repository, newRepository ) ) );
         }
         catch ( RepositoryServiceException e )
         {
@@ -2021,26 +2004,26 @@ public class ContinuumServiceImpl
     public BuildAgentConfiguration getBuildAgent( String url )
     {
         ConfigurationService configurationService = continuum.getConfiguration();
-        org.apache.continuum.configuration.BuildAgentConfiguration buildAgent =
-            configurationService.getBuildAgent( url );
+        org.apache.continuum.configuration.BuildAgentConfiguration buildAgent = configurationService.getBuildAgent(
+            url );
         BuildAgentConfiguration buildAgentConfiguration = buildAgent != null ? populateBuildAgent( buildAgent ) : null;
         return buildAgentConfiguration;
     }
-    
+
     /**
      * {@inheritDoc}
-     * 
+     *
      * @throws ContinuumException distributed build is not enabled or error during retrieval of build agent url
-     * 
-     *  @see DistributedBuildManager#getBuildAgentUrl(int, int)
+     * @see DistributedBuildManager#getBuildAgentUrl(int, int)
      */
-    public String getBuildAgentUrl( int projectId, int buildDefinitionId ) throws ContinuumException
+    public String getBuildAgentUrl( int projectId, int buildDefinitionId )
+        throws ContinuumException
     {
         if ( !continuum.getConfiguration().isDistributedBuildEnabled() )
         {
             throw new ContinuumException( "Method available only in distributed build mode." );
         }
-        
+
         return distributedBuildManager.getBuildAgentUrl( projectId, buildDefinitionId );
     }
 
@@ -2050,13 +2033,15 @@ public class ContinuumServiceImpl
         try
         {
             ConfigurationService configurationService = continuum.getConfiguration();
-            org.apache.continuum.configuration.BuildAgentConfiguration buildAgent =
-                configurationService.getBuildAgent( buildAgentConfiguration.getUrl() );
-            BuildAgentConfiguration buildAgentConfigurationToUpdate = buildAgent != null ? populateBuildAgent( buildAgent ) : null;
-    
+            org.apache.continuum.configuration.BuildAgentConfiguration buildAgent = configurationService.getBuildAgent(
+                buildAgentConfiguration.getUrl() );
+            BuildAgentConfiguration buildAgentConfigurationToUpdate = buildAgent != null ? populateBuildAgent(
+                buildAgent ) : null;
+
             if ( buildAgentConfigurationToUpdate != null )
             {
-                buildAgentConfigurationToUpdate.setDescription( StringEscapeUtils.escapeXml( buildAgentConfiguration.getDescription() ) );
+                buildAgentConfigurationToUpdate.setDescription( StringEscapeUtils.escapeXml(
+                    buildAgentConfiguration.getDescription() ) );
                 buildAgentConfigurationToUpdate.setEnabled( buildAgentConfiguration.isEnabled() );
                 configurationService.updateBuildAgent( populateBuildAgent( buildAgentConfigurationToUpdate ) );
                 configurationService.store();
@@ -2080,32 +2065,34 @@ public class ContinuumServiceImpl
         ConfigurationService configurationService = continuum.getConfiguration();
 
         boolean SUCCESS = false;
-        org.apache.continuum.configuration.BuildAgentConfiguration buildAgent =
-            configurationService.getBuildAgent( url );
-        BuildAgentConfiguration buildAgentConfiguration =
-            buildAgent != null ? populateBuildAgent( buildAgent ) : null;
+        org.apache.continuum.configuration.BuildAgentConfiguration buildAgent = configurationService.getBuildAgent(
+            url );
+        BuildAgentConfiguration buildAgentConfiguration = buildAgent != null ? populateBuildAgent( buildAgent ) : null;
         if ( buildAgentConfiguration != null )
         {
             if ( continuum.getDistributedBuildManager().isBuildAgentBusy( buildAgentConfiguration.getUrl() ) )
             {
-                throw new BuildAgentConfigurationException( "Cannot delete build agent because it's busy at the moment" );
+                throw new BuildAgentConfigurationException(
+                    "Cannot delete build agent because it's busy at the moment" );
             }
 
             if ( configurationService.getBuildAgentGroups() != null )
             {
                 for ( org.apache.continuum.configuration.BuildAgentGroupConfiguration buildAgentGroup : configurationService.getBuildAgentGroups() )
                 {
-                    if ( configurationService.containsBuildAgentUrl( buildAgentConfiguration.getUrl(), buildAgentGroup ) )
+                    if ( configurationService.containsBuildAgentUrl( buildAgentConfiguration.getUrl(),
+                                                                     buildAgentGroup ) )
                     {
                         throw new BuildAgentConfigurationException(
-                                                                    "Cannot delete build agent because it's in use at the moment" );
+                            "Cannot delete build agent because it's in use at the moment" );
                     }
                 }
             }
 
             try
             {
-                continuum.getDistributedBuildManager().removeDistributedBuildQueueOfAgent( buildAgentConfiguration.getUrl() );
+                continuum.getDistributedBuildManager().removeDistributedBuildQueueOfAgent(
+                    buildAgentConfiguration.getUrl() );
                 configurationService.removeBuildAgent( populateBuildAgent( buildAgentConfiguration ) );
                 configurationService.store();
                 SUCCESS = true;
@@ -2196,11 +2183,12 @@ public class ContinuumServiceImpl
         ConfigurationService configurationService = continuum.getConfiguration();
         org.apache.continuum.configuration.BuildAgentGroupConfiguration buildAgentGroup =
             configurationService.getBuildAgentGroup( name );
-        BuildAgentGroupConfiguration buildAgentGroupConfiguration = buildAgentGroup != null ? populateBuildAgentGroup( buildAgentGroup ) : null;
+        BuildAgentGroupConfiguration buildAgentGroupConfiguration = buildAgentGroup != null ? populateBuildAgentGroup(
+            buildAgentGroup ) : null;
         return buildAgentGroupConfiguration;
     }
 
-    public BuildAgentGroupConfiguration updateBuildAgentGroup( BuildAgentGroupConfiguration buildAgentGroup ) 
+    public BuildAgentGroupConfiguration updateBuildAgentGroup( BuildAgentGroupConfiguration buildAgentGroup )
         throws ConfigurationException, ConfigurationStoringException, ContinuumConfigurationException
     {
         try
@@ -2208,7 +2196,7 @@ public class ContinuumServiceImpl
             ConfigurationService configurationService = continuum.getConfiguration();
             org.apache.continuum.configuration.BuildAgentGroupConfiguration buildAgentGroupConfiguration =
                 configurationService.getBuildAgentGroup( buildAgentGroup.getName() );
-    
+
             if ( buildAgentGroupConfiguration != null )
             {
                 buildAgentGroupConfiguration.setName( StringEscapeUtils.escapeXml( buildAgentGroup.getName() ) );
@@ -2235,11 +2223,12 @@ public class ContinuumServiceImpl
         }
         catch ( ContinuumException e )
         {
-            throw new ContinuumConfigurationException( "Error in updating build agent group " + buildAgentGroup.getName(), e );
+            throw new ContinuumConfigurationException(
+                "Error in updating build agent group " + buildAgentGroup.getName(), e );
         }
     }
 
-    public int removeBuildAgentGroup( String name ) 
+    public int removeBuildAgentGroup( String name )
         throws ConfigurationException
     {
         ConfigurationService configurationService = continuum.getConfiguration();
@@ -2257,7 +2246,8 @@ public class ContinuumServiceImpl
     // Converters
     // ----------------------------------------------------------------------
 
-    private List<BuildProjectTask> populateBuildProjectTaskList( List<org.apache.continuum.taskqueue.BuildProjectTask> buildProjectTasks )
+    private List<BuildProjectTask> populateBuildProjectTaskList(
+        List<org.apache.continuum.taskqueue.BuildProjectTask> buildProjectTasks )
     {
         List<BuildProjectTask> responses = new ArrayList<BuildProjectTask>();
         for ( org.apache.continuum.taskqueue.BuildProjectTask buildProjectTask : buildProjectTasks )
@@ -2293,56 +2283,56 @@ public class ContinuumServiceImpl
         }
 
         // validate
-        if ( StringUtils.isNotBlank( projectSummary.getArtifactId() ) &&
-                        !projectSummary.getArtifactId().matches( PROJECT_ARTIFACT_ID_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getArtifactId() ) && !projectSummary.getArtifactId().matches(
+            PROJECT_ARTIFACT_ID_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Artifact Id contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getExecutorId() ) &&
-                        !projectSummary.getExecutorId().matches( PROJECT_EXECUTOR_OR_BUILDDEF_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getExecutorId() ) && !projectSummary.getExecutorId().matches(
+            PROJECT_EXECUTOR_OR_BUILDDEF_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Executor Id can only be 'maven2, maven-1, ant or shell'" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getGroupId() ) &&
-                        !projectSummary.getGroupId().matches( PROJECT_GROUP_ID_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getGroupId() ) && !projectSummary.getGroupId().matches(
+            PROJECT_GROUP_ID_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Group Id contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getName() ) && 
-                        !projectSummary.getName().matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getName() ) && !projectSummary.getName().matches(
+            NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Name contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getScmTag() ) && 
-                        !projectSummary.getScmTag().matches( PROJECT_SCM_TAG_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getScmTag() ) && !projectSummary.getScmTag().matches(
+            PROJECT_SCM_TAG_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Scm Tag contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getScmUrl() ) &&
-                        !projectSummary.getScmUrl().matches( PROJECT_SCM_URL_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getScmUrl() ) && !projectSummary.getScmUrl().matches(
+            PROJECT_SCM_URL_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Scm Url contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getUrl() ) && 
-                        !projectSummary.getUrl().matches( URL_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getUrl() ) && !projectSummary.getUrl().matches(
+            URL_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Url contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( projectSummary.getVersion() ) &&
-                        !projectSummary.getVersion().matches( PROJECT_VERSION_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( projectSummary.getVersion() ) && !projectSummary.getVersion().matches(
+            PROJECT_VERSION_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Version contains invalid characters" );
         }
 
         if ( StringUtils.isNotBlank( projectSummary.getWorkingDirectory() ) &&
-                        !projectSummary.getWorkingDirectory().matches( DIRECTORY_VALID_EXPRESSION ) )
+            !projectSummary.getWorkingDirectory().matches( DIRECTORY_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Project Working Directory contains invalid characters" );
         }
@@ -2357,8 +2347,8 @@ public class ContinuumServiceImpl
         project.setName( projectSummary.getName() );
         if ( projectSummary.getProjectGroup() != null )
         {
-            org.apache.maven.continuum.model.project.ProjectGroup g =
-                continuum.getProjectGroup( projectSummary.getProjectGroup().getId() );
+            org.apache.maven.continuum.model.project.ProjectGroup g = continuum.getProjectGroup(
+                projectSummary.getProjectGroup().getId() );
             project.setProjectGroup( populateProjectGroupSummary( projectSummary.getProjectGroup(), g ) );
         }
         else
@@ -2381,13 +2371,14 @@ public class ContinuumServiceImpl
         return (ProjectNotifier) mapper.map( notifier, ProjectNotifier.class );
     }
 
-    private ProjectGroupSummary populateProjectGroupSummary( org.apache.maven.continuum.model.project.ProjectGroup group )
+    private ProjectGroupSummary populateProjectGroupSummary(
+        org.apache.maven.continuum.model.project.ProjectGroup group )
     {
         return (ProjectGroupSummary) mapper.map( group, ProjectGroupSummary.class );
     }
 
-    private org.apache.maven.continuum.model.project.ProjectGroup populateProjectGroupSummary( ProjectGroupSummary group,
-                                                                                               org.apache.maven.continuum.model.project.ProjectGroup g )
+    private org.apache.maven.continuum.model.project.ProjectGroup populateProjectGroupSummary(
+        ProjectGroupSummary group, org.apache.maven.continuum.model.project.ProjectGroup g )
         throws ContinuumException
     {
         if ( group == null )
@@ -2395,14 +2386,13 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( group.getGroupId() ) &&
-                        !group.getGroupId().matches( PROJECTGROUP_ID_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( group.getGroupId() ) && !group.getGroupId().matches(
+            PROJECTGROUP_ID_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "ProjectGroup Id contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( group.getName() ) &&
-                        !group.getName().matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( group.getName() ) && !group.getName().matches( NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "ProjectGroup Name contains invalid characters" );
         }
@@ -2417,12 +2407,14 @@ public class ContinuumServiceImpl
         return g;
     }
 
-    private ProjectGroup populateProjectGroupWithAllDetails( org.apache.maven.continuum.model.project.ProjectGroup group )
+    private ProjectGroup populateProjectGroupWithAllDetails(
+        org.apache.maven.continuum.model.project.ProjectGroup group )
     {
         return (ProjectGroup) mapper.map( group, ProjectGroup.class );
     }
 
-    private BuildResultSummary populateBuildResultSummary( org.apache.maven.continuum.model.project.BuildResult buildResult )
+    private BuildResultSummary populateBuildResultSummary(
+        org.apache.maven.continuum.model.project.BuildResult buildResult )
     {
         return (BuildResultSummary) mapper.map( buildResult, BuildResultSummary.class );
     }
@@ -2480,8 +2472,8 @@ public class ContinuumServiceImpl
         return (BuildDefinition) mapper.map( buildDef, BuildDefinition.class );
     }
 
-    protected org.apache.maven.continuum.model.project.BuildDefinition populateBuildDefinition( BuildDefinition buildDef,
-                                                                                                org.apache.maven.continuum.model.project.BuildDefinition bd )
+    protected org.apache.maven.continuum.model.project.BuildDefinition populateBuildDefinition(
+        BuildDefinition buildDef, org.apache.maven.continuum.model.project.BuildDefinition bd )
         throws ProfileException, ContinuumException
     {
         if ( buildDef == null )
@@ -2489,26 +2481,26 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( buildDef.getArguments() ) &&
-                        !buildDef.getArguments().matches( BUILD_DEFINITION_ARGUMENTS_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( buildDef.getArguments() ) && !buildDef.getArguments().matches(
+            BUILD_DEFINITION_ARGUMENTS_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Build Definition Arguments contain invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( buildDef.getBuildFile() ) &&
-                        !buildDef.getBuildFile().matches( BUILD_DEFINITION_BUILD_FILE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( buildDef.getBuildFile() ) && !buildDef.getBuildFile().matches(
+            BUILD_DEFINITION_BUILD_FILE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Build Definition Build File contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( buildDef.getGoals() ) &&
-                        !buildDef.getGoals().matches( BUILD_DEFINITION_GOALS_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( buildDef.getGoals() ) && !buildDef.getGoals().matches(
+            BUILD_DEFINITION_GOALS_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Build Definition Goals contain invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( buildDef.getType() ) && 
-                        !buildDef.getType().matches( PROJECT_EXECUTOR_OR_BUILDDEF_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( buildDef.getType() ) && !buildDef.getType().matches(
+            PROJECT_EXECUTOR_OR_BUILDDEF_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Build Definition Type can only be 'maven2, maven-1, ant, or shell'" );
         }
@@ -2523,8 +2515,8 @@ public class ContinuumServiceImpl
         bd.setId( buildDef.getId() );
         if ( buildDef.getProfile() != null )
         {
-            bd.setProfile( populateProfile( buildDef.getProfile(),
-                                            continuum.getProfileService().getProfile( buildDef.getProfile().getId() ) ) );
+            bd.setProfile( populateProfile( buildDef.getProfile(), continuum.getProfileService().getProfile(
+                buildDef.getProfile().getId() ) ) );
         }
         else
         {
@@ -2532,8 +2524,8 @@ public class ContinuumServiceImpl
         }
         if ( buildDef.getSchedule() != null )
         {
-            bd.setSchedule( populateSchedule( buildDef.getSchedule(),
-                                              continuum.getSchedule( buildDef.getSchedule().getId() ) ) );
+            bd.setSchedule( populateSchedule( buildDef.getSchedule(), continuum.getSchedule(
+                buildDef.getSchedule().getId() ) ) );
         }
         else
         {
@@ -2548,13 +2540,15 @@ public class ContinuumServiceImpl
         return bd;
     }
 
-    protected void buildProjectWithBuildDefinition( int projectId, int buildDefinitionId, org.apache.continuum.utils.build.BuildTrigger buildTrigger )
+    protected void buildProjectWithBuildDefinition( int projectId, int buildDefinitionId,
+                                                    org.apache.continuum.utils.build.BuildTrigger buildTrigger )
         throws ContinuumException, NoBuildAgentException, NoBuildAgentInGroupException
     {
         continuum.buildProjectWithBuildDefinition( projectId, buildDefinitionId, buildTrigger );
     }
 
-    private BuildDefinitionTemplate populateBuildDefinitionTemplate( org.apache.maven.continuum.model.project.BuildDefinitionTemplate bdt )
+    private BuildDefinitionTemplate populateBuildDefinitionTemplate(
+        org.apache.maven.continuum.model.project.BuildDefinitionTemplate bdt )
     {
         return (BuildDefinitionTemplate) mapper.map( bdt, BuildDefinitionTemplate.class );
     }
@@ -2568,14 +2562,13 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( schedule.getCronExpression() ) && 
-                        !schedule.getCronExpression().matches( SCHEDULE_CRON_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( schedule.getCronExpression() ) && !schedule.getCronExpression().matches(
+            SCHEDULE_CRON_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Schedule Cron Expression contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( schedule.getName() ) &&
-                        !schedule.getName().matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( schedule.getName() ) && !schedule.getName().matches( NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Schedule Name contains invalid characters" );
         }
@@ -2604,8 +2597,7 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( profile.getName() ) 
-                        && !profile.getName().matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( profile.getName() ) && !profile.getName().matches( NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Build Environment Name contains invalid characters" );
         }
@@ -2675,26 +2667,25 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( install.getName() ) &&
-                        !install.getName().matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( install.getName() ) && !install.getName().matches( NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Installation Name contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( install.getType() ) &&
-                        !install.getType().matches( INSTALLATION_TYPE_VALID_EXPRESSION ))
+        if ( StringUtils.isNotBlank( install.getType() ) && !install.getType().matches(
+            INSTALLATION_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Installation Type contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( install.getVarName() ) &&
-                        !install.getVarName().matches( INSTALLATION_VARNAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( install.getVarName() ) && !install.getVarName().matches(
+            INSTALLATION_VARNAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Installation VarName contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( install.getVarValue() ) &&
-                        !install.getVarValue().matches( INSTALLATION_VARVALUE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( install.getVarValue() ) && !install.getVarValue().matches(
+            INSTALLATION_VARVALUE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Installation VarValue contains invalid characters" );
         }
@@ -2711,23 +2702,27 @@ public class ContinuumServiceImpl
         return (Installation) mapper.map( install, Installation.class );
     }
 
-    private SystemConfiguration populateSystemConfiguration( org.apache.maven.continuum.model.system.SystemConfiguration sysConf )
+    private SystemConfiguration populateSystemConfiguration(
+        org.apache.maven.continuum.model.system.SystemConfiguration sysConf )
     {
         return (SystemConfiguration) mapper.map( sysConf, SystemConfiguration.class );
     }
 
-    private ContinuumReleaseResult populateReleaseResult( org.apache.continuum.model.release.ContinuumReleaseResult releaseResult )
+    private ContinuumReleaseResult populateReleaseResult(
+        org.apache.continuum.model.release.ContinuumReleaseResult releaseResult )
     {
         return (ContinuumReleaseResult) mapper.map( releaseResult, ContinuumReleaseResult.class );
     }
 
-    private RepositoryPurgeConfiguration populateRepositoryPurgeConfiguration( org.apache.continuum.model.repository.RepositoryPurgeConfiguration repoPurgeConfig )
+    private RepositoryPurgeConfiguration populateRepositoryPurgeConfiguration(
+        org.apache.continuum.model.repository.RepositoryPurgeConfiguration repoPurgeConfig )
     {
         return (RepositoryPurgeConfiguration) mapper.map( repoPurgeConfig, RepositoryPurgeConfiguration.class );
     }
 
-    private org.apache.continuum.model.repository.RepositoryPurgeConfiguration populateRepositoryPurgeConfiguration( RepositoryPurgeConfiguration repoPurgeConfig,
-                                                                                                                     org.apache.continuum.model.repository.RepositoryPurgeConfiguration repoPurge )
+    private org.apache.continuum.model.repository.RepositoryPurgeConfiguration populateRepositoryPurgeConfiguration(
+        RepositoryPurgeConfiguration repoPurgeConfig,
+        org.apache.continuum.model.repository.RepositoryPurgeConfiguration repoPurge )
         throws RepositoryServiceException, ContinuumException
     {
         if ( repoPurgeConfig == null )
@@ -2745,7 +2740,8 @@ public class ContinuumServiceImpl
         if ( repoPurgeConfig.getRepository() != null )
         {
             repoPurge.setRepository( populateLocalRepository( repoPurgeConfig.getRepository(),
-                                                              continuum.getRepositoryService().getLocalRepository( repoPurgeConfig.getRepository().getId() ) ) );
+                                                              continuum.getRepositoryService().getLocalRepository(
+                                                                  repoPurgeConfig.getRepository().getId() ) ) );
         }
         else
         {
@@ -2753,8 +2749,8 @@ public class ContinuumServiceImpl
         }
         if ( repoPurgeConfig.getSchedule() != null )
         {
-            repoPurge.setSchedule( populateSchedule( repoPurgeConfig.getSchedule(),
-                                                     continuum.getSchedule( repoPurgeConfig.getSchedule().getId() ) ) );
+            repoPurge.setSchedule( populateSchedule( repoPurgeConfig.getSchedule(), continuum.getSchedule(
+                repoPurgeConfig.getSchedule().getId() ) ) );
         }
         else
         {
@@ -2764,13 +2760,15 @@ public class ContinuumServiceImpl
         return repoPurge;
     }
 
-    private DirectoryPurgeConfiguration populateDirectoryPurgeConfiguration( org.apache.continuum.model.repository.DirectoryPurgeConfiguration dirPurgeConfig )
+    private DirectoryPurgeConfiguration populateDirectoryPurgeConfiguration(
+        org.apache.continuum.model.repository.DirectoryPurgeConfiguration dirPurgeConfig )
     {
         return (DirectoryPurgeConfiguration) mapper.map( dirPurgeConfig, DirectoryPurgeConfiguration.class );
     }
 
-    private org.apache.continuum.model.repository.DirectoryPurgeConfiguration populateDirectoryPurgeConfiguration( DirectoryPurgeConfiguration dirPurgeConfig,
-                                                                                                                   org.apache.continuum.model.repository.DirectoryPurgeConfiguration dirPurge )
+    private org.apache.continuum.model.repository.DirectoryPurgeConfiguration populateDirectoryPurgeConfiguration(
+        DirectoryPurgeConfiguration dirPurgeConfig,
+        org.apache.continuum.model.repository.DirectoryPurgeConfiguration dirPurge )
         throws RepositoryServiceException, ContinuumException
     {
         if ( dirPurgeConfig == null )
@@ -2778,14 +2776,14 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( dirPurgeConfig.getDirectoryType() ) &&
-                        !dirPurgeConfig.getDirectoryType().matches( DIRECTORY_TYPE_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( dirPurgeConfig.getDirectoryType() ) && !dirPurgeConfig.getDirectoryType().matches(
+            DIRECTORY_TYPE_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Directory Purge Configuration Type can only be 'releases or buildOutput'" );
         }
 
-        if ( StringUtils.isNotBlank( dirPurgeConfig.getLocation() ) &&
-                        !dirPurgeConfig.getLocation().matches( DIRECTORY_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( dirPurgeConfig.getLocation() ) && !dirPurgeConfig.getLocation().matches(
+            DIRECTORY_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Directory Purge Configuration Location contains invalid characters" );
         }
@@ -2812,8 +2810,8 @@ public class ContinuumServiceImpl
         dirPurge.setRetentionCount( dirPurgeConfig.getRetentionCount() );
         if ( dirPurgeConfig.getSchedule() != null )
         {
-            dirPurge.setSchedule( populateSchedule( dirPurgeConfig.getSchedule(),
-                                                    continuum.getSchedule( dirPurgeConfig.getSchedule().getId() ) ) );
+            dirPurge.setSchedule( populateSchedule( dirPurgeConfig.getSchedule(), continuum.getSchedule(
+                dirPurgeConfig.getSchedule().getId() ) ) );
         }
         else
         {
@@ -2823,7 +2821,8 @@ public class ContinuumServiceImpl
         return dirPurge;
     }
 
-    private LocalRepository populateLocalRepository( org.apache.continuum.model.repository.LocalRepository localRepository )
+    private LocalRepository populateLocalRepository(
+        org.apache.continuum.model.repository.LocalRepository localRepository )
     {
         return (LocalRepository) mapper.map( localRepository, LocalRepository.class );
     }
@@ -2837,20 +2836,19 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( repository.getLayout() ) &&
-                        !repository.getLayout().matches( REPOSITORY_LAYOUT_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( repository.getLayout() ) && !repository.getLayout().matches(
+            REPOSITORY_LAYOUT_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Repository Layout can only be 'default or legacy'" );
         }
 
-        if ( StringUtils.isNotBlank( repository.getLocation() ) &&
-                        !repository.getLocation().matches( DIRECTORY_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( repository.getLocation() ) && !repository.getLocation().matches(
+            DIRECTORY_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Repository Location contains invalid characters" );
         }
 
-        if ( StringUtils.isNotBlank( repository.getName() ) &&
-                        !repository.getName().matches( NAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( repository.getName() ) && !repository.getName().matches( NAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Repository Name contains invalid characters" );
         }
@@ -2869,8 +2867,8 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( buildTrigger.getTriggeredBy() ) &&
-                        !buildTrigger.getTriggeredBy().matches( USERNAME_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( buildTrigger.getTriggeredBy() ) && !buildTrigger.getTriggeredBy().matches(
+            USERNAME_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "BuildTrigger Triggered By contains invalid characters" );
         }
@@ -2879,7 +2877,8 @@ public class ContinuumServiceImpl
                                                                   buildTrigger.getTriggeredBy() );
     }
 
-    private org.apache.continuum.configuration.BuildAgentConfiguration populateBuildAgent( BuildAgentConfiguration buildAgent )
+    private org.apache.continuum.configuration.BuildAgentConfiguration populateBuildAgent(
+        BuildAgentConfiguration buildAgent )
         throws ContinuumException
     {
         if ( buildAgent == null )
@@ -2887,8 +2886,7 @@ public class ContinuumServiceImpl
             return null;
         }
 
-        if ( StringUtils.isNotBlank( buildAgent.getUrl() ) &&
-                        !buildAgent.getUrl().matches( URL_VALID_EXPRESSION ) )
+        if ( StringUtils.isNotBlank( buildAgent.getUrl() ) && !buildAgent.getUrl().matches( URL_VALID_EXPRESSION ) )
         {
             throw new ContinuumException( "Build Agent URL contains invalid characters" );
         }
@@ -2901,13 +2899,15 @@ public class ContinuumServiceImpl
         return buildAgentConfiguration;
     }
 
-    private BuildAgentConfiguration populateBuildAgent( org.apache.continuum.configuration.BuildAgentConfiguration buildAgent )
+    private BuildAgentConfiguration populateBuildAgent(
+        org.apache.continuum.configuration.BuildAgentConfiguration buildAgent )
     {
-        BuildAgentConfiguration buildAgentConfiguration =
-            (BuildAgentConfiguration) mapper.map( buildAgent, BuildAgentConfiguration.class );
+        BuildAgentConfiguration buildAgentConfiguration = (BuildAgentConfiguration) mapper.map( buildAgent,
+                                                                                                BuildAgentConfiguration.class );
         try
         {
-            buildAgentConfiguration.setPlatform( distributedBuildManager.getBuildAgentPlatform( buildAgentConfiguration.getUrl() ) );
+            buildAgentConfiguration.setPlatform( distributedBuildManager.getBuildAgentPlatform(
+                buildAgentConfiguration.getUrl() ) );
             return buildAgentConfiguration;
         }
         catch ( ContinuumException e )
@@ -2918,7 +2918,8 @@ public class ContinuumServiceImpl
         }
     }
 
-    private org.apache.continuum.configuration.BuildAgentGroupConfiguration populateBuildAgentGroup( BuildAgentGroupConfiguration buildAgentGroup )
+    private org.apache.continuum.configuration.BuildAgentGroupConfiguration populateBuildAgentGroup(
+        BuildAgentGroupConfiguration buildAgentGroup )
         throws ContinuumException
     {
         if ( buildAgentGroup == null )
@@ -2939,15 +2940,16 @@ public class ContinuumServiceImpl
         buildAgentGroupConfiguration.getBuildAgents().clear();
         if ( buildAgentGroup.getBuildAgents() != null )
         {
-            for( BuildAgentConfiguration buildAgent : buildAgentGroup.getBuildAgents() )
+            for ( BuildAgentConfiguration buildAgent : buildAgentGroup.getBuildAgents() )
             {
                 buildAgentGroupConfiguration.getBuildAgents().add( populateBuildAgent( buildAgent ) );
             }
         }
         return buildAgentGroupConfiguration;
     }
-    
-    private BuildAgentGroupConfiguration populateBuildAgentGroup( org.apache.continuum.configuration.BuildAgentGroupConfiguration buildAgentGroup )
+
+    private BuildAgentGroupConfiguration populateBuildAgentGroup(
+        org.apache.continuum.configuration.BuildAgentGroupConfiguration buildAgentGroup )
     {
         return (BuildAgentGroupConfiguration) mapper.map( buildAgentGroup, BuildAgentGroupConfiguration.class );
     }
@@ -3361,14 +3363,15 @@ public class ContinuumServiceImpl
         throws Exception
     {
         return serializeObject( this.addBuildDefinitionToProjectGroup( projectGroupId,
-                                                                       (BuildDefinition) unserializeObject( buildDef ) ) );
+                                                                       (BuildDefinition) unserializeObject(
+                                                                           buildDef ) ) );
     }
 
     public Map<String, Object> addBuildDefinitionToProjectRPC( int projectId, Map<String, Object> buildDef )
         throws Exception
     {
-        return serializeObject( this.addBuildDefinitionToProject( projectId,
-                                                                  (BuildDefinition) unserializeObject( buildDef ) ) );
+        return serializeObject( this.addBuildDefinitionToProject( projectId, (BuildDefinition) unserializeObject(
+            buildDef ) ) );
     }
 
     public Map<String, Object> addMavenOneProjectRPC( String url, int projectGroupId )
@@ -3389,7 +3392,8 @@ public class ContinuumServiceImpl
         return serializeObject( this.addMavenTwoProject( url, projectGroupId ) );
     }
 
-    public Map<String, Object> addMavenTwoProjectRPC( String url, int projectGroupId, boolean checkoutInSingleDirectory )
+    public Map<String, Object> addMavenTwoProjectRPC( String url, int projectGroupId,
+                                                      boolean checkoutInSingleDirectory )
         throws Exception
     {
         return serializeObject( this.addMavenTwoProject( url, projectGroupId, checkoutInSingleDirectory ) );
@@ -3584,18 +3588,20 @@ public class ContinuumServiceImpl
         return serializeObject( this.removeBuildResult( (BuildResult) unserializeObject( br ) ) );
     }
 
-    public Map<String, Object> updateBuildDefinitionForProjectGroupRPC( int projectGroupId, Map<String, Object> buildDef )
+    public Map<String, Object> updateBuildDefinitionForProjectGroupRPC( int projectGroupId,
+                                                                        Map<String, Object> buildDef )
         throws Exception
     {
         return serializeObject( this.updateBuildDefinitionForProjectGroup( projectGroupId,
-                                                                           (BuildDefinition) unserializeObject( buildDef ) ) );
+                                                                           (BuildDefinition) unserializeObject(
+                                                                               buildDef ) ) );
     }
 
     public Map<String, Object> updateBuildDefinitionForProjectRPC( int projectId, Map<String, Object> buildDef )
         throws Exception
     {
-        return serializeObject( this.updateBuildDefinitionForProject( projectId,
-                                                                      (BuildDefinition) unserializeObject( buildDef ) ) );
+        return serializeObject( this.updateBuildDefinitionForProject( projectId, (BuildDefinition) unserializeObject(
+            buildDef ) ) );
     }
 
     public Map<String, Object> updateProjectRPC( Map<String, Object> project )
@@ -3631,8 +3637,8 @@ public class ContinuumServiceImpl
     public Map<String, Object> updateGroupNotifierRPC( int projectgroupid, Map<String, Object> newNotifier )
         throws Exception
     {
-        return serializeObject( this.updateGroupNotifier( projectgroupid,
-                                                          (ProjectNotifier) unserializeObject( newNotifier ) ) );
+        return serializeObject( this.updateGroupNotifier( projectgroupid, (ProjectNotifier) unserializeObject(
+            newNotifier ) ) );
     }
 
     public Map<String, Object> updateNotifierRPC( int projectid, Map<String, Object> newNotifier )
@@ -3644,8 +3650,8 @@ public class ContinuumServiceImpl
     public Map<String, Object> addGroupNotifierRPC( int projectgroupid, Map<String, Object> newNotifier )
         throws Exception
     {
-        return serializeObject( this.addGroupNotifier( projectgroupid,
-                                                       (ProjectNotifier) unserializeObject( newNotifier ) ) );
+        return serializeObject( this.addGroupNotifier( projectgroupid, (ProjectNotifier) unserializeObject(
+            newNotifier ) ) );
     }
 
     public Map<String, Object> addNotifierRPC( int projectid, Map<String, Object> newNotifier )
@@ -3699,13 +3705,15 @@ public class ContinuumServiceImpl
     public Map<String, Object> addRepositoryPurgeConfigurationRPC( Map<String, Object> repoPurge )
         throws Exception
     {
-        return serializeObject( this.addRepositoryPurgeConfiguration( (RepositoryPurgeConfiguration) unserializeObject( repoPurge ) ) );
+        return serializeObject( this.addRepositoryPurgeConfiguration( (RepositoryPurgeConfiguration) unserializeObject(
+            repoPurge ) ) );
     }
 
     public int updateRepositoryPurgeConfigurationRPC( Map<String, Object> repoPurge )
         throws Exception
     {
-        return serializeObject( this.updateRepositoryPurgeConfiguration( (RepositoryPurgeConfiguration) unserializeObject( repoPurge ) ) );
+        return serializeObject( this.updateRepositoryPurgeConfiguration(
+            (RepositoryPurgeConfiguration) unserializeObject( repoPurge ) ) );
     }
 
     public Map<String, Object> getRepositoryPurgeConfigurationRPC( int repoPurgeId )
@@ -3723,13 +3731,15 @@ public class ContinuumServiceImpl
     public Map<String, Object> addDirectoryPurgeConfigurationRPC( Map<String, Object> dirPurge )
         throws Exception
     {
-        return serializeObject( this.addDirectoryPurgeConfiguration( (DirectoryPurgeConfiguration) unserializeObject( dirPurge ) ) );
+        return serializeObject( this.addDirectoryPurgeConfiguration( (DirectoryPurgeConfiguration) unserializeObject(
+            dirPurge ) ) );
     }
 
     public int updateDirectoryPurgeConfigurationRPC( Map<String, Object> dirPurge )
         throws Exception
     {
-        return serializeObject( this.updateDirectoryPurgeConfiguration( (DirectoryPurgeConfiguration) unserializeObject( dirPurge ) ) );
+        return serializeObject( this.updateDirectoryPurgeConfiguration( (DirectoryPurgeConfiguration) unserializeObject(
+            dirPurge ) ) );
     }
 
     public Map<String, Object> getDirectoryPurgeConfigurationRPC( int dirPurgeId )
@@ -3771,7 +3781,8 @@ public class ContinuumServiceImpl
     public Map<String, Object> addBuildAgentRPC( Map<String, Object> buildAgentConfiguration )
         throws Exception
     {
-        return serializeObject( this.addBuildAgent( (BuildAgentConfiguration) unserializeObject( buildAgentConfiguration ) ) );
+        return serializeObject( this.addBuildAgent( (BuildAgentConfiguration) unserializeObject(
+            buildAgentConfiguration ) ) );
     }
 
     public Map<String, Object> getBuildAgentRPC( String url )
@@ -3782,7 +3793,8 @@ public class ContinuumServiceImpl
     public Map<String, Object> updateBuildAgentRPC( Map<String, Object> buildAgentConfiguration )
         throws ConfigurationStoringException, ContinuumConfigurationException
     {
-        return serializeObject( this.updateBuildAgent( (BuildAgentConfiguration) unserializeObject( buildAgentConfiguration ) ) );
+        return serializeObject( this.updateBuildAgent( (BuildAgentConfiguration) unserializeObject(
+            buildAgentConfiguration ) ) );
     }
 
     public List<Object> getAllBuildAgentsRPC()
@@ -3796,10 +3808,11 @@ public class ContinuumServiceImpl
         return serializeObject( this.getBuildAgentsWithInstallations() );
     }
 
-    public Map<String, Object> addBuildAgentGroupRPC( Map<String, Object> buildAgentGroup ) 
+    public Map<String, Object> addBuildAgentGroupRPC( Map<String, Object> buildAgentGroup )
         throws ConfigurationException, ConfigurationStoringException, ContinuumConfigurationException
     {
-        return serializeObject( this.addBuildAgentGroup( (BuildAgentGroupConfiguration) unserializeObject( buildAgentGroup ) ) );
+        return serializeObject( this.addBuildAgentGroup( (BuildAgentGroupConfiguration) unserializeObject(
+            buildAgentGroup ) ) );
     }
 
     public Map<String, Object> getBuildAgentGroupRPC( String name )
@@ -3810,7 +3823,8 @@ public class ContinuumServiceImpl
     public Map<String, Object> updateBuildAgentGroupRPC( Map<String, Object> buildAgentGroup )
         throws ConfigurationException, ConfigurationStoringException, ContinuumConfigurationException
     {
-        return serializeObject( this.updateBuildAgentGroup( (BuildAgentGroupConfiguration) unserializeObject( buildAgentGroup ) ) );
+        return serializeObject( this.updateBuildAgentGroup( (BuildAgentGroupConfiguration) unserializeObject(
+            buildAgentGroup ) ) );
     }
 
     public String releasePrepare( int projectId, Properties releaseProperties, Map<String, String> releaseVersions,
@@ -3836,8 +3850,8 @@ public class ContinuumServiceImpl
 
                 if ( environments != null )
                 {
-                    String m2Home =
-                        environments.get( continuum.getInstallationService().getEnvVar( InstallationService.MAVEN2_TYPE ) );
+                    String m2Home = environments.get( continuum.getInstallationService().getEnvVar(
+                        InstallationService.MAVEN2_TYPE ) );
                     if ( StringUtils.isNotEmpty( m2Home ) )
                     {
                         executable = m2Home + File.separator + "bin" + File.separator + executable;
@@ -3846,15 +3860,14 @@ public class ContinuumServiceImpl
 
                 releaseProperties.setProperty( "release-by", username );
                 return continuum.getReleaseManager().prepare( project, releaseProperties, releaseVersions,
-                                                              developmentVersions, null,
-                                                              continuum.getWorkingDirectory( projectId ).getPath(),
-                                                              environments, executable );
+                                                              developmentVersions, null, continuum.getWorkingDirectory(
+                    projectId ).getPath(), environments, executable );
             }
         }
         else
         {
-            throw new Exception( "Unable to prepare release project with id : " + projectId
-                + " because it doesn't exist" );
+            throw new Exception(
+                "Unable to prepare release project with id : " + projectId + " because it doesn't exist" );
         }
     }
 
@@ -3878,21 +3891,20 @@ public class ContinuumServiceImpl
             }
             else
             {
-                File performDirectory =
-                    new File( continuum.getConfiguration().getWorkingDirectory(), "releases-"
-                        + System.currentTimeMillis() );
+                File performDirectory = new File( continuum.getConfiguration().getWorkingDirectory(),
+                                                  "releases-" + System.currentTimeMillis() );
                 performDirectory.mkdirs();
 
-                continuum.getReleaseManager().perform( releaseId, performDirectory, goals, arguments,
-                                                       useReleaseProfile, null, repository );
+                continuum.getReleaseManager().perform( releaseId, performDirectory, goals, arguments, useReleaseProfile,
+                                                       null, repository );
             }
 
             return 0;
         }
         else
         {
-            throw new Exception( "Unable to perform release project with id : " + projectId
-                + " because it doesn't exist" );
+            throw new Exception(
+                "Unable to perform release project with id : " + projectId + " because it doesn't exist" );
         }
     }
 
@@ -3935,8 +3947,8 @@ public class ContinuumServiceImpl
         if ( project != null )
         {
             checkBuildProjectInGroupAuthorization( project.getProjectGroup().getName() );
-            org.apache.continuum.model.release.ContinuumReleaseResult result =
-                continuum.addContinuumReleaseResult( projectId, releaseId, releaseType );
+            org.apache.continuum.model.release.ContinuumReleaseResult result = continuum.addContinuumReleaseResult(
+                projectId, releaseId, releaseType );
 
             if ( continuum.getConfiguration().isDistributedBuildEnabled() )
             {
@@ -3970,8 +3982,8 @@ public class ContinuumServiceImpl
             }
             else
             {
-                continuum.getReleaseManager().rollback( releaseId,
-                                                        continuum.getWorkingDirectory( projectId ).getPath(), null );
+                continuum.getReleaseManager().rollback( releaseId, continuum.getWorkingDirectory( projectId ).getPath(),
+                                                        null );
                 continuum.getReleaseManager().getPreparedReleases().remove( releaseId );
             }
             return 0;
@@ -4036,9 +4048,8 @@ public class ContinuumServiceImpl
             {
                 if ( project.getScmUrl().startsWith( "scm:svn" ) )
                 {
-                    String scmTagBase =
-                        new SvnScmProviderRepository( project.getScmUrl(), project.getScmUsername(),
-                                                      project.getScmPassword() ).getTagBase();
+                    String scmTagBase = new SvnScmProviderRepository( project.getScmUrl(), project.getScmUsername(),
+                                                                      project.getScmPassword() ).getTagBase();
                     // strip the Maven scm protocol prefix
                     params.put( "scm-tagbase", scmTagBase.substring( "scm:svn".length() + 1 ) );
                 }
@@ -4069,7 +4080,8 @@ public class ContinuumServiceImpl
 
             if ( continuum.getConfiguration().isDistributedBuildEnabled() )
             {
-                projects = continuum.getDistributedReleaseManager().processProject( projectId, pomFilename, autoVersionSubmodules );
+                projects = continuum.getDistributedReleaseManager().processProject( projectId, pomFilename,
+                                                                                    autoVersionSubmodules );
             }
             else
             {
@@ -4127,7 +4139,8 @@ public class ContinuumServiceImpl
         return listenerSummary;
     }
 
-    private ReleaseListenerSummary populateReleaseListenerSummary( org.apache.continuum.model.release.ReleaseListenerSummary listener )
+    private ReleaseListenerSummary populateReleaseListenerSummary(
+        org.apache.continuum.model.release.ReleaseListenerSummary listener )
     {
         return (ReleaseListenerSummary) mapper.map( listener, ReleaseListenerSummary.class );
     }
@@ -4161,7 +4174,7 @@ public class ContinuumServiceImpl
     {
         this.continuum = continuum;
     }
-    
+
     public void setDistributedBuildManager( DistributedBuildManager distributedBuildManager )
     {
         this.distributedBuildManager = distributedBuildManager;

Modified: continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumXmlRpcMetaDataHandler.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumXmlRpcMetaDataHandler.java?rev=1372260&r1=1372259&r2=1372260&view=diff
==============================================================================
--- continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumXmlRpcMetaDataHandler.java (original)
+++ continuum/trunk/continuum-xmlrpc/continuum-xmlrpc-server/src/main/java/org/apache/maven/continuum/xmlrpc/server/ContinuumXmlRpcMetaDataHandler.java Mon Aug 13 04:29:09 2012
@@ -1,8 +1,5 @@
 package org.apache.maven.continuum.xmlrpc.server;
 
-import java.lang.reflect.InvocationTargetException;
-import java.lang.reflect.Method;
-
 import org.apache.xmlrpc.XmlRpcException;
 import org.apache.xmlrpc.XmlRpcHandler;
 import org.apache.xmlrpc.XmlRpcRequest;
@@ -15,6 +12,9 @@ import org.apache.xmlrpc.server.RequestP
 import org.codehaus.plexus.PlexusContainer;
 import org.codehaus.plexus.component.repository.exception.ComponentLifecycleException;
 
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+
 /**
  * @author <a href="mailto:evenisse@apache.org">Emmanuel Venisse</a>
  * @version $Id$



Mime
View raw message