You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by vs...@apache.org on 2008/04/24 14:13:59 UTC
svn commit: r651238 -
/maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java
Author: vsiveton
Date: Thu Apr 24 05:13:56 2008
New Revision: 651238
URL: http://svn.apache.org/viewvc?rev=651238&view=rev
Log:
o fixed PMD
Modified:
maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java
Modified: maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java?rev=651238&r1=651237&r2=651238&view=diff
==============================================================================
--- maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java (original)
+++ maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java Thu Apr 24 05:13:56 2008
@@ -806,39 +806,41 @@
Menu menu = decorationModel.getMenuRef( "parent" );
- if ( menu != null )
+ if ( menu == null )
{
- if ( !keepInheritedRefs || !menu.isInheritAsRef() )
- {
- String parentUrl = parentProject.getUrl();
-
- if ( parentUrl != null )
- {
- if ( parentUrl.endsWith( "/" ) )
- {
- parentUrl += "index.html";
- }
- else
- {
- parentUrl += "/index.html";
- }
-
- parentUrl = getRelativePath( parentUrl, project.getUrl() );
+ return;
+ }
- if ( menu.getName() == null )
- {
- menu.setName( i18n.getString( "site-tool", locale, "decorationModel.menu.parentproject" ) );
- }
+ if ( !keepInheritedRefs || !menu.isInheritAsRef() )
+ {
+ String parentUrl = parentProject.getUrl();
- MenuItem item = new MenuItem();
- item.setName( parentProject.getName() );
- item.setHref( parentUrl );
- menu.addItem( item );
+ if ( parentUrl != null )
+ {
+ if ( parentUrl.endsWith( "/" ) )
+ {
+ parentUrl += "index.html";
}
else
{
- decorationModel.removeMenuRef( "parent" );
+ parentUrl += "/index.html";
+ }
+
+ parentUrl = getRelativePath( parentUrl, project.getUrl() );
+
+ if ( menu.getName() == null )
+ {
+ menu.setName( i18n.getString( "site-tool", locale, "decorationModel.menu.parentproject" ) );
}
+
+ MenuItem item = new MenuItem();
+ item.setName( parentProject.getName() );
+ item.setHref( parentUrl );
+ menu.addItem( item );
+ }
+ else
+ {
+ decorationModel.removeMenuRef( "parent" );
}
}
}
@@ -872,64 +874,66 @@
Menu menu = decorationModel.getMenuRef( "modules" );
- if ( menu != null )
+ if ( menu == null )
+ {
+ return;
+ }
+
+ if ( !keepInheritedRefs || !menu.isInheritAsRef() )
{
- if ( !keepInheritedRefs || !menu.isInheritAsRef() )
+ // we require child modules and reactors to process module menu
+ if ( project.getModules().size() > 0 )
{
- // we require child modules and reactors to process module menu
- if ( project.getModules().size() > 0 )
- {
- List projects = reactorProjects;
+ List projects = reactorProjects;
- if ( menu.getName() == null )
- {
- menu.setName( i18n.getString( "site-tool", locale, "decorationModel.menu.projectmodules" ) );
- }
+ if ( menu.getName() == null )
+ {
+ menu.setName( i18n.getString( "site-tool", locale, "decorationModel.menu.projectmodules" ) );
+ }
- if ( projects.size() == 1 )
- {
- getLogger().debug( "Attempting to load module information from local filesystem" );
+ if ( projects.size() == 1 )
+ {
+ getLogger().debug( "Attempting to load module information from local filesystem" );
- // Not running reactor - search for the projects manually
- List models = new ArrayList( project.getModules().size() );
- for ( Iterator i = project.getModules().iterator(); i.hasNext(); )
+ // Not running reactor - search for the projects manually
+ List models = new ArrayList( project.getModules().size() );
+ for ( Iterator i = project.getModules().iterator(); i.hasNext(); )
+ {
+ String module = (String) i.next();
+ Model model;
+ File f = new File( project.getBasedir(), module + "/pom.xml" );
+ if ( f.exists() )
{
- String module = (String) i.next();
- Model model;
- File f = new File( project.getBasedir(), module + "/pom.xml" );
- if ( f.exists() )
+ try
{
- try
- {
- model = mavenProjectBuilder.build( f, localRepository, null ).getModel();
- }
- catch ( ProjectBuildingException e )
- {
- throw new SiteToolException( "Unable to read local module-POM", e );
- }
+ model = mavenProjectBuilder.build( f, localRepository, null ).getModel();
}
- else
+ catch ( ProjectBuildingException e )
{
- getLogger().warn( "No filesystem module-POM available" );
-
- model = new Model();
- model.setName( module );
- model.setUrl( module );
+ throw new SiteToolException( "Unable to read local module-POM", e );
}
- models.add( model );
}
- populateModulesMenuItemsFromModels( project, models, menu );
- }
- else
- {
- populateModulesMenuItemsFromReactorProjects( project, reactorProjects, menu );
+ else
+ {
+ getLogger().warn( "No filesystem module-POM available" );
+
+ model = new Model();
+ model.setName( module );
+ model.setUrl( module );
+ }
+ models.add( model );
}
+ populateModulesMenuItemsFromModels( project, models, menu );
}
else
{
- decorationModel.removeMenuRef( "modules" );
+ populateModulesMenuItemsFromReactorProjects( project, reactorProjects, menu );
}
}
+ else
+ {
+ decorationModel.removeMenuRef( "modules" );
+ }
}
}
@@ -960,30 +964,29 @@
}
// Default bundles are in English
- if ( !locale.getLanguage().equals( DEFAULT_LOCALE.getLanguage() ) )
- {
- if ( !i18n.getBundle( "site-tool", locale ).getLocale().getLanguage().equals(
- locale.getLanguage() ) )
- {
- StringBuffer sb = new StringBuffer();
-
- sb.append( "The locale '" ).append( locale ).append( "' (" );
- sb.append( locale.getDisplayName( Locale.ENGLISH ) );
- sb.append( ") is not currently support by Maven - IGNORING. " );
- sb.append( "\n" );
- sb.append( "Contribution are welcome and greatly appreciated! " );
- sb.append( "\n" );
- sb.append( "If you want to contribute a new translation, please visit " );
- sb.append( "http://maven.apache.org/plugins/maven-site-plugin/i18n.html " );
- sb.append( "for detailed instructions." );
-
- if ( getLogger().isWarnEnabled() )
- {
- getLogger().warn( sb.toString() );
- }
+ if ( !locale.getLanguage().equals( DEFAULT_LOCALE.getLanguage() )
+ && !i18n.getBundle( "site-tool", locale ).getLocale().getLanguage().equals(
+ locale
+ .getLanguage() ) )
+ {
+ StringBuffer sb = new StringBuffer();
+
+ sb.append( "The locale '" ).append( locale ).append( "' (" );
+ sb.append( locale.getDisplayName( Locale.ENGLISH ) );
+ sb.append( ") is not currently support by Maven - IGNORING. " );
+ sb.append( "\n" );
+ sb.append( "Contribution are welcome and greatly appreciated! " );
+ sb.append( "\n" );
+ sb.append( "If you want to contribute a new translation, please visit " );
+ sb.append( "http://maven.apache.org/plugins/maven-site-plugin/i18n.html " );
+ sb.append( "for detailed instructions." );
- continue;
+ if ( getLogger().isWarnEnabled() )
+ {
+ getLogger().warn( sb.toString() );
}
+
+ continue;
}
localesList.add( locale );