svn commit: r385567 - in /maven/plugins/branches/site-refactor: maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ maven-site-plugin/src/main/java/org/apache/maven/plugins/site/

[br...@apache.org]

Author: brett
Date: Mon Mar 13 07:02:10 2006
New Revision: 385567

URL: http://svn.apache.org/viewcvs?rev=385567&view=rev
Log:
[MSITE-80] clean remaining task list

Modified:
    maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectIndexPageReport.java
    maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectInfoSummaryReport.java
    maven/plugins/branches/site-refactor/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteMojo.java

Modified: maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectIndexPageReport.java
URL: http://svn.apache.org/viewcvs/maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectIndexPageReport.java?rev=385567&r1=385566&r2=385567&view=diff
==============================================================================
--- maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectIndexPageReport.java (original)
+++ maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectIndexPageReport.java Mon Mar 13 07:02:10 2006
@@ -33,7 +33,7 @@
  * @author <a href="mailto:vincent.siveton@gmail.com">Vincent Siveton </a>
  * @version $Id$
  * @goal index
- * @todo have not moved keys correctly in translations yet
+ * @todo [IMPORTANT] have not moved keys correctly in translations yet
  */
 public class ProjectIndexPageReport
     extends AbstractMavenReport

Modified: maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectInfoSummaryReport.java
URL: http://svn.apache.org/viewcvs/maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectInfoSummaryReport.java?rev=385567&r1=385566&r2=385567&view=diff
==============================================================================
--- maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectInfoSummaryReport.java (original)
+++ maven/plugins/branches/site-refactor/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/ProjectInfoSummaryReport.java Mon Mar 13 07:02:10 2006
@@ -35,9 +35,9 @@
  * @author <a href="mailto:vincent.siveton@gmail.com">Vincent Siveton </a>
  * @version $Id$
  * @goal project-info-summary
- * @todo this is the same for each category, should juts be one renderer. Maybe put it in Doxia instead, and instantiate directly from the site plugin?
- * @todo need to have a key prefix - report.information (Project Infos) vs report.project (Project Reports)
- * @todo have not moved keys correctly in translations yet
+ * @todo [IMPORTANT] this is the same for each category, should juts be one renderer. Maybe put it in Doxia instead, and instantiate directly from the site plugin?
+ * @todo [IMPORTANT] need to have a key prefix - report.information (Project Infos) vs report.project (Project Reports)
+ * @todo [IMPORTANT] have not moved keys correctly in translations yet
  */
 public class ProjectInfoSummaryReport
     extends AbstractMavenReport

Modified: maven/plugins/branches/site-refactor/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteMojo.java
URL: http://svn.apache.org/viewcvs/maven/plugins/branches/site-refactor/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteMojo.java?rev=385567&r1=385566&r2=385567&view=diff
==============================================================================
--- maven/plugins/branches/site-refactor/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteMojo.java (original)
+++ maven/plugins/branches/site-refactor/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/SiteMojo.java Mon Mar 13 07:02:10 2006
@@ -119,7 +119,7 @@
     public void execute()
         throws MojoExecutionException, MojoFailureException
     {
-        // TODO: use in :run as well? better way to call it?
+        // TODO [IMPORTANT]: use in :run as well? better way to call it?
         setDefaultAttributes();
 
         List reports = filterReports( this.reports );
@@ -224,8 +224,6 @@
             }
 
         }
-
-        // TODO: [IMPORTANT] Be good to generate a module's summary page thats referenced off the Modules menu item.
 
         siteRenderer.render( documents.values(), context, outputDirectory, outputEncoding );
     }