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 ) );
+ }
}
}
}