You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ev...@apache.org on 2007/05/24 15:26:20 UTC
svn commit: r541299 - in /maven/continuum/trunk/continuum-webapp/src/main:
java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java
resources/localization/Continuum.properties
Author: evenisse
Date: Thu May 24 06:26:19 2007
New Revision: 541299
URL: http://svn.apache.org/viewvc?view=rev&rev=541299
Log:
[CONTINUUM-1290] Validate Project group ID when adding a Project Group
Submitted by: Napoleon Esmundo C. Ramirez
Modified:
maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java
maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties
Modified: maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java
URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java?view=diff&rev=541299&r1=541298&r2=541299
==============================================================================
--- maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java (original)
+++ maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java Thu May 24 06:26:19 2007
@@ -67,6 +67,10 @@
{
addActionError( "projectGroup.error.groupId.required" );
}
+ else if ( groupId != null && groupId.trim().equals( "" ) )
+ {
+ addActionError( "projectGroup.error.groupId.cannot.be.spaces" );
+ }
}
public String execute()
Modified: maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties
URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties?view=diff&rev=541299&r1=541298&r2=541299
==============================================================================
--- maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties (original)
+++ maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties Thu May 24 06:26:19 2007
@@ -151,6 +151,7 @@
projectGroup.error.name.cannot.be.spaces = Project Group Name cannot contain spaces only.
projectGroup.error.name.already.exists = Project Group Name already exists.
projectGroup.error.groupId.required = Project Group ID required.
+projectGroup.error.groupId.cannot.be.spaces = Project Group ID cannot contain spaces only.
# ----------------------------------------------------------------------
# Page: Edit Project Group