elharo commented on code in PR #198:
URL: https://github.com/apache/maven-site-plugin/pull/198#discussion_r1685793015


##########
src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java:
##########
@@ -355,7 +354,7 @@ protected Map<String, MavenReport> locateReports(
         for (MavenReportExecution mavenReportExecution : reports) {
             MavenReport report = mavenReportExecution.getMavenReport();
 
-            String outputName = report.getOutputName();
+            String outputName = report.getOutputPath();

Review Comment:
   Here's what I see in MavenReport for getOutputName:
   
   ```
       /**
        * Get the path relative to {@link #getReportOutputDirectory()} where 
the report's main output
        * file will be written. The last component is the name of the file 
without any extension. The
        * actual output extension will by added by the sink implementation.
        * <p>
        * Note: This method won't be {@code default} anymore when {@link 
#getOutputName()} is removed.
        * You are advised to implement it as soon as possible.
        *
        * @since 4.0.0
        * @return the relative output path of this report
        */
       default String getOutputPath() {
           return getOutputName();
       }
   
       /**
        * @deprecated Method name does not properly reflect its purpose. 
Implement and use
        * {@link #getOutputPath()} instead.
        */
       @Deprecated
       String getOutputName();
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to