continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eveni...@apache.org
Subject svn commit: r540606 - in /maven/continuum/trunk/continuum-webapp/src/main: java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java resources/localization/Continuum.properties
Date Tue, 22 May 2007 15:03:04 GMT
Author: evenisse
Date: Tue May 22 08:03:04 2007
New Revision: 540606

URL: http://svn.apache.org/viewvc?view=rev&rev=540606
Log:
[CONTINUUM-1275] Don't allow spaces as a project group name
Submitted by: Teodoro Cue Jr.

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=540606&r1=540605&r2=540606
==============================================================================
--- 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
Tue May 22 08:03:04 2007
@@ -47,7 +47,10 @@
         {
             addActionError( "projectGroup.error.name.required" );
         }
-        if ( name != null && !name.equals( "" ) )
+        else if ( name != null && name.trim().equals( "" ) ) {
+            addActionError( "projectGroup.error.name.cannot.be.spaces" );
+        }
+        else if ( name != null && !name.equals( "" ) )
         {
             Iterator iterator = getContinuum().getAllProjectGroups().iterator();
             while ( iterator.hasNext() )

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=540606&r1=540605&r2=540606
==============================================================================
--- maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties
(original)
+++ maven/continuum/trunk/continuum-webapp/src/main/resources/localization/Continuum.properties
Tue May 22 08:03:04 2007
@@ -148,6 +148,7 @@
 projectGroup.add.page.title = Continuum - Add Project Group
 projectGroup.add.section.title = Add Project Group
 projectGroup.error.name.required = Project Group Name required.
+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.
 



Mime
View raw message