[MJAVADOC-694] Avoid empty warn message from getResolvePathResult

slawekjaranowski · slawekjaranowski · commit b51c5d801c1e · 2021-12-20T21:59:32.000+01:00
diff --git a/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java b/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
@@ -5543,12 +5543,16 @@ private ResolvePathResult getResolvePathResult( File artifactFile )
         }
         catch ( IOException | RuntimeException /* e.g java.lang.module.FindException */ e )
         {
-            Throwable cause = e;
-            while ( cause.getCause() != null )
+            if ( getLog().isDebugEnabled() )
             {
-                cause = cause.getCause();
+                Throwable cause = e;
+                while ( cause.getCause() != null )
+                {
+                    cause = cause.getCause();
+                }
+
+                getLog().debug( "resolve path for: " + artifactFile + " cause error: " + cause );
             }
-            getLog().warn( e.getMessage() );
         }
         return resolvePathResult;
     }

Original file line number	Diff line number	Diff line change
`@@ -5543,12 +5543,16 @@ private ResolvePathResult getResolvePathResult( File artifactFile )`
`5543`	`5543`	`}`
`5544`	`5544`	`catch ( IOException \| RuntimeException /* e.g java.lang.module.FindException */ e )`
`5545`	`5545`	`{`
`5546`		`- Throwable cause = e;`
`5547`		`- while ( cause.getCause() != null )`
	`5546`	`+ if ( getLog().isDebugEnabled() )`
`5548`	`5547`	`{`
`5549`		`- cause = cause.getCause();`
	`5548`	`+ Throwable cause = e;`
	`5549`	`+ while ( cause.getCause() != null )`
	`5550`	`+ {`
	`5551`	`+ cause = cause.getCause();`
	`5552`	`+ }`
	`5553`	`+`
	`5554`	`+ getLog().debug( "resolve path for: " + artifactFile + " cause error: " + cause );`
`5550`	`5555`	`}`
`5551`		`- getLog().warn( e.getMessage() );`
`5552`	`5556`	`}`
`5553`	`5557`	`return resolvePathResult;`
`5554`	`5558`	`}`