Author: vsiveton
Date: Tue Jan 29 15:02:59 2008
New Revision: 616553

URL: http://svn.apache.org/viewvc?rev=616553&view=rev
Log:
MPIR-81: DependenciesRenderer logs Exception when it can't find the artifact 
being built!!!

o log a warning if the artifact of the current project has never been deployed

Modified:
    
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
    
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java

Modified: 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java?rev=616553&r1=616552&r2=616553&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
 (original)
+++ 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/Dependencies.java
 Tue Jan 29 15:02:59 2008
@@ -41,6 +41,8 @@
  */
 public class Dependencies
 {
+    private final MavenProject project;
+
     private final List projectDependencies;
 
     private final DependencyTree dependencyTree;
@@ -49,6 +51,7 @@
 
     public Dependencies( MavenProject project, DependencyTree dependencyTree, 
JarClassesAnalysis classesAnalyzer )
     {
+        this.project = project;
         this.dependencyTree = dependencyTree;
         this.projectDependencies = dependencyTree.getRootNode().getChildren();
         this.classesAnalyzer = classesAnalyzer;
@@ -65,6 +68,16 @@
         }
 
         mapArtifactFiles( dependencyTree.getRootNode(), projectMap );
+    }
+
+    /**
+     * Getter for the project
+     *
+     * @return the project
+     */
+    public MavenProject getProject()
+    {
+        return project;
     }
 
     public boolean hasDependencies()

Modified: 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java?rev=616553&r1=616552&r2=616553&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
 (original)
+++ 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
 Tue Jan 29 15:02:59 2008
@@ -345,7 +345,17 @@
                     }
                     catch ( ArtifactNotFoundException e )
                     {
-                        log.error( "Artifact: " + artifact.getId() + " has no 
file.", e );
+                        if ( ( dependencies.getProject().getGroupId().equals( 
artifact.getGroupId() ) )
+                            && ( 
dependencies.getProject().getArtifactId().equals( artifact.getArtifactId() ) )
+                            && ( 
dependencies.getProject().getVersion().equals( artifact.getVersion() ) ) )
+                        {
+                            log.warn( "The artifact of this project has never 
been deployed." );
+                        }
+                        else
+                        {
+                            log.error( "Artifact: " + artifact.getId() + " has 
no file.", e );
+                        }
+
                         continue;
                     }
                 }


Reply via email to