You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ws...@apache.org on 2008/08/31 01:23:25 UTC

svn commit: r690610 - /continuum/branches/continuum-1.2.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigureAppearanceAction.java

Author: wsmoak
Date: Sat Aug 30 16:23:24 2008
New Revision: 690610

URL: http://svn.apache.org/viewvc?rev=690610&view=rev
Log:
CONTINUUM-1863 Guard against NPE when clicking Appearance menu button

Modified:
    continuum/branches/continuum-1.2.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigureAppearanceAction.java

Modified: continuum/branches/continuum-1.2.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigureAppearanceAction.java
URL: http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigureAppearanceAction.java?rev=690610&r1=690609&r2=690610&view=diff
==============================================================================
--- continuum/branches/continuum-1.2.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigureAppearanceAction.java (original)
+++ continuum/branches/continuum-1.2.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/admin/ConfigureAppearanceAction.java Sat Aug 30 16:23:24 2008
@@ -138,13 +138,16 @@
             for ( String profileId : profileIds )
             {
                 Profile profile = profilesAsMap.get( profileId );
-                List<Repository> repos = profile.getRepositories();
-                if ( repos != null && !repos.isEmpty() )
+                if (profile != null)
                 {
-                    for ( Repository repo : repos )
+                    List<Repository> repos = profile.getRepositories();
+                    if ( repos != null && !repos.isEmpty() )
                     {
+                        for ( Repository repo : repos )
+                        {
                         remoteRepositories.add( artifactRepositoryFactory.createArtifactRepository( repo.getId(), repo
                             .getUrl(), layout, null, null ) );
+                        }
                     }
                 }
             }