svn commit: r620110 - /maven/sandbox/trunk/plugins/maven-pdf-plugin/src/main/java/org/apache/maven/plugins/pdf/PdfMojo.java

[vsiveton]

Author: vsiveton
Date: Sat Feb  9 05:53:12 2008
New Revision: 620110

URL: http://svn.apache.org/viewvc?rev=620110&view=rev
Log:
o only write tmp file in debug mode

Modified:
    
maven/sandbox/trunk/plugins/maven-pdf-plugin/src/main/java/org/apache/maven/plugins/pdf/PdfMojo.java

Modified: 
maven/sandbox/trunk/plugins/maven-pdf-plugin/src/main/java/org/apache/maven/plugins/pdf/PdfMojo.java
URL: 
http://svn.apache.org/viewvc/maven/sandbox/trunk/plugins/maven-pdf-plugin/src/main/java/org/apache/maven/plugins/pdf/PdfMojo.java?rev=620110&r1=620109&r2=620110&view=diff
==============================================================================
--- 
maven/sandbox/trunk/plugins/maven-pdf-plugin/src/main/java/org/apache/maven/plugins/pdf/PdfMojo.java
 (original)
+++ 
maven/sandbox/trunk/plugins/maven-pdf-plugin/src/main/java/org/apache/maven/plugins/pdf/PdfMojo.java
 Sat Feb  9 05:53:12 2008
@@ -195,9 +195,7 @@
             return docRenderer.readDocumentModel( docDescriptor );
         }
 
-        File tmp = generateDefaultDocDescriptor( project );
-
-        return docRenderer.readDocumentModel( tmp );
+        return generateDefaultDocDescriptor();
     }
 
     /**
@@ -219,11 +217,10 @@
     }
 
     /**
-     * @param project not null
      * @return Generate a default document descriptor from the Maven project
      * @throws IOException if any
      */
-    private static File generateDefaultDocDescriptor( MavenProject project )
+    private DocumentModel generateDefaultDocDescriptor()
         throws IOException
     {
         File outputDir = new File( project.getBuild().getDirectory(), "pdf" );
@@ -236,9 +233,6 @@
             outputDir.mkdirs();
         }
 
-        File doc = FileUtils.createTempFile( "pdf", ".xml", outputDir );
-        doc.deleteOnExit();
-
         DocumentMeta meta = new DocumentMeta(); // TODO Improve metadata
         meta.setAuthor( ( project.getOrganization() != null
             && StringUtils.isNotEmpty( project.getOrganization().getName() ) ? 
project.getOrganization().getName()
@@ -252,18 +246,25 @@
         docModel.setOutputName( project.getArtifactId() + "-doc" );
         docModel.setMeta( meta );
 
-        DocumentXpp3Writer xpp3 = new DocumentXpp3Writer();
-        Writer w = null;
-        try
-        {
-            w = WriterFactory.newPlatformWriter( doc );
-            xpp3.write( w, docModel );
-        }
-        finally
+        if ( getLog().isDebugEnabled() )
         {
-            IOUtil.close( w );
+            File doc = FileUtils.createTempFile( "pdf", ".xml", outputDir );
+
+            getLog().debug( "Generated a default document model: " + 
doc.getAbsolutePath() );
+
+            DocumentXpp3Writer xpp3 = new DocumentXpp3Writer();
+            Writer w = null;
+            try
+            {
+                w = WriterFactory.newPlatformWriter( doc );
+                xpp3.write( w, docModel );
+            }
+            finally
+            {
+                IOUtil.close( w );
+            }
         }
 
-        return doc;
+        return docModel;
     }
 }