[MPLUGIN-470] *-mojo.xml (in PluginXdocGenerator) is overwritten when…

… multiple locales are defined
apache · May 12, 2023 · ca911b2 · ca911b2
1 parent bf04d07
commit ca911b2
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 4 deletions.
diff --git a/maven-plugin-report-plugin/src/it/plugin-report/pom.xml b/maven-plugin-report-plugin/src/it/plugin-report/pom.xml
@@ -91,7 +91,7 @@ under the License.
         <artifactId>maven-site-plugin</artifactId>
         <version>@sitePluginVersion@</version>
         <configuration>
-          <locales>en</locales>
+          <locales>en,de,fr</locales>
         </configuration>
       </plugin>
     </plugins>

diff --git a/maven-plugin-report-plugin/src/it/plugin-report/verify.groovy b/maven-plugin-report-plugin/src/it/plugin-report/verify.groovy
@@ -28,3 +28,11 @@ assert !pluginInfo.text.contains('Disk Space')
 // check JDK and Maven requirements
 assert pluginInfo.text.contains('1.8')
 assert pluginInfo.text.contains('3.2.5')
+
+assert new File( basedir, 'target/site/de/noop-mojo.html' ).isFile()
+assert new File( basedir, 'target/site/de/report-mojo.html' ).isFile()
+assert new File( basedir, 'target/site/de/plugin-info.html' ).isFile()
+
+assert new File( basedir, 'target/site/fr/noop-mojo.html' ).isFile()
+assert new File( basedir, 'target/site/fr/report-mojo.html' ).isFile()
+assert new File( basedir, 'target/site/fr/plugin-info.html' ).isFile()
diff --git a/...lugin-report-plugin/src/main/java/org/apache/maven/plugin/plugin/report/PluginReport.java b/...lugin-report-plugin/src/main/java/org/apache/maven/plugin/plugin/report/PluginReport.java
@@ -31,6 +31,7 @@
 import java.util.ResourceBundle;
 
 import org.apache.maven.doxia.sink.Sink;
+import org.apache.maven.doxia.tools.SiteTool;
 import org.apache.maven.model.Plugin;
 import org.apache.maven.model.Prerequisites;
 import org.apache.maven.plugin.descriptor.MojoDescriptor;
@@ -76,8 +77,8 @@ public class PluginReport extends AbstractMavenReport {
      *
      * @since 3.7.0
      */
-    @Parameter(defaultValue = "${project.build.directory}/generated-site/xdoc")
-    private File outputDirectory;
+    @Parameter(defaultValue = "${project.build.directory}/generated-site")
+    private File generatedSiteDirectory;
 
     /**
      * Set this to "true" to skip generating the report.
@@ -222,7 +223,12 @@ public String getOutputName() {
     private void generateMojosDocumentation(PluginDescriptor pluginDescriptor, Locale locale)
             throws MavenReportException {
         try {
-            File outputDir = outputDirectory;
+            File outputDir;
+            if (!locale.equals(SiteTool.DEFAULT_LOCALE)) {
+                outputDir = new File(new File(generatedSiteDirectory, locale.toString()), "xdoc");
+            } else {
+                outputDir = new File(generatedSiteDirectory, "xdoc");
+            }
             outputDir.mkdirs();
 
             PluginXdocGenerator generator = new PluginXdocGenerator(