Author: vsiveton
Date: Mon Nov 12 05:42:02 2007
New Revision: 594124

URL: http://svn.apache.org/viewvc?rev=594124&view=rev
Log:
o using the Plexus way

Modified:
    
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/GenerateHTMLIndex.java
    
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/JSDocTask.java

Modified: 
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/GenerateHTMLIndex.java
URL: 
http://svn.apache.org/viewvc/maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/GenerateHTMLIndex.java?rev=594124&r1=594123&r2=594124&view=diff
==============================================================================
--- 
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/GenerateHTMLIndex.java
 (original)
+++ 
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/GenerateHTMLIndex.java
 Mon Nov 12 05:42:02 2007
@@ -20,6 +20,7 @@
  */
 
 import org.apache.log4j.Logger;
+import org.codehaus.plexus.logging.AbstractLogEnabled;
 
 import java.io.BufferedReader;
 import java.io.File;
@@ -37,9 +38,13 @@
  * Searches all javascript files and creates a index HTML
  * with links to documentation
  *
+ * @author <a href="mailto:[EMAIL PROTECTED]">Vincent Siveton</a>
  * @version $Id$
+ * @plexus.component role="org.apache.maven.jxr.js.doc.JSDoc" 
role-hint="default"
  */
 public class GenerateHTMLIndex
+    extends AbstractLogEnabled
+    implements JSDoc
 {
     /** Logger for this class  */
     private static final Logger log = Logger.getLogger( 
GenerateHTMLIndex.class );
@@ -48,13 +53,13 @@
 
     private File destDir;
 
-    /**
-     * @param jsDirectory
-     * @param destDir
-     * @throws IllegalArgumentException if any
-     */
-    public GenerateHTMLIndex( String jsDirectory, String destDir )
-        throws IllegalArgumentException
+    // ----------------------------------------------------------------------
+    // Public
+    // ----------------------------------------------------------------------
+
+    /** [EMAIL PROTECTED] */
+    public void generate( String jsDirectory, String destDirectory )
+        throws IllegalArgumentException, IOException
     {
         if ( jsDirectory == null )
         {
@@ -71,51 +76,43 @@
         }
         if ( js.exists() && !js.isDirectory() )
         {
-            throw new IllegalArgumentException( "JS directory is a file." );
+            throw new IOException( "JS directory is a file." );
         }
         this.jsDir = js;
 
-        if ( destDir == null )
+        if ( destDirectory == null )
         {
             throw new IllegalArgumentException( "destDir attribute can't be 
empty" );
         }
-        if ( !"/".equals( destDir.substring( destDir.length() - 1 ) ) )
+        if ( !"/".equals( destDirectory.substring( destDirectory.length() - 1 
) ) )
         {
-            destDir = destDir + "/";
+            destDirectory = destDirectory + "/";
         }
-        File dest = new File( destDir );
+        File dest = new File( destDirectory );
         if ( dest.exists() && !dest.isDirectory() )
         {
-            throw new IllegalArgumentException( "Dest directory is a file." );
+            throw new IOException( "Dest directory is a file." );
         }
         if ( !dest.exists() && !dest.mkdirs() )
         {
-            throw new IllegalArgumentException( "Cannot create the dest 
directory." );
+            throw new IOException( "Cannot create the dest directory." );
         }
         this.destDir = dest;
-    }
 
-    /**
-     *
-     * @throws IOException if any
-     */
-    public void generate()
-        throws IOException
-    {
         List files = new ArrayList();
         collectFiles( jsDir, files );
 
         Writer writer = null;
         try
         {
-            writer = new FileWriter( new File( destDir, "index.htm" ) ); // 
platform encoding
+            writer = new FileWriter( new File( this.destDir, "index.htm" ) ); 
// platform encoding
         }
         catch ( FileNotFoundException fnfe )
         {
             try
             {
-                destDir.mkdir();
-                writer = new FileWriter( new File( destDir, "index.htm" ) );
+                this.destDir.mkdir();
+                writer = new FileWriter( new File( this.destDir, "index.htm" ) 
);
             }
             catch ( FileNotFoundException e )
             {
@@ -150,7 +147,7 @@
         {
             File file = (File) files.get( i );
 
-            GenerateHTMLDoc docGenerator = new GenerateHTMLDoc( file, destDir 
);
+            GenerateHTMLDoc docGenerator = new GenerateHTMLDoc( file, 
this.destDir );
             docGenerator.generate();
         }
 

Modified: 
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/JSDocTask.java
URL: 
http://svn.apache.org/viewvc/maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/JSDocTask.java?rev=594124&r1=594123&r2=594124&view=diff
==============================================================================
--- 
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/JSDocTask.java
 (original)
+++ 
maven/sandbox/trunk/jxr/maven-jxr/maven-jxr-js/src/main/java/org/apache/maven/jxr/js/doc/JSDocTask.java
 Mon Nov 12 05:42:02 2007
@@ -46,8 +46,8 @@
     {
         try
         {
-            GenerateHTMLIndex index = new GenerateHTMLIndex( jsDir, destDir );
-            index.generate();
+            GenerateHTMLIndex index = new GenerateHTMLIndex();
+            index.generate( jsDir, destDir );
         }
         catch ( IllegalArgumentException e )
         {


Reply via email to