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/08/07 17:30:47 UTC

svn commit: r563524 - /maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java

Author: evenisse
Date: Tue Aug  7 08:30:46 2007
New Revision: 563524

URL: http://svn.apache.org/viewvc?view=rev&rev=563524
Log:
[CONTINUUM-1372] Fix possible NPE when adding a new project group

Modified:
    maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectGroupAction.java

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=563524&r1=563523&r2=563524
==============================================================================
--- 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 Aug  7 08:30:46 2007
@@ -65,7 +65,7 @@
             while ( iterator.hasNext() )
             {
                 ProjectGroup projectGroup = (ProjectGroup) iterator.next();
-                if ( projectGroup.getName().equals( name ) )
+                if ( name.equals( projectGroup.getName() ) )
                 {
                     addActionError( "projectGroup.error.name.already.exists" );
                     break;