Author: vmassol
Date: Tue Jul 26 03:44:31 2005
New Revision: 225271

URL: http://svn.apache.org/viewcvs?rev=225271&view=rev
Log:
MNG-628: No more clover.html + no more site resources generated. Done this time 
by extending AbstractMavenReport and overriding it's main methods (generate() 
and execute()). AbstractMavenReport would need to add native support for 
external reports to have something clean (see MNG-645)

Modified:
    
maven/components/trunk/maven-plugins/maven-clover-plugin/src/main/java/org/apache/maven/plugin/clover/CloverReportMojo.java

Modified: 
maven/components/trunk/maven-plugins/maven-clover-plugin/src/main/java/org/apache/maven/plugin/clover/CloverReportMojo.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-clover-plugin/src/main/java/org/apache/maven/plugin/clover/CloverReportMojo.java?rev=225271&r1=225270&r2=225271&view=diff
==============================================================================
--- 
maven/components/trunk/maven-plugins/maven-clover-plugin/src/main/java/org/apache/maven/plugin/clover/CloverReportMojo.java
 (original)
+++ 
maven/components/trunk/maven-plugins/maven-clover-plugin/src/main/java/org/apache/maven/plugin/clover/CloverReportMojo.java
 Tue Jul 26 03:44:31 2005
@@ -39,7 +39,7 @@
  * @goal report
  * @execute phase="test" lifecycle="clover"
  */
-public class CloverReportMojo extends AbstractMojo implements MavenReport
+public class CloverReportMojo extends AbstractMavenReport
 {
     /**
      * @parameter expression="${project.build.directory}/clover/clover.db"
@@ -94,7 +94,7 @@
 
     public String getOutputName()
     {
-        return "clover";
+        return "clover/index";
     }
 
     /**
@@ -142,41 +142,11 @@
         return getBundle( locale ).getString( "report.clover.name" );
     }
 
-    // The methods below are required because we don't extend 
AbstractMavenReport. The reason is that
-    // AbstractMavenReport does not support externally generated HTML report 
files.
-
-    /**
-     * @see org.apache.maven.reporting.MavenReport#getReportOutputDirectory()
-     */
-    public File getReportOutputDirectory()
-    {
-        if ( this.reportOutputDirectory == null )
-        {
-            this.reportOutputDirectory = new File( getOutputDirectory() );
-        }
-        return this.reportOutputDirectory;
-    }
-
     /**
-     * @see MavenReport#setReportOutputDirectory(java.io.File)
+     * @see 
org.apache.maven.reporting.MavenReport#generate(org.codehaus.doxia.sink.Sink, 
java.util.Locale)
      */
-    public void setReportOutputDirectory( File reportOutputDirectory )
-    {
-        this.reportOutputDirectory = reportOutputDirectory;
-    }
-
-    /**
-     * @see org.apache.maven.reporting.MavenReport#getCategoryName()
-     */
-    public String getCategoryName()
-    {
-        return CATEGORY_PROJECT_REPORTS;
-    }
-
-    /**
-     * @see MavenReport#generate(org.codehaus.doxia.sink.Sink, 
java.util.Locale)
-     */
-    public void generate(Sink sink, Locale locale) throws MavenReportException
+    public void generate( Sink sink, Locale locale )
+        throws MavenReportException
     {
         executeReport( locale );
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to