Index: Continuum trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java =================================================================== --- Continuum trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java (revision 654023) +++ Continuum trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java (working copy) @@ -90,14 +90,16 @@ clearErrorsAndMessages(); try { + if ( ( projectName.trim().length() > 0 ) && ( projectVersion.trim().length() > 0 ) && ( projectScmUrl.trim().length() > 0 ) ) + { Iterator projects = getContinuum().getProjects().iterator(); while ( projects.hasNext() ) { Project project = projects.next(); // CONTINUUM-1445 - if ( StringUtils.equalsIgnoreCase( project.getName(), projectName ) && - StringUtils.equalsIgnoreCase( project.getVersion(), projectVersion ) && - StringUtils.equalsIgnoreCase( project.getScmUrl(), projectScmUrl ) ) + if ( StringUtils.equalsIgnoreCase( project.getName(), projectName.trim() ) && + StringUtils.equalsIgnoreCase( project.getVersion(), projectVersion.trim() ) && + StringUtils.equalsIgnoreCase( project.getScmUrl(), projectScmUrl.trim() ) ) { projectNameAlreadyExist = true; break; @@ -108,6 +110,23 @@ addActionError( "projectName.already.exist.error" ); this.input(); } + } + else + { + if ( !( projectName.trim().length() > 0 ) ) + { + addActionError( getText( "addProject.name.required" ) ); + } + if ( !( projectVersion.trim().length() > 0 ) ) + { + addActionError( getText( "addProject.version.required" ) ); + } + if ( !( projectScmUrl.trim().length() > 0 ) ) + { + addActionError( getText( "addProject.scmUrl.required" ) ); + } + this.input(); + } } catch ( ContinuumException e ) { @@ -143,11 +162,11 @@ Project project = new Project(); - project.setName( projectName ); + project.setName( projectName.trim() ); - project.setVersion( projectVersion ); + project.setVersion( projectVersion.trim() ); - project.setScmUrl( projectScmUrl ); + project.setScmUrl( projectScmUrl.trim() ); project.setScmUsername( projectScmUsername );